diff mbox series

[1/1] package/openswan: disable -Werror

Message ID 20210609201419.625739-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/openswan: disable -Werror | expand

Commit Message

Fabrice Fontaine June 9, 2021, 8:14 p.m. UTC
Disable -Werror to avoid the following build failure with -DNDEBUG
raised since commit 5a8c50fe05afacc3cbe8e7347e238da9f242fab0

/srv/storage/autobuild/run/instance-2/output-1/build/openswan-3.0.0/programs/rsasigkey/rsasigkey.c:524:6: error: variable 'success' set but not used [-Werror=unused-but-set-variable]
  524 |  int success;
      |      ^~~~~~~

Fixes:
 - http://autobuild.buildroot.org/results/327a0f2b8f0c51bcbb3edb1c3671870d593e93b9

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/openswan/openswan.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Arnout Vandecappelle June 20, 2021, 4:26 p.m. UTC | #1
On 09/06/2021 22:14, Fabrice Fontaine wrote:
> Disable -Werror to avoid the following build failure with -DNDEBUG
> raised since commit 5a8c50fe05afacc3cbe8e7347e238da9f242fab0
> 
> /srv/storage/autobuild/run/instance-2/output-1/build/openswan-3.0.0/programs/rsasigkey/rsasigkey.c:524:6: error: variable 'success' set but not used [-Werror=unused-but-set-variable]
>   524 |  int success;
>       |      ^~~~~~~
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/327a0f2b8f0c51bcbb3edb1c3671870d593e93b9
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

 Applied to master, thanks.

 Regards,
 Arnout

> ---
>  package/openswan/openswan.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/openswan/openswan.mk b/package/openswan/openswan.mk
> index 7f083f695d..bb56e4516c 100644
> --- a/package/openswan/openswan.mk
> +++ b/package/openswan/openswan.mk
> @@ -15,7 +15,7 @@ OPENSWAN_MAKE_OPTS = ARCH=$(BR2_ARCH) CC="$(TARGET_CC)" POD2MAN="" XMLTO="" \
>  	USERCOMPILE="$(TARGET_CFLAGS) $(if $(BR2_TOOLCHAIN_SUPPORTS_PIE),-fPIE)" \
>  	USERLINK="$(TARGET_LDFLAGS) $(if $(BR2_TOOLCHAIN_SUPPORTS_PIE),-fPIE)" \
>  	INC_USRLOCAL=/usr USE_KLIPS=false USE_MAST=false USE_NM=false \
> -	USE_NOMANINSTALL=true
> +	USE_NOMANINSTALL=true WERROR=""
>  
>  ifeq ($(BR2_PACKAGE_LIBCURL),y)
>  OPENSWAN_DEPENDENCIES += libcurl
>
Peter Korsgaard June 23, 2021, 6:33 a.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Disable -Werror to avoid the following build failure with -DNDEBUG
 > raised since commit 5a8c50fe05afacc3cbe8e7347e238da9f242fab0

 > /srv/storage/autobuild/run/instance-2/output-1/build/openswan-3.0.0/programs/rsasigkey/rsasigkey.c:524:6: error: variable 'success' set but not used [-Werror=unused-but-set-variable]
 >   524 |  int success;
 >       |      ^~~~~~~

 > Fixes:
 >  - http://autobuild.buildroot.org/results/327a0f2b8f0c51bcbb3edb1c3671870d593e93b9

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

While not strictly needed for 2021.02.x / 2021.05.x as they don't have
the NDEBUG logic, I still backported it to get rid of -Werror.
diff mbox series

Patch

diff --git a/package/openswan/openswan.mk b/package/openswan/openswan.mk
index 7f083f695d..bb56e4516c 100644
--- a/package/openswan/openswan.mk
+++ b/package/openswan/openswan.mk
@@ -15,7 +15,7 @@  OPENSWAN_MAKE_OPTS = ARCH=$(BR2_ARCH) CC="$(TARGET_CC)" POD2MAN="" XMLTO="" \
 	USERCOMPILE="$(TARGET_CFLAGS) $(if $(BR2_TOOLCHAIN_SUPPORTS_PIE),-fPIE)" \
 	USERLINK="$(TARGET_LDFLAGS) $(if $(BR2_TOOLCHAIN_SUPPORTS_PIE),-fPIE)" \
 	INC_USRLOCAL=/usr USE_KLIPS=false USE_MAST=false USE_NM=false \
-	USE_NOMANINSTALL=true
+	USE_NOMANINSTALL=true WERROR=""
 
 ifeq ($(BR2_PACKAGE_LIBCURL),y)
 OPENSWAN_DEPENDENCIES += libcurl