[Nix-dev] Re: [PATCH 1/2] t/scons

[Rob Vermaas]

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