aboutsummaryrefslogtreecommitdiff
path: root/package/redis
diff options
context:
space:
mode:
authorGravatar Thomas Petazzoni <thomas.petazzoni@free-electrons.com>2013-03-25 11:15:34 +0000
committerGravatar Peter Korsgaard <jacmet@sunsite.dk>2013-03-25 23:36:08 +0100
commit5b13261dc7be4b82d8e64b08744569090b7da71c (patch)
treea954834cfd910952096236d414b722ba8ae31f00 /package/redis
parentbbce7505232614e2ad8638bd20c5057e2fa7764d (diff)
downloadbuildroot-5b13261dc7be4b82d8e64b08744569090b7da71c.tar.gz
buildroot-5b13261dc7be4b82d8e64b08744569090b7da71c.tar.bz2
redis: build fix and minor improvement
The redis-001-uclibc.patch was properly adding conditionals to ensure HAVE_BACKTRACE is not defined when uClibc is used. However, __unix doesn't seem to be define with the toolchain Peter uses on gcc14, so the <features.h> doesn't get included, therefore __UCLIBC isn't defined, and the code believes that backtrace support is available. We fix this by including <features.h> when __linux__ is defined. Fixes: http://autobuild.buildroot.org/results/4ba818f877af145a547a1fede0bb738192c2f181/build-end.log. Also add another patch that do not redefine the _LARGEFILE_SOURCE and _FILE_OFFSET_BITS macros if they are already defined on the command line. It avoids the ugly warnings that we could see from the build logs, and therefore make it easier to spot real errors. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Diffstat (limited to 'package/redis')
-rw-r--r--package/redis/redis-001-uclibc.patch2
-rw-r--r--package/redis/redis-003-largefile-conditional-define.patch32
2 files changed, 33 insertions, 1 deletions
diff --git a/package/redis/redis-001-uclibc.patch b/package/redis/redis-001-uclibc.patch
index d7fd9c3c56..8762ca2fb9 100644
--- a/package/redis/redis-001-uclibc.patch
+++ b/package/redis/redis-001-uclibc.patch
@@ -15,7 +15,7 @@ diff -ur old/src/config.h new/src/config.h
#ifndef __CONFIG_H
#define __CONFIG_H
-+#ifdef __unix
++#if defined(__unix) || defined(__linux__)
+#include <features.h>
+#endif
+
diff --git a/package/redis/redis-003-largefile-conditional-define.patch b/package/redis/redis-003-largefile-conditional-define.patch
new file mode 100644
index 0000000000..747e0552b3
--- /dev/null
+++ b/package/redis/redis-003-largefile-conditional-define.patch
@@ -0,0 +1,32 @@
+Define _LARGEFILE_SOURCE and _FILE_OFFSET_BITS conditionally
+
+In order to avoid ugly warnings at compile time, only define
+_LARGEFILE_SOURCE and _FILE_OFFSET_BITS if they have not already been
+defined through the build command line.
+
+Avoids:
+
+ In file included from redis.h:33:0,
+ from migrate.c:1:
+ fmacros.h:45:0: warning: "_LARGEFILE_SOURCE" redefined
+ <command-line>:0:0: note: this is the location of the previous definition
+
+Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
+
+Index: redis-2.6.11/src/fmacros.h
+===================================================================
+--- redis-2.6.11.orig/src/fmacros.h 2013-03-25 22:09:15.000000000 +0100
++++ redis-2.6.11/src/fmacros.h 2013-03-25 22:09:40.000000000 +0100
+@@ -42,7 +42,12 @@
+ #define _XOPEN_SOURCE
+ #endif
+
++#ifndef _LARGEFILE_SOURCE
+ #define _LARGEFILE_SOURCE
++#endif
++
++#ifndef _FILE_OFFSET_BITS
+ #define _FILE_OFFSET_BITS 64
++#endif
+
+ #endif