Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix @rpath reference for macOS #455

Closed
wants to merge 1 commit into from
Closed

Fix @rpath reference for macOS #455

wants to merge 1 commit into from

Commits on Aug 5, 2023

  1. Fix @rpath reference for macOS 

    The semantic equivalent of `$ORIGIN` on macOS is `@loader_path`. That seems to be the intent here, so let's use `@loader_path` instead of `@rpath`.

Setting an RPATH of `@rpath` is, in any case, non-sensical. An `@rpath` reference tells dyld (the dynamic loader) to use the known RPATHs to resolve the reference, so adding `@rpath` to the list of known RPATHs doesn't give dyld any more information than it already has.
    @carlocab
    carlocab committed Aug 5, 2023
    Configuration menu
    Copy the full SHA
    02535bb View commit details
    Browse the repository at this point in the history