Skip to content
Permalink
Browse files

libomp: Disable universal on Mojave+

  • Loading branch information
eborisch committed Sep 28, 2018
1 parent b184c85 commit 06df52a7fda865f033c65f3923ea4c74b51be1a8
Showing with 8 additions and 2 deletions.
  1. +8 −2 lang/libomp/Portfile
@@ -2,7 +2,9 @@

PortSystem 1.0
PortGroup cmake 1.0
PortGroup muniversal 1.0
if {${os.major} <= 17} {
PortGroup muniversal 1.0
}
PortGroup compiler_blacklist_versions 1.0
PortGroup github 1.0

@@ -98,7 +100,11 @@ foreach ver {3.8 3.9 4.0 5.0 6.0 devel} {
}
}

default_variants +universal
if {${os.major} <= 17} {
default_variants +universal
} else {
universal_variant no
}

# Do actual install into ${prefix}/(install|lib)/libomp
# A little unorthodox, but to have clang automatically find the includes and

3 comments on commit 06df52a

@ryandesign

This comment has been minimized.

Copy link
Contributor

@ryandesign ryandesign replied Sep 28, 2018

Note that we don't want to have to fix each individual port for this issue; we want a fix in base that works for all ports. See macports/macports-base#107 for that effort.

@eborisch

This comment has been minimized.

Copy link
Member Author

@eborisch eborisch replied Sep 28, 2018

Absolutely, but since this port is required by all the recent clang versions, it has a large downstream footprint. Once it is addressed in base, we can back these changes out.

@pmetzger

This comment has been minimized.

Copy link
Member

@pmetzger pmetzger replied Sep 28, 2018

Addressing this in base quickly would indeed be a good idea.

Please sign in to comment.
You can’t perform that action at this time.