Patchwork lsof: do not remove WIDECHARINCL definition

login
register
mail settings
Submitter Benoît Thébaudeau
Date May 2, 2014, 2:59 p.m.
Message ID <1399042764-22388-1-git-send-email-benoit.thebaudeau@advansee.com>
Download mbox | patch
Permalink /patch/345017/
State Accepted
Headers show

Comments

Benoît Thébaudeau - May 2, 2014, 2:59 p.m.
WIDECHARINCL is enabled by HASWIDECHAR, so removing its definition if
BR2_USE_WCHAR is not set is useless.

Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
---
 package/lsof/lsof.mk | 2 --
 1 file changed, 2 deletions(-)
Thomas Petazzoni - June 9, 2014, 1:07 p.m.
Dear Benoît Thébaudeau,

On Fri,  2 May 2014 16:59:23 +0200, Benoît Thébaudeau wrote:
> WIDECHARINCL is enabled by HASWIDECHAR, so removing its definition if
> BR2_USE_WCHAR is not set is useless.
> 
> Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
> ---
>  package/lsof/lsof.mk | 2 --
>  1 file changed, 2 deletions(-)

Applied, thanks.

Thomas

Patch

diff --git a/package/lsof/lsof.mk b/package/lsof/lsof.mk
index f4b1b4c..aa101e4 100644
--- a/package/lsof/lsof.mk
+++ b/package/lsof/lsof.mk
@@ -27,8 +27,6 @@  ifeq ($(BR2_USE_WCHAR),)
 define LSOF_CONFIGURE_WCHAR_FIXUPS
 	$(SED) 's,^#define[[:space:]]*HASWIDECHAR.*,#undef HASWIDECHAR,' \
 		$(@D)/machine.h
-	$(SED) 's,^#define[[:space:]]*WIDECHARINCL.*,,' \
-		$(@D)/machine.h
 endef
 endif