mbox series

[v2,0/4] build: honor toolchain related environment vars

Message ID 20190905144835.6035-1-roger.pau@citrix.com (mailing list archive)
Headers show
Series build: honor toolchain related environment vars | expand

Message

Roger Pau Monne Sept. 5, 2019, 2:48 p.m. UTC
Hello,

Current Xen build system will ignore any toolchain related variables on
the environment when building (ie: CC, LD, CXX...), and the only way to
set those is to assign them directly on the make command line (ie: make
CC=foo CXX=bar ...).

The following series attempts to fix this, by removing the hardcoding of
the toolchain variables previously done in StdGNU.mk.

Note that this has the side effect that the build system will no longer
prepend CROSS_COMPILE to the toolchain variables if those are already
set. So if you are building Xen and setting CROSS_COMPILE make sure
toolchain variables are unset, or if set they should contain
CROSS_COMPILE. The Travis CI script is updated in patch 3/4 in order to
comply with the above.

This is v2 because v1 was missing the first patch, rendering the whole
series useless. Apart from that there are no changes from v1.

The series can be found at:

git://xenbits.xen.org/people/royger/xen.git env_tools

Results from Travis and gitlab CI loops are at:

https://travis-ci.org/royger/xen/builds/581139388
https://gitlab.com/xen-project/people/royger/xen/pipelines/80440648

Thanks, Roger.

Roger Pau Monne (4):
  build: set HOST{CC/CXX}, clang and gcc in StdGNU.mk
  kconfig: include default toolchain values
  build: allow picking the env values for compiler variables
  build: allow picking the env values for toolchain utilities

 Config.mk                          | 18 ----------
 config/StdGNU.mk                   | 53 ++++++++++++++++++++----------
 scripts/travis-build               |  8 +++++
 xen/Makefile                       |  6 ++--
 xen/tools/kconfig/Makefile.kconfig |  7 ++--
 5 files changed, 50 insertions(+), 42 deletions(-)