mirror of
https://github.com/boostorg/context.git
synced 2026-01-19 04:02:17 +00:00
`in` operator in bjam always returns true if its first argument has no elements[1]. This means that if `os.platform` is empty (not detected), the construction introduced in commit9084161b87sets ABI to `aapcs` on all platforms where `os.platform` is empty, including, e.g. loongarch64, and breaks build there. This commit refactors the condition to use '=' operator, to make sure that when `os.platform` is empty we get the default ABI value, and thus fixes build on loongarch64. See also819c2d6423for similar fix that was somehow lost in refactorings. This time a comment is added near the condition in hope that future edits will not reintroduce the issue. [1] https://www.boost.org/doc/libs/1_83_0/tools/build/doc/html/index.html#jam.language.flow_of_control Fixes:9084161b87
77 lines
2.0 KiB
Plaintext
77 lines
2.0 KiB
Plaintext
# Copyright René Ferdinand Rivera Morell 2023-2024
|
|
# Distributed under the Boost Software License, Version 1.0.
|
|
# (See accompanying file LICENSE_1_0.txt or copy at
|
|
# http://www.boost.org/LICENSE_1_0.txt)
|
|
|
|
import feature ;
|
|
import os ;
|
|
|
|
feature.feature segmented-stacks : on : optional propagated composite ;
|
|
feature.compose <segmented-stacks>on : <define>BOOST_USE_SEGMENTED_STACKS ;
|
|
|
|
feature.feature htm : tsx : optional propagated composite ;
|
|
feature.compose <htm>tsx : <define>BOOST_USE_TSX ;
|
|
|
|
feature.feature valgrind : on : optional propagated composite ;
|
|
feature.compose <valgrind>on : <define>BOOST_USE_VALGRIND ;
|
|
|
|
local rule default_binary_format ( )
|
|
{
|
|
local tmp = elf ;
|
|
if [ os.name ] = "NT" { tmp = pe ; }
|
|
else if [ os.name ] = "CYGWIN" { tmp = pe ; }
|
|
else if [ os.name ] = "AIX" { tmp = xcoff ; }
|
|
else if [ os.name ] = "MACOSX" { tmp = mach-o ; }
|
|
return $(tmp) ;
|
|
}
|
|
|
|
feature.feature binary-format
|
|
: elf
|
|
mach-o
|
|
pe
|
|
xcoff
|
|
: propagated
|
|
;
|
|
feature.set-default binary-format : [ default_binary_format ] ;
|
|
|
|
local rule default_abi ( )
|
|
{
|
|
local tmp = sysv ;
|
|
|
|
# Avoid using "in" operator here: it returns true if its left
|
|
# part is empty, which happens e.g. with os.platform on
|
|
# some uncommon architectures.
|
|
if [ os.name ] = "NT" { tmp = ms ; }
|
|
else if [ os.name ] = "CYGWIN" { tmp = ms ; }
|
|
else if [ os.platform ] = "ARM" { tmp = aapcs ; }
|
|
else if [ os.platform ] = "ARM64" { tmp = aapcs ; }
|
|
else if [ os.platform ] = "MIPS32" { tmp = o32 ; }
|
|
else if [ os.platform ] = "MIPS64" { tmp = n64 ; }
|
|
return $(tmp) ;
|
|
}
|
|
|
|
feature.feature abi
|
|
: aapcs
|
|
eabi
|
|
ms
|
|
n32
|
|
n64
|
|
o32
|
|
o64
|
|
sysv
|
|
x32
|
|
: propagated
|
|
;
|
|
feature.set-default abi : [ default_abi ] ;
|
|
|
|
feature.feature context-impl
|
|
: fcontext
|
|
ucontext
|
|
winfib
|
|
: propagated
|
|
composite
|
|
;
|
|
feature.set-default context-impl : fcontext ;
|
|
feature.compose <context-impl>ucontext : <define>BOOST_USE_UCONTEXT ;
|
|
feature.compose <context-impl>winfib : <define>BOOST_USE_WINFIB ;
|