diff mbox

[v2,3/3] travis: actually disable debug for non-debug

Message ID 1457122188-7389-3-git-send-email-cardoe@cardoe.com (mailing list archive)
State New, archived
Headers show

Commit Message

Douglas Goldstein March 4, 2016, 8:09 p.m. UTC
Non-debug builds need to explicitly disable debug due to debug being
defaulted to y in Config.mk

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
---
CC: Ian Jackson <ian.jackson@eu.citrix.com>
CC: Jan Beulich <jbeulich@suse.com>
CC: Keir Fraser <keir@xen.org>
CC: Tim Deegan <tim@xen.org>
CC: Andrew Cooper <andrew.cooper3@citrix.com>

change since v1:
- none

tested at: https://travis-ci.org/cardoe/xen/builds/113700670
this run shows a failure with BIGMEM=y and debug=n which was only exposed
by this change.
---
 .travis.yml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

Comments

Konrad Rzeszutek Wilk March 7, 2016, 3:32 p.m. UTC | #1
On Fri, Mar 04, 2016 at 02:09:48PM -0600, Doug Goldstein wrote:
> Non-debug builds need to explicitly disable debug due to debug being
> defaulted to y in Config.mk

Which is because by default staging has debug=y (this changes once
the rcX candidates become available).

Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
> ---
> CC: Ian Jackson <ian.jackson@eu.citrix.com>
> CC: Jan Beulich <jbeulich@suse.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Tim Deegan <tim@xen.org>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> 
> change since v1:
> - none
> 
> tested at: https://travis-ci.org/cardoe/xen/builds/113700670
> this run shows a failure with BIGMEM=y and debug=n which was only exposed
> by this change.
> ---
>  .travis.yml | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 4bcd5a0..741a8ab 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -11,33 +11,33 @@ branches:
>  matrix:
>      include:
>          - compiler: gcc
> -          env: XEN_TARGET_ARCH=x86_64
> +          env: XEN_TARGET_ARCH=x86_64 debug=n
>          - compiler: gcc
> -          env: XEN_TARGET_ARCH=x86_64 XEN_CONFIG_EXPERT=y RANDCONFIG=y
> +          env: XEN_TARGET_ARCH=x86_64 XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
>          - compiler: gcc-5
> -          env: XEN_TARGET_ARCH=x86_64
> +          env: XEN_TARGET_ARCH=x86_64 debug=n
>          - compiler: gcc
>            env: XEN_TARGET_ARCH=x86_64 debug=y
>          - compiler: gcc-5
>            env: XEN_TARGET_ARCH=x86_64 debug=y
>          - compiler: clang
> -          env: XEN_TARGET_ARCH=x86_64 clang=y
> +          env: XEN_TARGET_ARCH=x86_64 clang=y debug=n
>          - compiler: clang-3.8
> -          env: XEN_TARGET_ARCH=x86_64 clang=y
> +          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-
> +          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- debug=n
>          - compiler: gcc
> -          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- XEN_CONFIG_EXPERT=y RANDCONFIG=y
> +          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
>          - compiler: gcc
>            env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- debug=y
>          - compiler: gcc
> -          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu-
> +          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- debug=n
>          - compiler: gcc
> -          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- XEN_CONFIG_EXPERT=y RANDCONFIG=y
> +          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
>          - compiler: gcc
>            env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- debug=y
>  addons:
> -- 
> 2.4.10
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
diff mbox

Patch

diff --git a/.travis.yml b/.travis.yml
index 4bcd5a0..741a8ab 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -11,33 +11,33 @@  branches:
 matrix:
     include:
         - compiler: gcc
-          env: XEN_TARGET_ARCH=x86_64
+          env: XEN_TARGET_ARCH=x86_64 debug=n
         - compiler: gcc
-          env: XEN_TARGET_ARCH=x86_64 XEN_CONFIG_EXPERT=y RANDCONFIG=y
+          env: XEN_TARGET_ARCH=x86_64 XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
         - compiler: gcc-5
-          env: XEN_TARGET_ARCH=x86_64
+          env: XEN_TARGET_ARCH=x86_64 debug=n
         - compiler: gcc
           env: XEN_TARGET_ARCH=x86_64 debug=y
         - compiler: gcc-5
           env: XEN_TARGET_ARCH=x86_64 debug=y
         - compiler: clang
-          env: XEN_TARGET_ARCH=x86_64 clang=y
+          env: XEN_TARGET_ARCH=x86_64 clang=y debug=n
         - compiler: clang-3.8
-          env: XEN_TARGET_ARCH=x86_64 clang=y
+          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-
+          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- debug=n
         - compiler: gcc
-          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- XEN_CONFIG_EXPERT=y RANDCONFIG=y
+          env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
         - compiler: gcc
           env: XEN_TARGET_ARCH=arm32 CROSS_COMPILE=arm-linux-gnueabihf- debug=y
         - compiler: gcc
-          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu-
+          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- debug=n
         - compiler: gcc
-          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- XEN_CONFIG_EXPERT=y RANDCONFIG=y
+          env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- XEN_CONFIG_EXPERT=y RANDCONFIG=y debug=n
         - compiler: gcc
           env: XEN_TARGET_ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- debug=y
 addons: