Avoid CMP0026 – get_target_property( LOCATION ) #26

Merged
SlavekB merged 1 commits from feat/avoid-CMP0026 into master 5 years ago
Owner

I have tried many times to find a solution for the CMP0026 that would be functional and at the same time not too strange.

The problem is that in our use, we work with value already during configuration, while the recommended replacement by generators works at the build time. This makes the use of generators for our needs complicated and strange.

I still have not found a better solution than adding a function to determine the name of the library by our own strengths. Although this is not an elegant solution, it is functional and does not look strange.

Please look at it and do not hesitate to make comments.

I have tried many times to find a solution for the CMP0026 that would be functional and at the same time not too strange. The problem is that in our use, we work with value already during configuration, while the recommended replacement by generators works at the build time. This makes the use of generators for our needs complicated and strange. I still have not found a better solution than adding a function to determine the name of the library by our own strengths. Although this is not an elegant solution, it is functional and does not look strange. Please look at it and do not hesitate to make comments.
SlavekB added the PR/rfc label 5 years ago
SlavekB added this to the R14.0.6 release milestone 5 years ago
Owner

I think we can go ahead with this. If later we find a better way, we can always change it again 😉

I think we can go ahead with this. If later we find a better way, we can always change it again :wink:
SlavekB removed the PR/rfc label 5 years ago
SlavekB closed this pull request 5 years ago
SlavekB deleted branch feat/avoid-CMP0026 5 years ago
The pull request has been merged as 0a40752779.
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#26
Loading…
There is no content yet.