The strategy of setting OPENMP_CFLAG and OPENMP_LDFLAG to an empty
string instead of '-fopenmp' doesn't work for gcc because this then
tries to access("", F_OK) which then results in this error:

egcc: error: : No such file or directory

http://build-failures.rhaalovely.net/sparc64/2021-05-10/graphics/mypaint.log

Instead, we can disable openmp differently. This builds and packages
both on sparc64 and on amd64.

Presumably it's also possible to set an appropriate option in
MODPY_SETUP_ARGS but I couldn't figure out how.

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/mypaint/Makefile,v
retrieving revision 1.4
diff -u -p -r1.4 Makefile
--- Makefile    23 Feb 2021 19:39:26 -0000      1.4
+++ Makefile    16 May 2021 21:02:39 -0000
@@ -5,7 +5,7 @@ COMMENT =       raster graphics editor for dig
 MODPY_EGG_VERSION =    2.0.1
 DISTNAME =             mypaint-${MODPY_EGG_VERSION}
 EXTRACT_SUFX =         .tar.xz
-REVISION =             1
+REVISION =             2
 
 CATEGORIES =           graphics
 
Index: patches/patch-setup_py
===================================================================
RCS file: /cvs/ports/graphics/mypaint/patches/patch-setup_py,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 patch-setup_py
--- patches/patch-setup_py      12 Sep 2020 20:57:21 -0000      1.1.1.1
+++ patches/patch-setup_py      16 May 2021 21:04:50 -0000
@@ -5,14 +5,12 @@ OpenBSD does not have OpenMP available i
 Index: setup.py
 --- setup.py.orig
 +++ setup.py
-@@ -29,8 +29,8 @@ from setuptools.command.install_scripts import install
+@@ -437,7 +437,7 @@ class BuildExt (build_ext):
  
- # Some versions of clang requires different flag configurations than gcc
- # to link correctly, so we enable configuration via environment variables.
--OPENMP_CFLAG = os.getenv("OPENMP_CFLAG", "-fopenmp")
--OPENMP_LDFLAG = os.getenv("OPENMP_LDFLAG", "-fopenmp")
-+OPENMP_CFLAG = os.getenv("OPENMP_CFLAG", "")
-+OPENMP_LDFLAG = os.getenv("OPENMP_LDFLAG", "")
+     def initialize_options(self):
+         self.set_rpath = False
+-        self.disable_openmp = False
++        self.disable_openmp = True
+         build_ext.initialize_options(self)
  
- 
- # Libmypaint dependency
+     def finalize_options(self):

Reply via email to