Re: [Scons-dev] Hummm... rant on

2014-03-03 Thread Bill Deegan
I'll take a look to see how much work it would be, and then let the group know. I see the following as a reasonable way. Tools query some package for valid path values. That package can check for: SCONS_TOOLS_PATH And/or provide a way for SConsctrucs and site_init.py to set the PATH(s) -Bill On

Re: [Scons-dev] Hummm... rant on

2014-03-03 Thread Russel Winder
On Mon, 2014-03-03 at 06:04 -0500, Gary Oberbrunner wrote: > On Mon, Mar 3, 2014 at 2:35 AM, Russel Winder wrote: > > > Having an entry 'test_base_path = '...' in a ~/.scons/settings.py or > > something similar would work fine for me. This only alters the behaviour > > for tests, not for normal e

Re: [Scons-dev] Hummm... rant on

2014-03-03 Thread Gary Oberbrunner
On Mon, Mar 3, 2014 at 2:35 AM, Russel Winder wrote: > Having an entry 'test_base_path = '...' in a ~/.scons/settings.py or > something similar would work fine for me. This only alters the behaviour > for tests, not for normal execution. > +1 -- though I'd suggest it be called test_settings.py a

Re: [Scons-dev] Hummm... rant on

2014-03-02 Thread Russel Winder
On Sun, 2014-03-02 at 12:37 -0800, Bill Deegan wrote: […] I don't remember that episode, but I can imagine. I guess we need 2 VMs per platform, one pure "out of the box", and one "with all the tools supported in the distribution". All the other combinations would amount to needing a near infinite

Re: [Scons-dev] Hummm... rant on

2014-03-02 Thread Russel Winder
On Sun, 2014-03-02 at 14:55 -0500, Gary Oberbrunner wrote: > As far as I understand it, env.Detect calls env.WhereIs with path=None, > which means it uses env['ENV']['PATH']. By default, a new Environment > doesn't inherit the user's shell environment into env['ENV'], but users can > modify it as

Re: [Scons-dev] Hummm... rant on

2014-03-02 Thread Bill Deegan
That's cruft leftover from Greg Noel. I actually made a change many moons ago so you're test environment could be setup based on your PATH, thus reducing the number of VM's needed to test different combinations of tools. It gets ridiculous really fast otherwise. Greg pulled it all out with some cu

Re: [Scons-dev] Hummm... rant on

2014-03-02 Thread Gary Oberbrunner
As far as I understand it, env.Detect calls env.WhereIs with path=None, which means it uses env['ENV']['PATH']. By default, a new Environment doesn't inherit the user's shell environment into env['ENV'], but users can modify it as they want. So in a real user's code, the path Detect will use depe

[Scons-dev] Hummm… rant on

2014-03-02 Thread Russel Winder
I think I have ranted about this before, but the red haze must have been so strong I failed to note the well reasoned responses you folk will have provided. The core problem is that TestSCons.TestSCons().where_is(x) uses the user's path for executable look up whereas Environment.Detect(x) does not