commit:     d98ddafb2aef042c7ac141b041e8bfebf915910f
Author:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
AuthorDate: Tue Dec 27 14:06:43 2022 +0000
Commit:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
CommitDate: Tue Dec 27 19:58:59 2022 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d98ddafb

dev-lang/R: attempt to fix the parallel install.

We have had an "emake -j1 ... install" in this ebuild for a long time,
but bug 883063 shows that -j1 only minimizes the risk of failure, and
does not completely eliminate it. I've tried to reproduce the failure
and have found a missing dependency in the upstream Makefile. This
commit adds a patch to fix it.

Now, we have one fewer build system bugs. I've removed the old -j1
workaround, and am marking bug 883063 as resolved so that future
installation failures are reported as new bugs: we need new reports to
let us know that the bug fixed in this commit was not the only one, if
that happens to be the case.

Closes: https://bugs.gentoo.org/883063
Signed-off-by: Michael Orlitzky <mjo <AT> gentoo.org>

 dev-lang/R/R-4.2.2.ebuild                          |  3 ++-
 .../R/files/R-4.2.2-parallel-rmath-h-install.patch | 23 ++++++++++++++++++++++
 2 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/dev-lang/R/R-4.2.2.ebuild b/dev-lang/R/R-4.2.2.ebuild
index b1adfb0f9f9a..3d3cbf497f10 100644
--- a/dev-lang/R/R-4.2.2.ebuild
+++ b/dev-lang/R/R-4.2.2.ebuild
@@ -77,6 +77,7 @@ PATCHES=(
        "${FILESDIR}"/R-3.6.2-no-gzip-doc.patch
        "${FILESDIR}"/R-4.2.2-browser-pdf-prefs.patch
        "${FILESDIR}"/R-4.2.2-isspace-without-readline.patch
+       "${FILESDIR}/"R-4.2.2-parallel-rmath-h-install.patch
 )
 
 pkg_pretend() {
@@ -178,7 +179,7 @@ src_compile() {
 
 src_install() {
        default
-       emake -j1 -C src/nmath/standalone DESTDIR="${D}" install
+       emake -C src/nmath/standalone DESTDIR="${D}" install
 
        if use doc; then
                emake DESTDIR="${D}" install-info install-pdf

diff --git a/dev-lang/R/files/R-4.2.2-parallel-rmath-h-install.patch 
b/dev-lang/R/files/R-4.2.2-parallel-rmath-h-install.patch
new file mode 100644
index 000000000000..756ec8ff5428
--- /dev/null
+++ b/dev-lang/R/files/R-4.2.2-parallel-rmath-h-install.patch
@@ -0,0 +1,23 @@
+From ee1428e77095e52a7396fa7b455f63b7c5b9495b Mon Sep 17 00:00:00 2001
+From: kalibera <kalibera@00db46b3-68df-0310-9c12-caf00c1e9a41>
+Date: Tue, 27 Dec 2022 11:21:33 +0000
+Subject: [PATCH] Declare dependency to ensure include dir exists (PR#17743).
+
+git-svn-id: https://svn.r-project.org/R/trunk@83504 
00db46b3-68df-0310-9c12-caf00c1e9a41
+---
+ src/nmath/standalone/Makefile.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/nmath/standalone/Makefile.in 
b/src/nmath/standalone/Makefile.in
+index 68b45006155..946795a0439 100644
+--- a/src/nmath/standalone/Makefile.in
++++ b/src/nmath/standalone/Makefile.in
+@@ -151,7 +151,7 @@ install-strip: installdirs install-header install-pc
+       fi
+       @!(test -f libRmath.a) || $(SHELL) $(top_srcdir)/tools/copy-if-change 
libRmath.a $(DESTDIR)$(libdir)/libRmath.a
+ 
+-install-header:
++install-header: installdirs
+       @(cd ../../include; $(MAKE) Rmath.h)
+ @WANT_R_FRAMEWORK_FALSE@      @$(SHELL) $(top_srcdir)/tools/copy-if-change 
../../include/Rmath.h $(DESTDIR)$(includedir)/Rmath.h
+ @WANT_R_FRAMEWORK_TRUE@       @$(SHELL) $(top_srcdir)/tools/copy-if-change 
../../include/Rmath.h 
$(R_FRAMEWORK_DIR)/Versions/$(FW_VERSION)/Resources/include/Rmath.h

Reply via email to