[issue7742] please avoid 'which' in Modules/ld_so_aix

Stefan Krah added the comment: Michael Haubenwallner wrote: > @Stefan: Nope, setting some PATH in ~/.cshrc is the wrong thing to do, and it > actually is the reason for this issue at all: > 'which' is intended to tell the location of some command that would be used > when started on current c

[issue7742] please avoid 'which' in Modules/ld_so_aix

Michael Haubenwallner added the comment: @Stefan: Nope, setting some PATH in ~/.cshrc is the wrong thing to do, and it actually is the reason for this issue at all: 'which' is intended to tell the location of some command that would be used when started on current commandline or within current

[issue7742] please avoid 'which' in Modules/ld_so_aix

Stefan Krah added the comment: Sridhar, thanks for the information. /usr/bin/which is a shell script on Linux, too, but it does not source any config files. I think the behavior is odd, but as you say, it can be worked around. In the worst case, one can set the PATH in ~/.cshrc to a sane PATH.

[issue7742] please avoid 'which' in Modules/ld_so_aix

Sridhar Ratnakumar added the comment: On 2010-07-22, at 11:56 AM, Stefan Krah wrote: > > Stefan Krah added the comment: > > I'm not sure this should be changed: > > 1) Why is .cshrc sourced? It should only get sourced for a login > shell. /usr/bin/which is a csh script on our AIX mach

[issue7742] please avoid 'which' in Modules/ld_so_aix

Stefan Krah added the comment: I'm not sure this should be changed: 1) Why is .cshrc sourced? It should only get sourced for a login shell. 2) If the user sets a PATH that excludes the compiler, then `which` will also not find the compiler on other systems (like Linux). Why

[issue7742] please avoid 'which' in Modules/ld_so_aix

Mark Lawrence added the comment: @Michael: do you intend putting in a new patch for review as hinted at in msg98071? -- nosy: +BreamoreBoy ___ Python tracker ___ ___

[issue7742] please avoid 'which' in Modules/ld_so_aix

Changes by Brian Curtin : -- components: +None keywords: +needs review priority: -> normal stage: -> patch review type: -> behavior versions: +Python 2.7 ___ Python tracker ___

[issue7742] please avoid 'which' in Modules/ld_so_aix

Michael Haubenwallner added the comment: Uhm, as I read the patch again, there's room for improvement of pywhich(): it likely fails with CC=/full/path/to/cc -- ___ Python tracker ___

[issue7742] please avoid 'which' in Modules/ld_so_aix

New submission from Michael Haubenwallner : In Modules/ld_so_aix there is `which $CC` to search for the full compiler's path. Unfortunately, /usr/bin/which on AIX is a csh-script and thus ~/.cshrc gets sourced before doing the path search. Now, when the user does set some PATH in ~/.cshrc, `w