Blame SOURCES/0109-multipathd-replace-libreadline-with-libedit.patch

7805a0
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
7805a0
From: Martin Wilck <mwilck@suse.com>
7805a0
Date: Fri, 12 Aug 2022 18:58:15 +0200
7805a0
Subject: [PATCH] multipathd: replace libreadline with libedit
7805a0
7805a0
Linking multipathd with libreadline may cause a license conflict,
7805a0
because libreadline is licensed under GPL-3.0-or-later, and
7805a0
libmultipath contains several files under GPL-2.0.
7805a0
7805a0
See:
7805a0
   https://github.com/opensvc/multipath-tools/issues/36
7805a0
   https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=979095
7805a0
   https://www.gnu.org/licenses/gpl-faq.html#AllCompatibility
7805a0
7805a0
Replace the readline functionality with libedit, which comes under
7805a0
a BSD license. The readline library can still be enabled (e.g. for
7805a0
binaries not intended to be distributed) by running
7805a0
"make READLINE=libreadline".
7805a0
7805a0
Signed-off-by: Martin Wilck <mwilck@suse.com>
7805a0
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
7805a0
---
7805a0
 Makefile.inc        |  5 +++++
7805a0
 multipathd/Makefile | 11 ++++++++++-
7805a0
 multipathd/cli.c    |  5 +++++
7805a0
 multipathd/uxclnt.c |  6 ++++++
7805a0
 4 files changed, 26 insertions(+), 1 deletion(-)
7805a0
7805a0
diff --git a/Makefile.inc b/Makefile.inc
7805a0
index 25c16f4e..d471f045 100644
7805a0
--- a/Makefile.inc
7805a0
+++ b/Makefile.inc
7805a0
@@ -14,6 +14,11 @@
7805a0
 #
7805a0
 # Uncomment to disable dmevents polling support
7805a0
 # ENABLE_DMEVENTS_POLL = 0
7805a0
+#
7805a0
+# Readline library to use, libedit or libreadline
7805a0
+# Caution: Using libreadline may make the multipathd binary undistributable,
7805a0
+# see https://github.com/opensvc/multipath-tools/issues/36
7805a0
+READLINE = libedit
7805a0
 
7805a0
 ifeq ($(TOPDIR),)
7805a0
 	TOPDIR	= ..
7805a0
diff --git a/multipathd/Makefile b/multipathd/Makefile
7805a0
index 835edd93..4874ec3a 100644
7805a0
--- a/multipathd/Makefile
7805a0
+++ b/multipathd/Makefile
7805a0
@@ -15,7 +15,16 @@ CFLAGS += $(BIN_CFLAGS) -I$(multipathdir) -I$(mpathpersistdir) \
7805a0
 LDFLAGS += $(BIN_LDFLAGS)
7805a0
 LIBDEPS += -L$(multipathdir) -lmultipath -L$(mpathpersistdir) -lmpathpersist \
7805a0
 	   -L$(mpathcmddir) -lmpathcmd -ludev -ldl -lurcu -lpthread \
7805a0
-	   -ldevmapper -lreadline
7805a0
+	   -ldevmapper
7805a0
+
7805a0
+ifeq ($(READLINE),libedit)
7805a0
+CFLAGS += -DUSE_LIBEDIT
7805a0
+LIBDEPS += -ledit
7805a0
+endif
7805a0
+ifeq ($(READLINE),libreadline)
7805a0
+CFLAGS += -DUSE_LIBREADLINE
7805a0
+LIBDEPS += -lreadline
7805a0
+endif
7805a0
 
7805a0
 ifdef SYSTEMD
7805a0
 	CFLAGS += -DUSE_SYSTEMD=$(SYSTEMD)
7805a0
diff --git a/multipathd/cli.c b/multipathd/cli.c
7805a0
index bdc9fb10..85d73dfb 100644
7805a0
--- a/multipathd/cli.c
7805a0
+++ b/multipathd/cli.c
7805a0
@@ -11,7 +11,12 @@
7805a0
 #include "parser.h"
7805a0
 #include "util.h"
7805a0
 #include "version.h"
7805a0
+#ifdef USE_LIBEDIT
7805a0
+#include <editline/readline.h>
7805a0
+#endif
7805a0
+#ifdef USE_LIBREADLINE
7805a0
 #include <readline/readline.h>
7805a0
+#endif
7805a0
 
7805a0
 #include "mpath_cmd.h"
7805a0
 #include "cli.h"
7805a0
diff --git a/multipathd/uxclnt.c b/multipathd/uxclnt.c
7805a0
index f16a7309..2c17d8fc 100644
7805a0
--- a/multipathd/uxclnt.c
7805a0
+++ b/multipathd/uxclnt.c
7805a0
@@ -16,8 +16,14 @@
7805a0
 #include <sys/socket.h>
7805a0
 #include <sys/un.h>
7805a0
 #include <poll.h>
7805a0
+
7805a0
+#ifdef USE_LIBEDIT
7805a0
+#include <editline/readline.h>
7805a0
+#endif
7805a0
+#ifdef USE_LIBREADLINE
7805a0
 #include <readline/readline.h>
7805a0
 #include <readline/history.h>
7805a0
+#endif
7805a0
 
7805a0
 #include "mpath_cmd.h"
7805a0
 #include "uxsock.h"