gcc-darwin-arm64 icon indicating copy to clipboard operation
gcc-darwin-arm64 copied to clipboard

Darwin: Use 'enable-darwin-at-rpath' on newer macOS versions

Open markmentovai opened this issue 2 years ago • 2 comments

Hi, Iain. This one makes sense for both https://github.com/iains/gcc-darwin-arm64 and https://github.com/iains/gcc-12-branch, and perhaps older branches.

Along with https://github.com/gcc-mirror/gcc/commit/6725f186cb70d48338f69456864bf469a12ee5be and https://patchwork.ozlabs.org/project/gcc/patch/[email protected]/, it should bring things into shape for macOS 13, modulo one SDK fix required in Xcode 14.0b1 and already reported to Apple.


https://patchwork.ozlabs.org/project/gcc/patch/[email protected]/ added enable-darwin-at-runpath and enabled it by default on macOS versions from 10.11 through 12 inclusive, but was not future-proof. macOS 13 is expected for public release later this year.

This change extends support through macOS 19, which is expected to be good for 7 more years under the current OS versioning scheme.

ChangeLog:

    * libtool.m4: Future-proof default setting of
    'enable-darwin-at-runpath'.

gcc/ChangeLog:

    * configure: Regenerate.

libatomic/ChangeLog:

    * configure: Regenerate.

libcc1/ChangeLog:

    * configure: Regenerate.

libffi/ChangeLog:

    * configure: Regenerate.

libgfortran/ChangeLog:

    * configure: Regenerate.

libgomp/ChangeLog:

    * configure: Regenerate.

libitm/ChangeLog:

    * configure: Regenerate.

libobjc/ChangeLog:

    * configure: Regenerate.

liboffloadmic/ChangeLog:

    * configure: Regenerate.

libphobos/ChangeLog:

    * configure: Regenerate.

libquadmath/ChangeLog:

    * configure: Regenerate.

libsanitizer/ChangeLog:

    * configure: Regenerate.

libssp/ChangeLog:

    * configure: Regenerate.

libstdc++-v3/ChangeLog:

    * configure: Regenerate.

libvtv/ChangeLog:

    * configure: Regenerate.

lto-plugin/ChangeLog:

    * configure: Regenerate.

zlib/ChangeLog:

    * configure: Regenerate.

Signed-off-by: Mark Mentovai [email protected]

markmentovai avatar Jun 13 '22 04:06 markmentovai

Hi Mark, thanks for the patch.

.. this seems reasonable. However, the master version of the aarch64 port really needs rebasing (I have been very busy with other work) so I think I'll probably apply this to 12.1r1 and 11.3r2 (maybe even 10.4r0) first and pull it into the master version at a later date.

iains avatar Jun 13 '22 13:06 iains

That’s no problem. I was primarily hoping to get this into iains/gcc-12-branch as that’s where MacPorts gcc-devel on mac-arm64 is currently pulling from (although it’d be great if that could move to the gcc trunk as well). Thanks.

markmentovai avatar Jun 13 '22 13:06 markmentovai

I think this is now handled OK in 12.x and master, right?

iains avatar Oct 22 '22 20:10 iains

I think this is now handled OK in 12.x and master, right?

Think so. I think that this was only a problem in your branches. Upstream doesn’t have enable_darwin_at_rpath at all.

markmentovai avatar Oct 24 '22 19:10 markmentovai

OK, thanks again for the patch - as it happened, libtool.m4 Darwin changes were reworked to deal with some other issues and we now cover a wider range of OS versions (at least until Darwin30) as part of that, so closing this pull request.

iains avatar Oct 24 '22 20:10 iains