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

compile error with dyld[29812]: Library not loaded: @rpath/libparaconf.1.dylib #566

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

compile error with dyld[29812]: Library not loaded: @rpath/libparaconf.1.dylib #566

wants to merge 5 commits into from

Conversation

Yushan-Wang
Copy link
Member

attempt to address the dyld issue

List of things to check before making a PR

Before merging your code, please check the following:

  • you have added a line describing your changes to the Changelog;
  • you have added unit tests for any new or improved feature;
  • In case you updated dependencies, you have checked pdi/docs/CheckList.md
  • you have checked your code format:
    • you have checked that you respect all conventions specified in CONTRIBUTING.md;
    • you have checked that the indentation and formatting conforms to the .clang-format;
    • you have documented with doxygen any new or changed function / class;
  • you have correctly updated the copyright headers:
    • your institution is in the copyright header of every file you (substantially) modified;
    • you have checked that the end-year of the copyright there is the current one;
  • you have updated the AUTHORS file:
    • you have added yourself to the AUTHORS file;
    • if this is a new contribution, you have added it to the AUTHORS file;
  • you have added everything to the user documentation:
    • any new CMake configuration option;
    • any change in the yaml config;
    • any change to the public or plugin API;
    • any other new or changed user-facing feature;
    • any change to the dependencies;
  • you have correctly linked your MR to one or more issues:
    • your MR solves an identified issue;
    • your commit contain the Fix #issue keyword to autoclose the issue when merged.

@Yushan-Wang
Update SuperBuild.cmake
a23a227 
attempt to address the dyld issue
@Yushan-Wang Yushan-Wang linked an issue Mar 21, 2025 that may be closed by this pull request
compile error with dyld[29812]: Library not loaded: @rpath/libparaconf.1.dylib #565
Open
@Yushan-Wang Yushan-Wang marked this pull request as draft March 21, 2025 14:55
@Yushan-Wang
Copy link
Member Author

Last Commit a23a227 does not work

jbigot
jbigot reviewed Mar 22, 2025
View reviewed changes
cmake/SuperBuild.cmake Outdated
Comment on lines 355 to 361
if("x${LD_LIBRARY_PATH}x" STREQUAL xx)
set(ENV{LD_LIBRARY_PATH} "${ADDPATH}")
set(ENV{DYLD_LIBRARY_PATH} "${ADDPATH}")
else()
set(ENV{LD_LIBRARY_PATH} "${ADDPATH}:${LD_LIBRARY_PATH}")
set(ENV{DYLD_LIBRARY_PATH} "${ADDPATH}:${DYLD_LIBRARY_PATH}")
endif()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the whole if should be duplicated to check the previous state of DYLD_LIBRARY_PATH

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the whole if should be duplicated to check the previous state of DYLD_LIBRARY_PATH

The last commit did not solve the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbigot jbigot jbigot left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

compile error with dyld[29812]: Library not loaded: @rpath/libparaconf.1.dylib
2 participants
@Yushan-Wang @jbigot