|
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