[PATCH 19/19] scripts/kconfig/mconf: fix build error on Mac OS X

2012-09-25 Thread Florian Fainelli 
From: Felix Fietkau n...@openwrt.org

SIGWINCH is not defined for this platform, provide a definition of it if
needed.

Signed-off-by: Felix Fietkau n...@openwrt.org
---
 scripts/kconfig/mconf.c |4 
 1 file changed, 4 insertions(+)

diff --git a/scripts/kconfig/mconf.c b/scripts/kconfig/mconf.c
index 1b0b6ab..84d7662 100644
--- a/scripts/kconfig/mconf.c
+++ b/scripts/kconfig/mconf.c
@@ -27,6 +27,10 @@
 #include unistd.h
 #include locale.h
 
+#ifndef SIGWINCH
+#define SIGWINCH 28
+#endif
+
 #define LKC_DIRECT_LINK
 #include lkc.h
 
-- 
1.7.9.5

___
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Re: [PATCH 19/19] scripts/kconfig/mconf: fix build error on Mac OS X

2012-09-25 Thread Michael Tokarev 
On 25.09.2012 16:06, Florian Fainelli wrote:
 From: Felix Fietkau n...@openwrt.org
 
 SIGWINCH is not defined for this platform, provide a definition of it if
 needed.
 
 Signed-off-by: Felix Fietkau n...@openwrt.org
 ---
  scripts/kconfig/mconf.c |4 
  1 file changed, 4 insertions(+)
 
 diff --git a/scripts/kconfig/mconf.c b/scripts/kconfig/mconf.c
 index 1b0b6ab..84d7662 100644
 --- a/scripts/kconfig/mconf.c
 +++ b/scripts/kconfig/mconf.c
 @@ -27,6 +27,10 @@
  #include unistd.h
  #include locale.h
  
 +#ifndef SIGWINCH
 +#define SIGWINCH 28
 +#endif

Isn't it better to #ifdef the relevant code instead
of defining something which will never be used or
may even clash with some other signal?

/mjt
___
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox