-
Notifications
You must be signed in to change notification settings - Fork 11
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
Do not overwrite default linker flags so that LDFLAGS environment variable is considered #110
Conversation
…iable is considered
FYI @Yeshasvitvs @lrapetti |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tweaking the linker flags is a legacy that originated here in wearables few years ago and I personally spread all over the projects I either developed or maintained. Thanks for taking care of all the repositories that were affected, I'm going to accept all the PRs without commenting all of them 😉
I leave handling the releasing process to the current developers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @traversaro
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @traversaro, I'll take care of creating a new tag
@traversaro Done https://github.com/robotology/wearables/releases/tag/v1.2.1 |
If one wants to specify additional linked flags, those should be appended to
CMAKE_SHARED_LINKER_FLAGS
, not overwrite it as this would lead to theLDFLAGS
environment variable to be ignored, leading to linking errors in some environments (for example conda).For more details see:
It would be convenient to tag a released after this fix is integrated to fix the conda binary generation at the robotology-superbuild level (see robotology/robotology-superbuild#681). Let me know if you want to do that or you like me to do it.
Fix robotology/robotology-superbuild#681 .