| Message ID | 1461776486-31484-7-git-send-email-andrew.cooper3@citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Apr 27, 2016 at 06:01:25PM +0100, Andrew Cooper wrote: > The package appears to have been renamed in Ubuntu. The only reason this test > is currently passing is because the hypervisor build falls back to clang, at > version 3.5 > > Add an explicit test in the build script that out desired compiler is > available. Note that travis already performs this step, but in a way which > isn't fatal to the build. > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Reviewed-by: Wei Liu <wei.liu2@citrix.com>
On 4/27/16 12:01 PM, Andrew Cooper wrote: > The package appears to have been renamed in Ubuntu. The only reason this test > is currently passing is because the hypervisor build falls back to clang, at > version 3.5 > > Add an explicit test in the build script that out desired compiler is > available. Note that travis already performs this step, but in a way which > isn't fatal to the build. > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Doug Goldstein <cardoe@cardoe.com>
diff --git a/.travis.yml b/.travis.yml index 0eea94e..d2e1bec 100644 --- a/.travis.yml +++ b/.travis.yml @@ -22,12 +22,8 @@ matrix: env: XEN_TARGET_ARCH=x86_64 debug=y - compiler: clang env: XEN_TARGET_ARCH=x86_64 clang=y debug=n - - compiler: clang-3.8 - env: XEN_TARGET_ARCH=x86_64 clang=y debug=n - compiler: clang env: XEN_TARGET_ARCH=x86_64 clang=y debug=y - - compiler: clang-3.8 - env: XEN_TARGET_ARCH=x86_64 clang=y debug=y - compiler: gcc env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- debug=n - compiler: gcc diff --git a/scripts/travis-build b/scripts/travis-build index b553f20..9a5c1c8 100755 --- a/scripts/travis-build +++ b/scripts/travis-build @@ -1,5 +1,7 @@ #!/bin/bash -ex +$CC --version + # random config or default config if [[ "${RANDCONFIG}" == "y" ]]; then make -C xen randconfig
The package appears to have been renamed in Ubuntu. The only reason this test is currently passing is because the hypervisor build falls back to clang, at version 3.5 Add an explicit test in the build script that out desired compiler is available. Note that travis already performs this step, but in a way which isn't fatal to the build. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: Ian Jackson <Ian.Jackson@eu.citrix.com> CC: Wei Liu <wei.liu2@citrix.com> CC: Doug Goldstein <cardoe@cardoe.com> --- .travis.yml | 4 ---- scripts/travis-build | 2 ++ 2 files changed, 2 insertions(+), 4 deletions(-)