[Scons-users] SConf Configure log_file argument ignores dir path

[Daniel Moody]

Yeah my change did not pass the tests:
https://travis-ci.org/dmoody256/scons/builds/311674594

I will compare differences and try to isolate the issue in my setup.

On Dec 5, 2017 12:19 AM, "Bill Deegan" <bill at baddogconsulting.com> wrote:

> Your sample is very similar to
>
> test/Configure/VariantDir.py
> Test configure logic.
>
> Here:
> env = Environment(LOGFILE='build/config.log')
> import os
> env.AppendENVPath('PATH', os.environ['PATH'])
> VariantDir( 'build', '.' )
> conf = env.Configure(conf_dir='build/config.tests', log_file='$LOGFILE')
> r1 = conf.CheckCHeader( 'math.h' )
> r2 = conf.CheckCHeader( 'no_std_c_header.h' ) # leads to compile error
> env = conf.Finish()
> Export( 'env' )
> # print open( 'build/config.log' ).readlines()
> SConscript( 'build/SConscript' )
>
>
>
> Does your change break any tests?
>
> Can you make a simple test which duplicates your issue?
> -Bill
>
> On Mon, Dec 4, 2017 at 8:39 PM, Daniel Moody <dmoody256 at gmail.com> wrote:
>
>> I am using Configure in this manner:
>>
>> conf = Configure(env, conf_dir = env.build_dir + "/conf_tests", log_file
>> = env.build_dir + "/conf.log",
>>             custom_tests = {...
>>
>> where env.build_dir is 'build' in this case, but conf.log appears in the
>> same directory as SConstruct.
>>
>> I would expect the logfile to use my path for the location of the log
>> file as conf_dir is doing so I think this is a bug.
>>
>> I found that if I change this:
>> https://github.com/dmoody256/scons/commit/d83e80a45feeca8d93
>> 2e7d2f97a804cb6035c546
>>
>> then it works as I expected. If this seems to be the intended function I
>> will submit a bug and PR with a test.
>>
>> any can confirm this bug?
>>
>>
>>
>> _______________________________________________
>> Scons-users mailing list
>> Scons-users at scons.org
>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>>
>>
>
> _______________________________________________
> 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/20171205/adb90e2f/attachment-0001.html>