Mtdhanmani / git-osx-installer

Automatically exported from code.google.com/p/git-osx-installer
0 stars 0 forks source link

setup git PATH for non-terminal programs.sh should use /usr/libexec/path_helper #48

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. run "setup git PATH for non-terminal programs.sh"

What is the expected output? What do you see instead?
It creates a .MacOSX/environment.plist file that has a path that does not match 
the output of 
/usr/libexec/path_helper. In particular, it includes /opt/local/bin which is 
not always used. I 
personally use homebrew for package management and put it in /opt/homebrew. 
This script should 
just be a little more aware of the user environment. path_helper is a more 
accurate way to do it (in 
fact, git installer correctly adds itself to the path via /etc/paths.d)

What version of the product are you using? On what operating system?
1.6.6.2 Intel Leopard on Mac OS X 10.6.2

Original issue reported on code.google.com by ascar...@gmail.com on 13 Feb 2010 at 11:33

GoogleCodeExporter commented 8 years ago
To be fair, Mac OS X non-terminal programs, by default, don't respect the 
environment set up by path_helper.

It seems what you are asking for is to add git, PLUS the rest of your bash 
environment, to your Mac OS X non-terminal PATH environment.  Is that right?

Granted... it appears the script is tacking on an /opt/local/bin, so it's 
already doing a little more than it should.

Original comment by timchar...@gmail.com on 9 Dec 2010 at 10:26

GoogleCodeExporter commented 8 years ago

Original comment by timchar...@gmail.com on 4 Sep 2011 at 4:42

GoogleCodeExporter commented 8 years ago
I get a syntaxerror from the script??? Expect end of line but finds "/" (read 
backslash)

Original comment by hebhan...@gmail.com on 27 Jun 2013 at 3:49

GoogleCodeExporter commented 8 years ago
Crome, Safri,Jungledrums, why we can not work togther

Original comment by 256MainS...@gmail.com on 10 Dec 2013 at 4:33