Skip to content

CMake Policy Update#3411

Merged
scottfurry merged 2 commits into
sqlitebrowser:masterfrom
scottfurry:CMakeCMP
Jul 22, 2023
Merged

CMake Policy Update#3411
scottfurry merged 2 commits into
sqlitebrowser:masterfrom
scottfurry:CMakeCMP

Conversation

@scottfurry

Copy link
Copy Markdown
Contributor

Move the existing CMake Policy action statements to as high as possible in script.

Also adding action for CMP0102. URL discussing policy in code.

Move the existing CMake Policy action statements to as high as possible in script.

Also adding action for CMP0102. URL discussing policy in code.
Move the existing CMake Policy action statements to as high as possible in script.

Also adding action for CMP0102. URL discussing policy in code.
Added URL for CMP0025 for clarity.
@scottfurry

Copy link
Copy Markdown
Contributor Author

Added URL for CMP0025 for completeness. Amended/Rebase Pull.

@justinclift justinclift left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No real objections here. Everything seems to work, which is the important point. 😁

@scottfurry

Copy link
Copy Markdown
Contributor Author

Thanks @justinclift
Pushing through now.

@scottfurry scottfurry marked this pull request as ready for review July 22, 2023 06:47
@scottfurry scottfurry merged commit 9a717bc into sqlitebrowser:master Jul 22, 2023
@scottfurry scottfurry deleted the CMakeCMP branch July 22, 2023 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants