Re: [Boost-build] Patch to fix dependency properties in project requirements

From: Rene Rivera (grafikrobot_at_[hidden])
Date: 2008-07-19 12:12:29

Jurko Gospodnetic' wrote:
>   Hi Rene.
> 
>> Steven Watanabe wrote:
>>> The attached patch fixes the problem described at
>>> http://zigzag.cs.msu.su/boost.build/ticket/13.
>>
>> I'm assuming the patch matches this changeset 
>> <http://svn.boost.org/trac/boost/changeset/47536>. If so, it so 
>> happens that it breaks use of source targets specified as absolute paths.
> 
>   I've reopened the corresponding ticket 
> (https://zigzag.lvk.cs.msu.su:7813/boost.build/ticket/13).
> 
>   I looked at the problems this patch causes a bit and found problems 
> with two types of dependency features:
>     1. Those whose values are given as absolute paths.
>     2. Those whose values identify targets in the same project.
> 
>   Is it ok if I commit back Steven's Boost Build tests? I'll also add 
> try to add the above test cases to it.

Sure, I blindly reverted the whole changeset, because of lack of time to 
do anything else. It would be good to have tests for all the additional 
failures we encountered for this one. So that we can make sure we don't 
break it the next time.

>   Any thoughts on this?

Just to add my use case to it:

   project another ;
   local location = C:/some/path ;
   lib some_module : [ glob $(location)/src/*.cpp ] ;

   project mine ;
   exe my_exe : main.cpp /another//some_module ;

Which would produce errors about missing sources of the form 
"/another//some_module/C:/some/path/src/foobar.cpp".

-- 
-- Grafik - Don't Assume Anything
-- Redshift Software, Inc. - http://redshift-software.com
-- rrivera/acm.org (msn) - grafik/redshift-software.com
-- 102708583/icq - grafikrobot/aim,yahoo,skype,efnet,gmail