On Sun, May 16, 2021 at 11:10:32PM +0200, Theo Buehler wrote:
> 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.

This fixed the build of mypaint on the last sparc64 bulk.

ok kmos

--Kurt

> 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