Set the LINK_INTERFACE_LIBRARIES and INTERFACE_LINK_LIBRARIES #54

Merged
SlavekB merged 1 commits from fix/link-private-ftbfs into master 4 years ago
Owner

This resolve FTBFS when using private linking if the build is
of type Debug. This is the case I use on my FreeBSD test machine.

This resolve FTBFS when using private linking if the build is of type Debug. This is the case I use on my FreeBSD test machine.
SlavekB added this to the R14.0.9 release milestone 4 years ago
MicheleC reviewed 4 years ago
MicheleC left a comment
Owner

perhaps there is room for improving the code?

perhaps there is room for improving the code?
set_target_properties( ${_target} PROPERTIES
LINK_INTERFACE_LIBRARIES "${_shared_libs}"
INTERFACE_LINK_LIBRARIES "${_shared_libs}" )
string( TOUPPER ${CMAKE_BUILD_TYPE} _build_type )
Owner

Why don't do this before line 1075 and then have a single call to "set_target_properties" instead of calling it twice? Am I missing something?

Why don't do this before line 1075 and then have a single call to "set_target_properties" instead of calling it twice? Am I missing something?
Poster
Owner

Yes, good point, I'll combine it into one call and test.

Yes, good point, I'll combine it into one call and test.
MicheleC approved these changes 4 years ago
MicheleC left a comment
Owner

LGTM now

LGTM now
SlavekB closed this pull request 4 years ago
SlavekB deleted branch fix/link-private-ftbfs 4 years ago
The pull request has been merged as e769cc0003.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tde-cmake#54
Loading…
There is no content yet.