[PATCH] docs: Update README to include Clang

[PATCH] docs: Update README to include Clang

[Andrew Cooper]

Xen now builds on x86 with Clang 3.5 and 3.8.  Update README to reflect this.

Mark Clang as no longer a permitted failure in Travis, to prevent future
regressions slipping back in.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Ian Campbell <Ian.Campbell@citrix.com>
CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
CC: Doug Goldstein <cardoe@cardoe.com>

N.B. Older versions of Clang are expected to work, but I currently don't have
an easy way of testing this.
---
 .travis.yml |  2 --
 README      | 15 +++++++++------
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 189d568..b84d38f 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -2,8 +2,6 @@ language: c
 dist: trusty
 sudo: required
 matrix:
-    allow_failures:
-        - compiler: clang
     include:
         - compiler: gcc
           env: XEN_TARGET_ARCH=x86_64
diff --git a/README b/README
index dd36ec8..fe2e7c6 100644
--- a/README
+++ b/README
@@ -35,13 +35,16 @@ Second, there are a number of prerequisites for building a Xen source
 release. Make sure you have all the following installed, either by
 visiting the project webpage or installing a pre-built package
 provided by your OS distributor:
-    * GCC
-      - For x86 4.1.2_20070115 or later
-      - For ARM 4.8 or later
     * GNU Make v3.80 or later
-    * GNU Binutils:
-      - For x86 2.16.91.0.5 or later
-      - For ARM 2.24 or later
+    * C compiler and linker:
+      - For x86:
+        - GCC 4.1.2_20070115 or later
+        - GNU Binutils 2.16.91.0.5 or later
+        or
+        - Clang/LLVM 3.5 or later
+      - For ARM:
+        - GCC 4.8 or later
+        - GNU Binutils 2.24 or later
     * Development install of zlib (e.g., zlib-dev)
     * Development install of Python v2.3 or later (e.g., python-dev)
     * Development install of curses (e.g., libncurses-dev)
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] docs: Update README to include Clang

[Doug Goldstein]

[-- Attachment #1.1: Type: text/plain, Size: 2289 bytes --]

On 2/25/16 9:59 AM, Andrew Cooper wrote:
> Xen now builds on x86 with Clang 3.5 and 3.8.  Update README to reflect this.
> 
> Mark Clang as no longer a permitted failure in Travis, to prevent future
> regressions slipping back in.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> ---
> CC: Ian Campbell <Ian.Campbell@citrix.com>
> CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Doug Goldstein <cardoe@cardoe.com>
> 
> N.B. Older versions of Clang are expected to work, but I currently don't have
> an easy way of testing this.
> ---
>  .travis.yml |  2 --
>  README      | 15 +++++++++------
>  2 files changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 189d568..b84d38f 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -2,8 +2,6 @@ language: c
>  dist: trusty
>  sudo: required
>  matrix:
> -    allow_failures:
> -        - compiler: clang
>      include:
>          - compiler: gcc
>            env: XEN_TARGET_ARCH=x86_64
> diff --git a/README b/README
> index dd36ec8..fe2e7c6 100644
> --- a/README
> +++ b/README
> @@ -35,13 +35,16 @@ Second, there are a number of prerequisites for building a Xen source
>  release. Make sure you have all the following installed, either by
>  visiting the project webpage or installing a pre-built package
>  provided by your OS distributor:
> -    * GCC
> -      - For x86 4.1.2_20070115 or later
> -      - For ARM 4.8 or later
>      * GNU Make v3.80 or later
> -    * GNU Binutils:
> -      - For x86 2.16.91.0.5 or later
> -      - For ARM 2.24 or later
> +    * C compiler and linker:
> +      - For x86:
> +        - GCC 4.1.2_20070115 or later
> +        - GNU Binutils 2.16.91.0.5 or later
> +        or
> +        - Clang/LLVM 3.5 or later
> +      - For ARM:
> +        - GCC 4.8 or later
> +        - GNU Binutils 2.24 or later
>      * Development install of zlib (e.g., zlib-dev)
>      * Development install of Python v2.3 or later (e.g., python-dev)
>      * Development install of curses (e.g., libncurses-dev)
> 

Reviewed-by: Doug Goldstein <cardoe@cardoe.com> for the README
Acked-by: Doug Goldstein <cardoe@cardoe.com> for the .travis.yml

-- 
Doug Goldstein


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 959 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] docs: Update README to include Clang

[Ian Jackson]

Andrew Cooper writes ("[PATCH] docs: Update README to include Clang"):
> Xen now builds on x86 with Clang 3.5 and 3.8.  Update README to reflect this.
> 
> Mark Clang as no longer a permitted failure in Travis, to prevent future
> regressions slipping back in.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel