On Sep 30, 2011, at 2:25 PM, Daniel Dunbar wrote:

> To be honest, I had forgotten we already had implemented that.
> 
> I was being a bit lazy here (ok maybe more than a bit), I just wanted
> to be able to test out the impact of being able to do the chdir() in
> process and avoiding an extra process spawn. To properly fix this to
> use -working-directory= (and guarantee no behavior change) will take a
> bit more work, as there are:
> (a) a few places in the driver that access the file system.

These sound like bugs to be fixed.

> (b) the driver will have to do something to honor that option when it
> is calling out to other processes (like the linker), or be able to
> adjust paths.

I think it's entirely reasonable to temporarily chdir() when calling those 
other processes.

> I think for now I'm going to just revert my patch and think about this more.

Okay.

        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to