Skip to content
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

Add SPDX tags to get_cpm.cmake to comply with MIT #496

Merged

Conversation

aloisklink
Copy link
Contributor

Add SPDX licensing/copyright tags to the get_cpm.cmake script, so that anybody that copies the get_cpm.cmake script into their repo (following the instructions in the README.md) will automatically comply with CPM's MIT license.

It's possible that we could just copy the entire MIT License info over (aka like what cmake/CPM.cmake does, but most modern open-source projects (like the Linux kernel) just use SPDX tags to keep things simpler.

Add [SPDX][1] licensing/copyright tags to the `get_cpm.cmake` script,
so that anybody that copies the `get_cpm.cmake` script into their repo
(following the instructions in the README.md)
will automatically comply with CPM's MIT license.

[1]: https://spdx.dev/
@@ -1,3 +1,7 @@
# SPDX-License-Identifier: MIT
#
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This empty line is needed to prevent cmake-format from automatically line-wrapping these comments.

Copy link
Member

@TheLartians TheLartians left a comment

Choose a reason for hiding this comment

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

That's an elegant solution for this, thanks!

@TheLartians TheLartians enabled auto-merge (squash) August 19, 2023 12:43
@TheLartians TheLartians merged commit 02ecc46 into cpm-cmake:master Aug 19, 2023
8 checks passed
@aloisklink aloisklink deleted the add-spdx-license-info-to-get-cpm branch August 19, 2023 15:28
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