[Buildroot] [PATXH v3 0/2] Update Xen and fix build issue

[Buildroot] [PATXH v3 0/2] Update Xen and fix build issue

[Alistair Francis]

This patch series updates Xen to 4.7.1 and fixes a build issue that previously existed.

NOTE: Xen 4.8.0 was released today, but that failed to compile so it will be more work.
At the moment Xen doesn't build so fix that first.

V3:
 - Add a link to an autobuilder issue
V2:
 - Fix typo and add commit message to patch 2

Alistair Francis (2):
  package/xen: Bump Xen to 4.7.1
  package/xen: Disable strict unused-const-variable checking

 ...-Disable-strict-unused-const-variable-che.patch | 27 ++++++++++++++++++++++
 package/xen/xen.hash                               |  2 +-
 package/xen/xen.mk                                 |  2 +-
 3 files changed, 29 insertions(+), 2 deletions(-)
 create mode 100644 package/xen/0003-tools-libxl-Disable-strict-unused-const-variable-che.patch

-- 
2.7.4

[Buildroot] [PATXH v3 1/2] package/xen: Bump Xen to 4.7.1

[Alistair Francis]

Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
---
 package/xen/xen.hash | 2 +-
 package/xen/xen.mk   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/xen/xen.hash b/package/xen/xen.hash
index 602e6c7..c3df243 100644
--- a/package/xen/xen.hash
+++ b/package/xen/xen.hash
@@ -1,2 +1,2 @@
 # Locally computed
-sha256 be5876144d49729572ae06142e0bb93f1c1f2695578141eff2931995add24623 xen-4.7.0.tar.gz
+sha256 e87f4b0575e78657ee23d31470a15ecf1ce8c3a92a771cda46bbcd4d0d671ffe xen-4.7.1.tar.gz
diff --git a/package/xen/xen.mk b/package/xen/xen.mk
index e2c797f..7d32e52 100644
--- a/package/xen/xen.mk
+++ b/package/xen/xen.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-XEN_VERSION = 4.7.0
+XEN_VERSION = 4.7.1
 XEN_SITE = http://bits.xensource.com/oss-xen/release/$(XEN_VERSION)
 XEN_LICENSE = GPLv2
 XEN_LICENSE_FILES = COPYING
-- 
2.7.4

[Buildroot] [PATXH v3 1/2] package/xen: Bump Xen to 4.7.1

[Thomas Petazzoni]

Hello,

On Thu, 8 Dec 2016 12:11:11 -0800, Alistair Francis wrote:
> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
> ---
>  package/xen/xen.hash | 2 +-
>  package/xen/xen.mk   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATXH v3 1/2] package/xen: Bump Xen to 4.7.1

[Alistair Francis]

On Thu, Dec 8, 2016 at 1:42 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Thu, 8 Dec 2016 12:11:11 -0800, Alistair Francis wrote:
>> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
>> ---
>>  package/xen/xen.hash | 2 +-
>>  package/xen/xen.mk   | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> Applied to master, thanks.

Thanks,

Alistair

>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com

[Buildroot] [PATXH v3 2/2] package/xen: Disable strict unused-const-variable checking

[Alistair Francis]

The strict unused-const-variable checking was causing autobuilder errors
when trying to build Xen tools/libxl as the migrate_*[] arrays are not
always accessed.

To avoid the error edit the Makefile to stop the warnings being treated
as errors.

Fixes:
  http://autobuild.buildroot.net/results/0e0/0e0d4aa4a05da5804821951289c0a4049b009c61/

Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
---
 ...-Disable-strict-unused-const-variable-che.patch | 27 ++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 package/xen/0003-tools-libxl-Disable-strict-unused-const-variable-che.patch

diff --git a/package/xen/0003-tools-libxl-Disable-strict-unused-const-variable-che.patch b/package/xen/0003-tools-libxl-Disable-strict-unused-const-variable-che.patch
new file mode 100644
index 0000000..05575fa
--- /dev/null
+++ b/package/xen/0003-tools-libxl-Disable-strict-unused-const-variable-che.patch
@@ -0,0 +1,27 @@
+From 2e04c2db466b2d7881b46505741798701b1aa271 Mon Sep 17 00:00:00 2001
+From: Alistair Francis <alistair.francis@xilinx.com>
+Date: Wed, 7 Dec 2016 11:35:10 -0800
+Subject: [PATCH] tools/libxl: Disable strict unused-const-variable checking
+
+Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
+---
+ tools/libxl/Makefile | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/tools/libxl/Makefile b/tools/libxl/Makefile
+index defeb40..9d320d1 100644
+--- a/tools/libxl/Makefile
++++ b/tools/libxl/Makefile
+@@ -12,7 +12,8 @@ XLUMAJOR = 4.7
+ XLUMINOR = 0
+ 
+ CFLAGS += -Werror -Wno-format-zero-length -Wmissing-declarations \
+-	-Wno-declaration-after-statement -Wformat-nonliteral
++	-Wno-declaration-after-statement -Wformat-nonliteral \
++	-Werror=unused-const-variable=0
+ CFLAGS += -I. -fPIC
+ 
+ ifeq ($(CONFIG_Linux),y)
+-- 
+2.7.4
+
-- 
2.7.4

[Buildroot] [PATXH v3 2/2] package/xen: Disable strict unused-const-variable checking

[Thomas Petazzoni]

Hello,

On Thu, 8 Dec 2016 12:11:12 -0800, Alistair Francis wrote:

> + CFLAGS += -Werror -Wno-format-zero-length -Wmissing-declarations \
> +-	-Wno-declaration-after-statement -Wformat-nonliteral
> ++	-Wno-declaration-after-statement -Wformat-nonliteral \
> ++	-Werror=unused-const-variable=0

Sorry for not catching this earlier, but what we want instead is to
completely remove -Werror. It's pointless for releases, and is causing
to cause other build failures.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[Buildroot] [PATXH v3 2/2] package/xen: Disable strict unused-const-variable checking

[Alistair Francis]

On Thu, Dec 8, 2016 at 1:06 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Thu, 8 Dec 2016 12:11:12 -0800, Alistair Francis wrote:
>
>> + CFLAGS += -Werror -Wno-format-zero-length -Wmissing-declarations \
>> +-    -Wno-declaration-after-statement -Wformat-nonliteral
>> ++    -Wno-declaration-after-statement -Wformat-nonliteral \
>> ++    -Werror=unused-const-variable=0
>
> Sorry for not catching this earlier, but what we want instead is to
> completely remove -Werror. It's pointless for releases, and is causing
> to cause other build failures.

No worries, I have removed the -Werror. I left the other ones in there though.

Thanks,

Alistair

>
> Thanks,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com