py36-graph-tool @2.27+clang60+openmp configuration error

[bakergilx (Dennis Baker)]

Error: Failed to configure py36-graph-tool, consult /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_python_py-graph-tool/py36-graph-tool/work/graph-tool-2.27/config.log
Error: Failed to configure py36-graph-tool: configure failure: command execution failed
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_python_py-graph-tool/py36-graph-tool/main.log for details.
Error: Follow https://guide.macports.org/#project.tickets to report a bug.
Error: Processing of port py36-graph-tool failed

[bakergilx (Dennis Baker)]

main log file

[mamoll (Mark Moll)]

Can you attach /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_python_py-graph-tool/py36-graph-tool/config.log as well?

[bakergilx (Dennis Baker)]

config log

[mamoll (Mark Moll)]

What is the output of port installed ld64? Can you try sudo port install ld64 +ld64_xcode and and try install py36-graph-tool again?

[bakergilx (Dennis Baker)]

port installed ld64 yielded: ld64 @3_0+universal (active)

installed ld64 with the ld64_xcode variant

py36-graph-tool has now built correctly.

Thanks!

[kencu (Ken)]

ld: unexpected token: !tapi-tbd-v3 file '/System/Library/Frameworks//CoreFoundation.framework/CoreFoundation.tbd' for architecture x86_64

We really should make ld64 +ld64_xcode the default ld64 variant for systems that are using tapi.

[kencu (Ken)]

We could probably just add it to this:

    if {![variant_isset ld64_97] && ![variant_isset ld64_127]} {
        if {${os.major} < 9} {
            default_variants +ld64_97
        } elseif {${os.major} < 11} {
            default_variants +ld64_127
        }
    }

like this maybe:

    if {![variant_isset ld64_97] && ![variant_isset ld64_127]} {
        if {${os.major} < 9} {
            default_variants +ld64_97
        } elseif {${os.major} < 11} {
            default_variants +ld64_127
        } elseif {${os.major} > 16} {
            default_variants +ld64_xcode
        } 
    }

and bump the revision to propagate that through.

Perhaps it should be Xcode-version based rather than os-version based...

[cjones051073 (Chris Jones)]

I completely agree that the xcode variant should be made the default on mac OS 10.13 (and newer...). At least until Apple releases newer version of the source to allow the tool kit it installs to be updated. As it stands the ld64 port installs binaries built using Xcode 8 vintage code (the latest currently made open source by Apple) and in fact are older than the Xcode 9 used on 10.13.

Note, for the record, we need to do the same for the cctools port, which has exactly the same issue as ld64 (and might become more of a problem for MacPorts in the future).

[kencu (Ken)]

Well, let's see if this flys or dies:

<​https://github.com/macports/macports-ports/pull/2247>

[cjones051073 (Chris Jones)]

Just FYI I've just submitted

​https://github.com/macports/macports-ports/pull/2253

That adds a similar xcode variant to cctools.

[ken-cunningham-webuse]

In 5698d9483b44984a826d0f41154a4381b7957223/macports-ports (master):

ld64: default to +ld64_xcode on Xcode9+

ld64 does not understand tapi in the open-source
versions at present.

closes: #56843
closes: #53784
closes: #56398
closes: #56277
see: #56647
see: #54510
see: #54506
see: #53151