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.

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

Enable MACOSX_RAPTH for Mac building #73

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

mguiral
Copy link
Member

@mguiral mguiral commented Mar 13, 2017

No description provided.

@coveralls
Copy link

coveralls commented Mar 13, 2017

Coverage Status

Coverage remained the same at 69.743% when pulling ff2299c on macosbuild into c519c45 on develop.

@cchampet
Copy link
Collaborator

Why do you need to use the rpaths on Mac OS X?

@mguiral
Copy link
Member Author

mguiral commented Mar 13, 2017

If i don't use it i cannot build sequenceparser on macosx. Do you know an another way to prevent this error on build ?
CMake Warning (dev):
Policy CMP0042 is not set: MACOSX_RPATH is enabled by default. Run "cmake
--help-policy CMP0042" for policy details. Use the cmake_policy command to
set the policy and suppress this warning.

MACOSX_RPATH is not specified for the following targets:

sequenceparser

This warning is for project developers. Use -Wno-dev to suppress it.

@cchampet
Copy link
Collaborator

Because of this warning you cannot build the project on OS X? I didn't know... Let me check tonight and i'll merge tomorrow ;)

@fabiencastan
Copy link
Contributor

@mguiral Should we merge this PR?

@mguiral
Copy link
Member Author

mguiral commented Sep 9, 2019

yes we can

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants