diff mbox

[for-4.7,7/7] travis: Enable tools when building with clang

Message ID 1461776486-31484-8-git-send-email-andrew.cooper3@citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andrew Cooper April 27, 2016, 5:01 p.m. UTC
tools now build under clang, so let them be tested.

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>
---
 scripts/travis-build | 6 ------
 1 file changed, 6 deletions(-)

Comments

Wei Liu April 27, 2016, 6:03 p.m. UTC | #1
On Wed, Apr 27, 2016 at 06:01:26PM +0100, Andrew Cooper wrote:
> tools now build under clang, so let them be tested.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Reviewed-by: Wei Liu <wei.liu2@citrix.com>

> ---
> CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Doug Goldstein <cardoe@cardoe.com>
> ---
>  scripts/travis-build | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/scripts/travis-build b/scripts/travis-build
> index 9a5c1c8..584d008 100755
> --- a/scripts/travis-build
> +++ b/scripts/travis-build
> @@ -22,12 +22,6 @@ else
>      cfgargs+=("--disable-tools") # we don't have the cross depends installed
>  fi
>  
> -# Due to multiple build failures and the desire to get more
> -# build testing (GCC only) enabled this is disabled
> -if [[ "${clang}" == "y" ]]; then
> -    cfgargs+=("--disable-tools")
> -fi
> -
>  ./configure "${cfgargs[@]}"
>  
>  make dist
> -- 
> 2.1.4
>
Douglas Goldstein April 27, 2016, 7:14 p.m. UTC | #2
On 4/27/16 12:01 PM, Andrew Cooper wrote:
> tools now build under clang, so let them be tested.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> ---

Acked-by: Doug Goldstein <cardoe@cardoe.com>
diff mbox

Patch

diff --git a/scripts/travis-build b/scripts/travis-build
index 9a5c1c8..584d008 100755
--- a/scripts/travis-build
+++ b/scripts/travis-build
@@ -22,12 +22,6 @@  else
     cfgargs+=("--disable-tools") # we don't have the cross depends installed
 fi
 
-# Due to multiple build failures and the desire to get more
-# build testing (GCC only) enabled this is disabled
-if [[ "${clang}" == "y" ]]; then
-    cfgargs+=("--disable-tools")
-fi
-
 ./configure "${cfgargs[@]}"
 
 make dist