Skip to content

Change license to Apache-2.0 with LLVM exceptions#3793

Merged
ericjbohm merged 5 commits intomainfrom
license-apache
May 14, 2024
Merged

Change license to Apache-2.0 with LLVM exceptions#3793
ericjbohm merged 5 commits intomainfrom
license-apache

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
@matthiasdiener matthiasdiener marked this pull request as ready for review March 26, 2024 21:54
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
@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 <ebohm@illinois.edu>
Co-authored-by: Abhinav Bhatele <bhatele@illinois.edu>
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