[Scons-users] scons 3.0.0 bug: Help() and site_tools

[Bill Deegan]

o.k found the issue.
The py3.5+ code path didn't restore the sys.path it changed to load a new
Tool.
Testing patch.

_Bill

On Thu, Oct 12, 2017 at 7:48 PM, Bill Deegan <bill at baddogconsulting.com>
wrote:

> Clearly lots of repeated paths there, but does it cause any actual
> functional issue?
>
> The logic for importing tools is different for 2.7 and 3.5+.
> It seems likely you're seeing:
> '/path/to/src//site_scons/site_tools',
>
> Once for each tool loaded from site_tools.
>
> Does that sound right?
>
> -Bill
>
> On Thu, Oct 12, 2017 at 12:40 AM, Bassem Girgis <brgirgis at gmail.com>
> wrote:
>
>> Hi All,
>>
>> I have found another issue with scons 3.0.0 and python3. This issue is
>> visible if you have a simple project that uses both the Help() function and
>> also have a site_scons/site_tools directory. It is not there with python
>> 2.7.13.
>>
>> As you can see below the path to site_tools is added many times
>> unnecessarily.
>>
>> scons: Reading SConscript files ...
>> session python path:['/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src//site_scons/site_tools',
>>                           '/path/to/src/',
>>                           '/path/to/src//site_scons',
>>                           '/usr/bin/../engine',
>>                           '/usr/bin/scons-local-3.0.0',
>>                           '/usr/bin/scons-local',
>>                           '/usr/lib/scons-3.0.0',
>>                           '/usr/lib/scons-3.0.0',
>>                           '/usr/local/lib/scons-3.0.0',
>>                           '/usr/lib/python3.6/site-packages/scons-3.0.0',
>>                           '/usr/lib/python3.6/site-packages/scons-3.0.0',
>>                           '/usr/local/lib/python3.6/site
>> -packages/scons-3.0.0',
>>                           '/usr/lib64/scons-3.0.0',
>>                           '/usr/lib/scons',
>>                           '/usr/lib/scons',
>>                           '/usr/local/lib/scons',
>>                           '/usr/lib/python3.6/site-packages/scons',
>>                           '/usr/lib/python3.6/site-packages/scons',
>>                           '/usr/local/lib/python3.6/site-packages/scons',
>>                           '/usr/lib64/scons',
>>                           '/usr/lib/python36.zip',
>>                           '/usr/lib64/python3.6',
>>                           '/usr/lib64/python3.6/lib-dynload',
>>                           '/home/username/.local/lib/pyt
>> hon3.6/site-packages',
>>                           '/usr/lib64/python3.6/site-packages',
>>                           '/usr/lib64/python3.6/site-packages/PIL',
>>                           '/usr/lib64/python3.6/_import_failed',
>>                           '/usr/lib/python3.6/site-packages']
>> scons: done reading SConscript files.
>> scons: Building targets ...
>> scons: `.' is up to date.
>> scons: done building targets.
>>
>> Please find attached a simple project that reproduces the issue.
>>
>> Bassem Girgis, PhD
>>
>> Email: brgirgis at gmail.com
>>
>> _______________________________________________
>> Scons-users mailing list
>> Scons-users at scons.org
>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist4.pair.net/pipermail/scons-users/attachments/20171012/863aa29a/attachment-0001.html>