* [Buildroot] [PATCH] openvmtools: disable -Werror
@ 2014-09-15 14:56 Karoly Kasza
2014-09-21 12:57 ` Yann E. MORIN
2014-09-21 17:37 ` Thomas Petazzoni
0 siblings, 2 replies; 3+ messages in thread
From: Karoly Kasza @ 2014-09-15 14:56 UTC (permalink / raw)
To: buildroot
Disable the mandatory flag -Werror in configure.ac.
This also makes -Wno-deprecated-declarations flag unnecessary.
Signed-off-by: Karoly Kasza <kaszak@gmail.com>
---
.../openvmtools/openvmtools-04-no_cflags_werror.patch | 17 +++++++++++++++++
package/openvmtools/openvmtools.mk | 4 ----
2 files changed, 17 insertions(+), 4 deletions(-)
create mode 100644 package/openvmtools/openvmtools-04-no_cflags_werror.patch
diff --git a/package/openvmtools/openvmtools-04-no_cflags_werror.patch b/package/openvmtools/openvmtools-04-no_cflags_werror.patch
new file mode 100644
index 0000000..96bd355
--- /dev/null
+++ b/package/openvmtools/openvmtools-04-no_cflags_werror.patch
@@ -0,0 +1,17 @@
+configure.ac: disable -Werror
+
+Disable the mandatory flag -Werror in configure.ac.
+
+Signed-off-by: Karoly Kasza <kaszak@gmail.com>
+
+--- open-vm-tools-9.4.6-1770165.orig/configure.ac 2014-07-02 00:21:14.000000000 +0200
++++ open-vm-tools-9.4.6-1770165/configure.ac 2014-09-15 16:03:47.565900402 +0200
+@@ -832,7 +832,7 @@
+
+ ### General flags / actions
+ CFLAGS="$CFLAGS -Wall"
+-CFLAGS="$CFLAGS -Werror"
++# CFLAGS="$CFLAGS -Werror"
+
+ # -Wno-unknown-pragmas is due to gcc not understanding '#pragma ident' in Xlib.h on OpenSolaris.
+ for TEST_CFLAG in -Wno-pointer-sign -Wno-unused-value -fno-strict-aliasing \
diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
index 6361d6b..9191643 100644
--- a/package/openvmtools/openvmtools.mk
+++ b/package/openvmtools/openvmtools.mk
@@ -13,10 +13,6 @@ OPENVMTOOLS_LICENSE_FILES = COPYING
# See http://sourceforge.net/p/open-vm-tools/mailman/message/32550385/
OPENVMTOOLS_AUTORECONF = YES
OPENVMTOOLS_CONF_OPT = --without-icu --without-x --without-gtk2 --without-gtkmm --without-kernel-modules
-# -Wno-deprecated-declarations is a workaround for a bug in open-vm-tools
-# See http://sourceforge.net/p/open-vm-tools/mailman/message/31473171/
-OPENVMTOOLS_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -Wno-deprecated-declarations"
-
OPENVMTOOLS_DEPENDENCIES = libglib2
# When libfuse is available, openvmtools can build vmblock-fuse, so
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] openvmtools: disable -Werror
2014-09-15 14:56 [Buildroot] [PATCH] openvmtools: disable -Werror Karoly Kasza
@ 2014-09-21 12:57 ` Yann E. MORIN
2014-09-21 17:37 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2014-09-21 12:57 UTC (permalink / raw)
To: buildroot
Karoly, All,
On 2014-09-15 16:56 +0200, Karoly Kasza spake thusly:
> Disable the mandatory flag -Werror in configure.ac.
> This also makes -Wno-deprecated-declarations flag unnecessary.
>
> Signed-off-by: Karoly Kasza <kaszak@gmail.com>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
I guess this is fixing a bunch of autobuild errors; these should have
been referenced in the commit log, as such:
Fixes:
http://autobuild.buildroot.net/results/440/4400abc17115d0f5eb65c4eb012f11766cbe5a56/
http://autobuild.buildroot.net/results/02b/02b24a012dbc76e366c10d3758978eba5e597499/
[...]
(No need to put them all, just a few; the [...] means there are a lot
more, hence it is important to apply this patch.)
Regards,
Yann E. MORIN.
> ---
> .../openvmtools/openvmtools-04-no_cflags_werror.patch | 17 +++++++++++++++++
> package/openvmtools/openvmtools.mk | 4 ----
> 2 files changed, 17 insertions(+), 4 deletions(-)
> create mode 100644 package/openvmtools/openvmtools-04-no_cflags_werror.patch
>
> diff --git a/package/openvmtools/openvmtools-04-no_cflags_werror.patch b/package/openvmtools/openvmtools-04-no_cflags_werror.patch
> new file mode 100644
> index 0000000..96bd355
> --- /dev/null
> +++ b/package/openvmtools/openvmtools-04-no_cflags_werror.patch
> @@ -0,0 +1,17 @@
> +configure.ac: disable -Werror
> +
> +Disable the mandatory flag -Werror in configure.ac.
> +
> +Signed-off-by: Karoly Kasza <kaszak@gmail.com>
> +
> +--- open-vm-tools-9.4.6-1770165.orig/configure.ac 2014-07-02 00:21:14.000000000 +0200
> ++++ open-vm-tools-9.4.6-1770165/configure.ac 2014-09-15 16:03:47.565900402 +0200
> +@@ -832,7 +832,7 @@
> +
> + ### General flags / actions
> + CFLAGS="$CFLAGS -Wall"
> +-CFLAGS="$CFLAGS -Werror"
> ++# CFLAGS="$CFLAGS -Werror"
> +
> + # -Wno-unknown-pragmas is due to gcc not understanding '#pragma ident' in Xlib.h on OpenSolaris.
> + for TEST_CFLAG in -Wno-pointer-sign -Wno-unused-value -fno-strict-aliasing \
> diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
> index 6361d6b..9191643 100644
> --- a/package/openvmtools/openvmtools.mk
> +++ b/package/openvmtools/openvmtools.mk
> @@ -13,10 +13,6 @@ OPENVMTOOLS_LICENSE_FILES = COPYING
> # See http://sourceforge.net/p/open-vm-tools/mailman/message/32550385/
> OPENVMTOOLS_AUTORECONF = YES
> OPENVMTOOLS_CONF_OPT = --without-icu --without-x --without-gtk2 --without-gtkmm --without-kernel-modules
> -# -Wno-deprecated-declarations is a workaround for a bug in open-vm-tools
> -# See http://sourceforge.net/p/open-vm-tools/mailman/message/31473171/
> -OPENVMTOOLS_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -Wno-deprecated-declarations"
> -
> OPENVMTOOLS_DEPENDENCIES = libglib2
>
> # When libfuse is available, openvmtools can build vmblock-fuse, so
> --
> 1.7.10.4
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 3+ messages in thread* [Buildroot] [PATCH] openvmtools: disable -Werror
2014-09-15 14:56 [Buildroot] [PATCH] openvmtools: disable -Werror Karoly Kasza
2014-09-21 12:57 ` Yann E. MORIN
@ 2014-09-21 17:37 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2014-09-21 17:37 UTC (permalink / raw)
To: buildroot
Dear Karoly Kasza,
On Mon, 15 Sep 2014 16:56:35 +0200, Karoly Kasza wrote:
> Disable the mandatory flag -Werror in configure.ac.
> This also makes -Wno-deprecated-declarations flag unnecessary.
>
> Signed-off-by: Karoly Kasza <kaszak@gmail.com>
> ---
> .../openvmtools/openvmtools-04-no_cflags_werror.patch | 17 +++++++++++++++++
> package/openvmtools/openvmtools.mk | 4 ----
> 2 files changed, 17 insertions(+), 4 deletions(-)
> create mode 100644 package/openvmtools/openvmtools-04-no_cflags_werror.patch
Applied, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-09-21 17:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-15 14:56 [Buildroot] [PATCH] openvmtools: disable -Werror Karoly Kasza
2014-09-21 12:57 ` Yann E. MORIN
2014-09-21 17:37 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox