Subject: Re: CVS commit: basesrc/usr.bin/make
To: Todd Vierling <tv@wasabisystems.com>
From: Christos Zoulas <christos@zoulas.com>
List: source-changes
Date: 11/02/2001 15:26:27
On Nov 2,  2:11pm, tv@wasabisystems.com (Todd Vierling) wrote:
-- Subject: Re: CVS commit: basesrc/usr.bin/make

| On Fri, 2 Nov 2001, Christos Zoulas wrote:
| 
| I have a feeling that you're referring to getenv("PWD") being used for other
| purposes, though -- would you mind pointing out what breaks by removing
| this?  I can imagine a case where a builder may want .depend files to work
| on multiple systems with different automount paths, but that's still going
| to be afflicted by the fragility of PWD, and would be *much* safer using an
| explicit MAKEOBJDIR :C,,, translation.

With the automounter you end up in the physical path, that you are not
supposed to be using directly [under amd typically /a/machine/dir/blah].
Now, pmake decides to run on a different machine and it remotely asks
for a job to be forked in /a/machine/dir/blah. This might not be currently
mounted in which case you will lose. You need to leave it in, at least if
REMOTE is defined. Feel free to add the above explanation why it is needed.

I am not aware of a shell that does not set PWD.

christos