aboutsummaryrefslogtreecommitdiffstats
path: root/meta-oe/recipes-support/multipath-tools/files/shared-libs-avoid-linking-.so-as-executable.patch
blob: ae6bb89a77b016c281a540930bff86ebe8d6af73 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
From 4311856200480288494df8df737f71ea316592d0 Mon Sep 17 00:00:00 2001
From: Patrick Ohly <patrick.ohly@intel.com>
Date: Thu, 26 Jan 2017 15:28:34 +0100
Subject: [PATCH] shared libs: avoid linking .so as executable

When the OE build systems sets CFLAGS including -pie, gcc overrides
the earlier -shared parameter and tries to link a binary, leading to
errors about missing functions like main().

Reordering the parameters so that -shared comes after CFLAGS
fixes this.

Upstream-Status: Inappropriate [OE specific, -pie shouldn't be in CFLAGS?!]

Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
---
 libmpathpersist/Makefile | 2 +-
 libmultipath/Makefile    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libmpathpersist/Makefile b/libmpathpersist/Makefile
index c4ec1c5..790692a 100644
--- a/libmpathpersist/Makefile
+++ b/libmpathpersist/Makefile
@@ -20,7 +20,7 @@ all: $(LIBS)
 
 $(LIBS): 
 	$(CC) -Wall -fPIC -c $(CFLAGS) *.c 
-	$(CC)  -shared $(LIBDEPS) -Wl,-soname=$@ $(CFLAGS) -o $@ $(OBJS)
+	$(CC) $(LIBDEPS) $(CFLAGS) $(SHARED_FLAGS) -Wl,-soname=$@ -o $@ $(OBJS)
 	ln -s $(LIBS) $(DEVLIB)
 	$(GZIP) mpath_persistent_reserve_in.3 > mpath_persistent_reserve_in.3.gz	
 	$(GZIP) mpath_persistent_reserve_out.3 > mpath_persistent_reserve_out.3.gz	
diff --git a/libmultipath/Makefile b/libmultipath/Makefile
index fc0f3d6..2287d91 100644
--- a/libmultipath/Makefile
+++ b/libmultipath/Makefile
@@ -55,7 +55,7 @@ endif
 all: $(LIBS)
 
 $(LIBS): $(OBJS)
-	$(CC) $(LDFLAGS) $(SHARED_FLAGS) -Wl,-soname=$@ $(CFLAGS) -o $@ $(OBJS) $(LIBDEPS)
+	$(CC) $(LDFLAGS) $(CFLAGS) $(SHARED_FLAGS) -Wl,-soname=$@ -o $@ $(OBJS) $(LIBDEPS)
 	ln -sf $@ $(DEVLIB)
 
 install:
-- 
2.11.0