]> code.ossystems Code Review - openembedded-core.git/commitdiff
boost: Pass CFLAGS, CXXFLAGS and LDFLAGS correctly into boost.build
authorMike Crowe <mac@mcrowe.com>
Thu, 13 Mar 2014 10:01:54 +0000 (10:01 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 17 Mar 2014 14:01:07 +0000 (14:01 +0000)
The compileflags and linkflags do_boostconfig puts into user-config.jam are
having no effect. According to
http://www.boost.org/boost-build2/doc/html/bbv2/reference/tools.html the
correct syntax would be <compileflags>"the flags" <linkflags>"the flags".

Since the flags specified were having no effect they can't be necessary and
can be safely removed. We should be passing ${CFLAGS}, ${CXXFLAGS} and
${LDFLAGS} instead so that users of the recipe can pass arbitrary flags as
they choose.

Signed-off-by: Mike Crowe <mac@mcrowe.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-support/boost/boost.inc

index c7642befa4ba9a1b36359614b93f55f21f5dda07..356ca22f4a0a220bd69fd645b09d3ac0b26d3039 100644 (file)
@@ -134,7 +134,7 @@ do_boostconfig() {
        # D2194:Fixing the failure of "error: duplicate initialization of gcc with the following parameters" during compilation.
        if ! grep -qe "^using gcc : 4.3.1" ${S}/tools/build/v2/user-config.jam 
        then
-               echo 'using gcc : 4.3.1 : ${CXX} : compileflags -DBOOST_SP_USE_PTHREADS -I${includedir} linkflags -L${libdir} ;' >> ${S}/tools/build/v2/user-config.jam
+               echo 'using gcc : 4.3.1 : ${CXX} : <cflags>"${CFLAGS}" <cxxflags>"${CXXFLAGS}" <linkflags>"${LDFLAGS}" ;' >> ${S}/tools/build/v2/user-config.jam
        fi      
 
        echo "using python : ${PYTHON_BASEVERSION} : : ${STAGING_INCDIR}/python${PYTHON_BASEVERSION} ;" >> ${S}/tools/build/v2/user-config.jam