Skip to content
This repository has been archived by the owner on Jan 5, 2025. It is now read-only.

Do not overwrite default linker flags so that LDFLAGS environment variable is considered #18

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

traversaro
Copy link
Member

If a CMake build system of a project wants to specify additional linked flags, those should be appended to the CMAKE_EXE_LINKER_FLAGS CMake variable. However, the original value of CMAKE_EXE_LINKER_FLAGS should not been overwritten, as this would lead to the LDFLAGS environment variable to be ignored, leading to linking errors in some environments that exploit this environment variable to specify the location of the libraries.

For more details see:

@Nicogene Nicogene merged commit 075d963 into master Aug 5, 2021
@Nicogene Nicogene deleted the fix_ldflags branch August 5, 2021 13:46
@Nicogene
Copy link
Member

Nicogene commented Aug 5, 2021

Merged, thanks!

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

Successfully merging this pull request may close these issues.

2 participants