summaryrefslogtreecommitdiffstats
path: root/meta/recipes-extended/gawk
diff options
context:
space:
mode:
authorPaul Gortmaker <paul.gortmaker@windriver.com>2014-06-13 14:54:45 -0400
committerRichard Purdie <richard.purdie@linuxfoundation.org>2014-06-17 08:59:04 +0100
commitc7bfbabab7274f31be9e484bbe277cf9d011072c (patch)
tree7719b1fe7569e8be9e62911a4fc9593887b87311 /meta/recipes-extended/gawk
parentf6ca3ce0ae7bbe32a2f044e7882310df20ce6af2 (diff)
downloadpoky-c7bfbabab7274f31be9e484bbe277cf9d011072c.tar.gz
gawk: ensure cross compiling doesn't try to remove host files
Fixes [YOCTO #6440] When cross compiling gawk, it is possible to see this fail: make[4]: Entering directory '/mnt/home/paul/poky/build/tmp/work/corei7-64-poky- linux/gawk/4.1.1-r0/build/extension' for i in filefuncs.la fnmatch.la fork.la inplace.la ordchr.la readdir.la readfi le.la revoutput.la revtwoway.la rwarray.la testext.la time.la ; do \ rm -f /usr/lib/gawk/$i ; \ done rm: cannot remove '/usr/lib/gawk/filefuncs.la': Permission denied rm: cannot remove '/usr/lib/gawk/fnmatch.la': Permission denied rm: cannot remove '/usr/lib/gawk/fork.la': Permission denied rm: cannot remove '/usr/lib/gawk/inplace.la': Permission denied rm: cannot remove '/usr/lib/gawk/ordchr.la': Permission denied rm: cannot remove '/usr/lib/gawk/readdir.la': Permission denied rm: cannot remove '/usr/lib/gawk/readfile.la': Permission denied rm: cannot remove '/usr/lib/gawk/revoutput.la': Permission denied rm: cannot remove '/usr/lib/gawk/revtwoway.la': Permission denied rm: cannot remove '/usr/lib/gawk/rwarray.la': Permission denied rm: cannot remove '/usr/lib/gawk/testext.la': Permission denied rm: cannot remove '/usr/lib/gawk/time.la': Permission denied Makefile:1235: recipe for target 'install-data-hook' failed The problem only manifests itself on hosts where the above files are already present; for if they are absent then the rm -f does not fail with -EPERM. Before looking in mainline gawk for a fix, I fixed it myself. Then in comparing with mainline gawk, I found their fix was not 100% complete. So here we get a backport of the mainline gawk commit, plus the delta as a commit that I've sent to the gawk mailing list. (From OE-Core rev: 8c7a883964e45e7b48b943731bd3b3da0cc289d9) Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-extended/gawk')
-rw-r--r--meta/recipes-extended/gawk/gawk-4.1.1/Use-DESTDIR-in-extension-Makefile.am-when-removing-..patch56
-rw-r--r--meta/recipes-extended/gawk/gawk-4.1.1/extension-Add-DESTDIR-prefix-to-remaining-pkgextensi.patch86
-rw-r--r--meta/recipes-extended/gawk/gawk_4.1.1.bb2
3 files changed, 144 insertions, 0 deletions
diff --git a/meta/recipes-extended/gawk/gawk-4.1.1/Use-DESTDIR-in-extension-Makefile.am-when-removing-..patch b/meta/recipes-extended/gawk/gawk-4.1.1/Use-DESTDIR-in-extension-Makefile.am-when-removing-..patch
new file mode 100644
index 0000000000..b8b1fcaea7
--- /dev/null
+++ b/meta/recipes-extended/gawk/gawk-4.1.1/Use-DESTDIR-in-extension-Makefile.am-when-removing-..patch
@@ -0,0 +1,56 @@
1From 976f73ab03569bb836aa5c949be1c1cc973b2d2e Mon Sep 17 00:00:00 2001
2From: "Arnold D. Robbins" <arnold@skeeve.com>
3Date: Fri, 11 Apr 2014 07:42:16 +0300
4Subject: [PATCH] Use $(DESTDIR) in extension/Makefile.am when removing .la
5 files.
6
7commit 976f73ab03569bb836aa5c949be1c1cc973b2d2e upstream.
8
9[PG: note there was no Author SOB or long log in original.]
10Upstream-Status: Backport [ gawk-4.1.1-3-g976f73ab0356 ]
11Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
12---
13
14diff --git a/extension/ChangeLog b/extension/ChangeLog
15index afd64ff764b6..f3a1c7a81e7d 100644
16--- a/extension/ChangeLog
17+++ b/extension/ChangeLog
18@@ -1,3 +1,9 @@
19+2014-04-11 Arnold D. Robbins <arnold@skeeve.com>
20+
21+ * Makefile.am (install-data-hook): Use $(DESTDIR) when removing
22+ the .la files. Thanks to Lars Wendler <polynomial-c@gentoo.org>
23+ for the report and fix.
24+
25 2014-04-08 Arnold D. Robbins <arnold@skeeve.com>
26
27 * 4.1.1: Release tar ball made.
28diff --git a/extension/Makefile.am b/extension/Makefile.am
29index 9c49bb81abb2..11826e2b315e 100644
30--- a/extension/Makefile.am
31+++ b/extension/Makefile.am
32@@ -100,7 +100,7 @@ testext_la_LIBADD = $(MY_LIBS)
33
34 install-data-hook:
35 for i in $(pkgextension_LTLIBRARIES) ; do \
36- $(RM) $(pkgextensiondir)/$$i ; \
37+ $(RM) $(DESTDIR)$(pkgextensiondir)/$$i ; \
38 done
39
40 # Keep the uninstall check working:
41diff --git a/extension/Makefile.in b/extension/Makefile.in
42index 040cdb8f4e23..d81b16960935 100644
43--- a/extension/Makefile.in
44+++ b/extension/Makefile.in
45@@ -1231,7 +1231,7 @@ uninstall-man: uninstall-man3
46
47 install-data-hook:
48 for i in $(pkgextension_LTLIBRARIES) ; do \
49- $(RM) $(pkgextensiondir)/$$i ; \
50+ $(RM) $(DESTDIR)$(pkgextensiondir)/$$i ; \
51 done
52
53 # Keep the uninstall check working:
54--
551.9.1
56
diff --git a/meta/recipes-extended/gawk/gawk-4.1.1/extension-Add-DESTDIR-prefix-to-remaining-pkgextensi.patch b/meta/recipes-extended/gawk/gawk-4.1.1/extension-Add-DESTDIR-prefix-to-remaining-pkgextensi.patch
new file mode 100644
index 0000000000..b402587615
--- /dev/null
+++ b/meta/recipes-extended/gawk/gawk-4.1.1/extension-Add-DESTDIR-prefix-to-remaining-pkgextensi.patch
@@ -0,0 +1,86 @@
1From a9f3bd754e9e269099cf7a2c931c632fcc12bd8a Mon Sep 17 00:00:00 2001
2From: Paul Gortmaker <paul.gortmaker@windriver.com>
3Date: Fri, 13 Jun 2014 13:42:12 -0400
4Subject: [PATCH] extension: Add $(DESTDIR) prefix to remaining pkgextensiondir
5 instances
6
7Commit 976f73ab03569bb836aa5c949be1c1cc973b2d2e ["Use $(DESTDIR) in
8extension/Makefile.am when removing .la files."] fixed an issue
9that manifested itself as follows when cross compiling:
10
11make[4]: Entering directory 'gawk/4.1.1-r0/build/extension'
12for i in filefuncs.la fnmatch.la fork.la inplace.la ordchr.la readdir.la readfi
13le.la revoutput.la revtwoway.la rwarray.la testext.la time.la ; do \
14 rm -f /usr/lib/gawk/$i ; \
15done
16rm: cannot remove '/usr/lib/gawk/filefuncs.la': Permission denied
17rm: cannot remove '/usr/lib/gawk/fnmatch.la': Permission denied
18rm: cannot remove '/usr/lib/gawk/fork.la': Permission denied
19rm: cannot remove '/usr/lib/gawk/inplace.la': Permission denied
20rm: cannot remove '/usr/lib/gawk/ordchr.la': Permission denied
21rm: cannot remove '/usr/lib/gawk/readdir.la': Permission denied
22rm: cannot remove '/usr/lib/gawk/readfile.la': Permission denied
23rm: cannot remove '/usr/lib/gawk/revoutput.la': Permission denied
24rm: cannot remove '/usr/lib/gawk/revtwoway.la': Permission denied
25rm: cannot remove '/usr/lib/gawk/rwarray.la': Permission denied
26rm: cannot remove '/usr/lib/gawk/testext.la': Permission denied
27rm: cannot remove '/usr/lib/gawk/time.la': Permission denied
28Makefile:1235: recipe for target 'install-data-hook' failed
29
30The problem only manifests itself on hosts where the above files
31are already present; for if they are absent then the rm -f does
32not fail with -EPERM. The fix in 976f73ab0356 ensured that DESTDIR
33was used for the prefix so that it didn't try to delete host files.
34
35However there still remains less used instances of where it is used
36w/o $(DESTDIR) that may still cause similar breakage in the future.
37Here we apply the same change to them.
38
39Upstream-Status: Submitted [http://lists.gnu.org/archive/html/bug-gawk/2014-06/index.html]
40Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
41---
42
43diff --git a/extension/ChangeLog b/extension/ChangeLog
44index f3a1c7a81e7d..b87ac372edc6 100644
45--- a/extension/ChangeLog
46+++ b/extension/ChangeLog
47@@ -1,3 +1,10 @@
48+2014-06-13 Paul Gortmaker <paul.gortmaker@windriver.com>
49+ * Makefile.am (uninstall-so): Came across below bug while cross
50+ compiling, and changed both install-data-hook and uninstall-so
51+ to use $(DESTDIR) on v4.1.1 before seeing most of the fix in
52+ gawk-4.1.1-3-g976f73ab0356; here we ensure uninstall-so also
53+ uses the $(DESTDIR) prefix on its use of pkgextensiondir.
54+
55 2014-04-11 Arnold D. Robbins <arnold@skeeve.com>
56
57 * Makefile.am (install-data-hook): Use $(DESTDIR) when removing
58diff --git a/extension/Makefile.am b/extension/Makefile.am
59index 11826e2b315e..b6beaee3fb47 100644
60--- a/extension/Makefile.am
61+++ b/extension/Makefile.am
62@@ -105,7 +105,7 @@ install-data-hook:
63
64 # Keep the uninstall check working:
65 uninstall-so:
66- $(RM) $(pkgextensiondir)/*.so
67+ $(RM) $(DESTDIR)$(pkgextensiondir)/*.so
68
69 uninstall-recursive: uninstall-so
70
71diff --git a/extension/Makefile.in b/extension/Makefile.in
72index d81b16960935..294e4f887812 100644
73--- a/extension/Makefile.in
74+++ b/extension/Makefile.in
75@@ -1236,7 +1236,7 @@ install-data-hook:
76
77 # Keep the uninstall check working:
78 uninstall-so:
79- $(RM) $(pkgextensiondir)/*.so
80+ $(RM) $(DESTDIR)$(pkgextensiondir)/*.so
81
82 uninstall-recursive: uninstall-so
83
84--
851.9.1
86
diff --git a/meta/recipes-extended/gawk/gawk_4.1.1.bb b/meta/recipes-extended/gawk/gawk_4.1.1.bb
index a81be3137a..1298ee2f4c 100644
--- a/meta/recipes-extended/gawk/gawk_4.1.1.bb
+++ b/meta/recipes-extended/gawk/gawk_4.1.1.bb
@@ -15,6 +15,8 @@ DEPENDS += "readline"
15 15
16SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \ 16SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \
17 file://run-ptest \ 17 file://run-ptest \
18 file://Use-DESTDIR-in-extension-Makefile.am-when-removing-..patch \
19 file://extension-Add-DESTDIR-prefix-to-remaining-pkgextensi.patch \
18" 20"
19 21
20SRC_URI[md5sum] = "45f5b09aa87b4744c4c53bf274e96ed0" 22SRC_URI[md5sum] = "45f5b09aa87b4744c4c53bf274e96ed0"