|
|
Boost Testing : |
From: Beman Dawes (bdawes_at_[hidden])
Date: 2008-06-28 08:46:47
Rene Rivera wrote:
> Richard Webb wrote:
>> Nicola Musatti <Nicola.Musatti <at> gmail.com> writes:
>>
>>> Is there a way to look at the process_jam_log command line that
>>> causes the
>>> problem? I'll try and perform a regression run to see what is going
>>> wrong,
>>> but
>>> unfortunately I won't be able to do it before sunday or monday.
>>>
>>> Cheers,
>>> Nicola Musatti
>>>
>>
>> The command line is:
>>
>> E:\test\boost\status>"E:\test\tools_regression\build\bin\msvc-7.1\release\threading-multi\process_jam_log.exe"
>>
>> "E:\test\results" 0<"E:\test\results\bjam.log"
>>
>> I had a look at process_jam_log.cpp, and i think the problem might be
>> in the
>> target_directory() function:
>> The full pjl log contains things like:
>>
>> line 3244: compile-c-c++
>> E:\test\results\boost\bin.v2\libs\regex\build\msvc-7.1\debug\threading-multi\c_regex_traits.obj
>>
>> line_start: compile-c-c++
>> target_directory( "compile-c-c++
>> E:\test\results\boost\bin.v2\libs\regex\build\msvc-7.1\debug\threading-multi\c_regex_traits.obj")
>>
>> ->
>> "E:/test/results/boost/bin.v2/libs/regex/build/msvc-7.1/debug/threading-multi"
>>
>> *****Warning - can't open output file:
>> E:/test/results/E:/test/results/boost/bin.v2/libs/array/test/array0.test/msvc-7.1/debug/threading-multi/test_log.xml
>>
>>
>> One of the recent changes in pjl was to change line 158 from
>>
>> //
>> else temp.erase( 0, locate_root.string().size()+1 );
>> //
>>
>> to
>>
>> //
>> else if ( temp[0] == '/' ) temp.erase( 0,
>> locate_root.string().size()+1 );
>> //
>>
>> I tried changing it back, and that seems to have fixed it.
>
> Many, many, thanks for finding that! I've gone ahead and reverted that
> line.
Thanks, Rene.
That line was part of a patch from Nicola Musatti. Nicola, what was that
change trying to accomplish?
--Beman