Boost logo

Boost-Build :

Subject: Re: [Boost-build] Suppressing auto-detected address-model from paths
From: Vladimir Prus (vladimir.prus_at_[hidden])
Date: 2015-04-04 13:16:14


On 04/04/2015 05:36 PM, Steven Watanabe wrote:
> AMDG
>
> On 04/04/2015 05:19 AM, Vladimir Prus wrote:
>>
>> Yes, I woke up with exactly same conclusion today. With the original
>> patch, explicitly specifying
>> address-model=64 on the command line will add a path element even if
>> compiler default is also 64-bit.
>>
>> The attached boost.diff handles it. However, it requires a further
>> change to build, also attached.
>> Right now, when indirect conditional returns a composite property, its
>> constituents override any
>> existing properties. In case of deduce-address-model, we want it to be
>> expanded, but not override
>> any value of address-model that is already specified.
>>
>
> Couldn't we fix this by including the given
> address-model in the result if it's already
> specified? I'd really like to avoid potentially
> breaking changes to Boost.Build this late
> in the release cycle.

Fair enough. Here's what I've checked in:

     https://github.com/boostorg/boost/commit/945e3c0bbdd31ce8c297e0aff5340f0adb196cea

Seems to work fine in all testing.

-- 
Vladimir Prus
http://vladimirprus.com

Boost-Build list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk