Re: [jamboost] Intel toolset for V2

From: Vladimir Prus (ghost_at_[hidden])
Date: 2004-05-28 05:50:31

Hi Michael,

> I have switched over from my original and tested the new version. Works
> fine for me.

Good.

> > I've decided that in the end, "intel" is better that "icc", in particular
> > because the compiler is called "icl" on windows.
>
> As long as Intel don't change their name this should all work out fine :-)
>
>
> The use of the subfeature <platform>linux in the naming of <toolset>intel
> brings up a few issues with conditional properties. Since conditional
> properties are currently being discussed I will mention them now.
>
> The simplest syntax using <toolset>intel-linux work fine as expected.
>
> exe test : tests.cpp
>
> : <toolset>intel-linux:<cxxflags>TEST
>
> ;
>
> In the documentation a comma seperated syntax is defined for conditional
> properties.
> property ( "," property ) * ":" property
> trying
>
> : <toolset>intel,<platform>linux:<cxxflags>TEST
>
> results in the error
> intel,<platform>linux" is not a known value of feature <toolset>

The syntax is not correct, the right one is:

: <toolset>intel,<toolset-intel:platform>linux:<cxxflags>TEST

but that does not work either, due to bugfix I've made yesterday -- it made 
<toolset>intel-linux work, but broke this case. I hope I've fixed this now.

> Other syntaxes that explicity mention the platform subfeature also fail in
> this context.
>
> : <toolset-intel:platform>linux:<cxxflags>TEST

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.

Thanks for testing this!

- Volodya