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

Change license to Apache-2.0 with LLVM exceptions #3793

Merged
merged 5 commits into from
May 14, 2024
Merged

Conversation

matthiasdiener
Copy link
Contributor

@matthiasdiener matthiasdiener commented Mar 26, 2024

(per our email discussion)

Same could be done for charm4py, projections, ...

@matthiasdiener matthiasdiener self-assigned this Mar 26, 2024
@ericjbohm ericjbohm added this to the 7.1 milestone Mar 26, 2024
Copy link
Contributor

@PhilMiller PhilMiller left a comment

Choose a reason for hiding this comment

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

Not that you need my approval, but I'm glad to see this finally come out!

@matthiasdiener
Copy link
Contributor Author

Next step:

add a NOTICE file (similar to https://github.com/spack/spack/blob/develop/NOTICE) with the following information:

  • acknowledging UIUC, and Charmworks, and multiple grants (not individually listed) from NSF, NIH, DoE that directly or indirectly supported this work.

@lvkale
Copy link
Contributor

lvkale commented Apr 20, 2024

When we change this, we should also change the license in the readthedocs (or remove it from there)
https://charm.readthedocs.io/en/latest/license.html

@ericjbohm ericjbohm modified the milestones: 7.1, 8.0 May 1, 2024
@ericjbohm
Copy link
Contributor

Changed to target Charm-8.0.0, which will just be Charm-7.0.0 with the license and related doc changes.

@matthiasdiener
Copy link
Contributor Author

When we change this, we should also change the license in the readthedocs (or remove it from there) https://charm.readthedocs.io/en/latest/license.html

That license is just #included from the main LICENSE file, so a change there will automatically be reflected on readthedocs.

@ericjbohm ericjbohm requested review from bhatele and lvkale and removed request for lvkale May 13, 2024 15:48
@ericjbohm
Copy link
Contributor

Next step:

add a NOTICE file (similar to https://github.com/spack/spack/blob/develop/NOTICE) with the following information:

  • acknowledging UIUC, and Charmworks, and multiple grants (not individually listed) from NSF, NIH, DoE that directly or indirectly supported this work.

Draft of NOTICE file added to this PR.

@ericjbohm ericjbohm merged commit b00a533 into main May 14, 2024
23 checks passed
@ericjbohm ericjbohm deleted the license-apache branch May 14, 2024 21:33
ericjbohm pushed a commit that referenced this pull request May 15, 2024
* Change license to Apache-2.0 with LLVM exceptions

* docs : add NOTICE file for legal disclaimer

* docs: Add CHANGES for license change only release 7.0.1

* Update NOTICE

* Update NOTICE

---------

Co-authored-by: Eric J Bohm <[email protected]>
Co-authored-by: Abhinav Bhatele <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants