[Nix-dev] Re: [PATCH 1/2] t/scons
Rob Vermaas
rob.vermaas at gmail.com
Fri Dec 24 20:32:36 CET 2010
Hi,
> Initially I wrote the patch for v8 (js engine). Meanwhile Rob packaged
> it differently. So the whole patch has no use case at
> the moment. So we can stop talking about it.
>
> He solved it this way:
> scons snapshot=on importenv=PATH arch=${arch}
>
> So it looks like this workaround should be documented at very least so
> that it can be found faster. A comment in the scons expression is a nice
> place IMHO.
>
> Maybe Rob can talk about how long it took him to find that fix using
> importenv=PATH. I didn't knew about it. I still think that tools should
> "just work".
As I cannot really remember much about it, I doubt it took me a long
time to find it on google somewhere ;-)
I don't really have a strong opinion on this issue, although I think I
Peter's solution on using the unaltered scons as default, and adding a
wrapper/alternativefor people wanting the 'patched' behavior.
gr.
Rob
--
Rob Vermaas
[email] rob.vermaas at gmail.com
More information about the nix-dev
mailing list