Re: [jamboost] Re: [V2] Setting project defaults for different toolsets

From: Vladimir Prus (ghost_at_[hidden])
Date: 2003-11-21 09:24:43

Leaf Garland wrote:

> But this doesn't:
>
> project
>
> : requirements
>
> <toolset>msvc:<include>"d:/dev/" ;
> exe pdtest : test.cpp : <toolset>msvc ;
>
> In this case the compiler command line doesn't have -I"d:/dev/" on it and
> the included file is not found.
>
> Is that the same for you?

Yes, unfortunately. I start to think that the algorithm used to construct 
build properties needs to be reviewed for such corner-cases (and also 
documented).

But that will take some time... so let me try some solution which can work 
now... oh, it turns out there was a blatant bug.

I've committed a fix for it, and you can get it either from CVS, when it's 
there, or apply the attached patch, or get 

http://zigzag.cs.msu.su:7813/working_copy/v2/build/property.jam

directly.

Thanks,
Volodya
 --Boundary-00=_rAiv/QjgXJcqUQg Content-Type: text/x-diff;
charset="iso-8859-1";
name="property.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
filename="property.diff"

Index: property.jam
===================================================================
RCS file: /cvsroot/boost/boost/tools/build/v2/build/property.jam,v
retrieving revision 1.42
diff -u -r1.42 property.jam
--- property.jam	26 Sep 2003 10:22:21 -0000	1.42
+++ property.jam	21 Nov 2003 14:04:41 -0000
@@ -35,9 +35,14 @@
}

for local p in $(properties)
- {
+ { 
# No processing for free properties
- if free in [ feature.attributes $(p:G) ]
+ if [ MATCH (:) : $(p:G=) ]
+ {
+ # Skip conditional properties
+ result += $(p) ;
+ } 
+ else if free in [ feature.attributes $(p:G) ]
{
result += $(p) ;
}
@@ -68,7 +73,7 @@
{
__require__$(r:G) = ;
}
-
+ 
if $(error)
{
return $(error) ;
 --Boundary-00=_rAiv/QjgXJcqUQg--