libmemcached: fix build with gcc 7.x
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Fri, 21 Jul 2017 21:22:47 +0000 (23:22 +0200)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Fri, 21 Jul 2017 21:22:47 +0000 (23:22 +0200)
This commit adds a patch to the libmemcached package that fixes the
build with gcc 7.x. Since libmemcached is barely maintained upstream,
the patch comes from the Fedora packages.

Fixes:

  http://autobuild.buildroot.net/results/872b8e0e6a24cbc96e3ad9e0b8b47acdf6160ce0/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
package/libmemcached/0005-fix-pointer-comparaison.patch [new file with mode: 0644]

diff --git a/package/libmemcached/0005-fix-pointer-comparaison.patch b/package/libmemcached/0005-fix-pointer-comparaison.patch
new file mode 100644 (file)
index 0000000..c9f8dd1
--- /dev/null
@@ -0,0 +1,30 @@
+Fix pointer comparaison
+
+opt_servers is a pointer, not a boolean, so testing against false to
+know if the pointer is NULL no longer works with the more strict gcc
+7.x checks.
+
+[Taken from http://pkgs.fedoraproject.org/cgit/rpms/libmemcached.git/plain/libmemcached-build.patch.]
+Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
+
+diff -up ./clients/memflush.cc.old ./clients/memflush.cc
+--- ./clients/memflush.cc.old  2017-02-12 10:12:59.615209225 +0100
++++ ./clients/memflush.cc      2017-02-12 10:13:39.998382783 +0100
+@@ -39,7 +39,7 @@ int main(int argc, char *argv[])
+ {
+   options_parse(argc, argv);
+-  if (opt_servers == false)
++  if (!opt_servers)
+   {
+     char *temp;
+@@ -48,7 +48,7 @@ int main(int argc, char *argv[])
+       opt_servers= strdup(temp);
+     }
+-    if (opt_servers == false)
++    if (!opt_servers)
+     {
+       std::cerr << "No Servers provided" << std::endl;
+       exit(EXIT_FAILURE);