Fixing cdf in pos.rb #7799

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@dsanson
Contributor
dsanson commented Sep 24, 2011

The current pos.rb formula creates a shell script, cdf, containing

#!/bin/bash
cd "$(posd)"

The idea is that this script should change the terminal directory to match the current finder directory, as reported by posd. But this doesn't work, since it only changes the directory within the subshell spawned by the script.

The right thing to do is to add an alias or function to one's .bash_profile (or equivalent for other shells).

So I've deleted the bits of the formula that generate the cdf script, and I've added a caveats section with instructions for adding an appropriate alias to .bash_profile.

@Sharpie Sharpie added a commit that closed this pull request Sep 24, 2011
@dsanson @Sharpie dsanson + Sharpie Pos: Remove non-functional cdf command
Closes #7799.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
9883e3c
@Sharpie Sharpie closed this in 9883e3c Sep 24, 2011
@Sharpie
Contributor
Sharpie commented Sep 24, 2011

Makes sense to me. Thanks for the tweaks and cleanup!

@dewback dewback added a commit to dewback/homebrew that referenced this pull request Sep 26, 2011
@dsanson @dewback dsanson + dewback Pos: Remove non-functional cdf command
Closes #7799.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
aff9c6e
@martinploeger martinploeger added a commit to martinploeger/homebrew that referenced this pull request Oct 8, 2011
@dsanson @martinploeger dsanson + martinploeger Pos: Remove non-functional cdf command
Closes #7799.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
7adcd4f
@Sharpie Sharpie added a commit to Sharpie/homebrew that referenced this pull request Sep 12, 2012
@dsanson @Sharpie dsanson + Sharpie Pos: Remove non-functional cdf command
Closes #7799.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
a0d260b
@snakeyroc3 snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request Dec 17, 2012
@dsanson @Sharpie dsanson + Sharpie Pos: Remove non-functional cdf command
Closes #7799.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
adc9512
@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.