Re: [jamboost] Intel toolset for V2

From: Vladimir Prus (ghost_at_[hidden])
Date: 2004-05-31 01:13:54

Hi Michael,

> > I've verified that this works after my fixes. Could you update and try
> > again? The fix is in revision 1.56 of build/feature.jam and 1.46 of
> > build/property.jam.
>
> Just picked these up from CVS HEAD.
>
> The fix looks good. All three syntaxes work as expected. I'm impressed!

Good ;-)

> I made an interestingly discovery by accidentally having one to many ':'.
> When the conditional properties appear in the "default-build" arguments the
> error messages become very confusing.
>
> With
>
> : <toolset>intel,<toolset-intel:platform>linux:<cxxflags>TEST
>
> I see
> error: "intel,<toolset" is not a known value of feature <toolset>

Yeah, I've made the same mistake -- putting conditional property in default 
build -- while fixing this last bug. And got the same not very helpfull 
error.

> In a future (far of) it might be interesting to generate a more meaningful
> error in these cases. I will put it on my TODO list!

Thanks!

- Volodya