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

[Peter Simons]

Hi Marc,

this patch changes SCons' documented behavior. The authors of the tool have
made the conscious choice to ignore the user's shell environment, as the manual
explains:

 | scons does not automatically propagate the external environment used to
 | execute scons to the commands used to build target files. This is so that
 | builds will be guaranteed repeatable regardless of the environment variables
 | set at the time scons is invoked.

This is a sensible policy, and one that is very much in line with the
principles of Nix, i.e. the fact that builds ought to be deterministic and
repeatable. Please do not change that behavior. (meaning: please revert the
commit)

IMHO, it is fine to add that patch to the expression as an optional feature
that is disabled by default. I'm also fine with introducing a separate package
for this, say scons-wrapper, or something.

Take care,
Peter