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

License Change to BSD-3-Clause #1613

Merged
merged 4 commits into from
Aug 13, 2024
Merged

License Change to BSD-3-Clause #1613

merged 4 commits into from
Aug 13, 2024

Conversation

havogt
Copy link
Contributor

@havogt havogt commented Aug 12, 2024

GT4Py is now licensed under the BSD license. SPDX-License-Identifier: BSD-3-Clause

Please, refer to the LICENSE file in the root directory.

Contributors to GT4Py are not required to sign the GridTools Contributor Assignment Agreement.

@havogt havogt requested a review from mbianco August 12, 2024 15:40
Copy link
Contributor

@edopao edopao left a comment

Choose a reason for hiding this comment

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

One reference to GPL license to check:

docs/development/GDPs/gdp-0001-standalone-cli.rst
Using only the GT4Py generated stencils in a project without depending on GT4Py at runtime allows to use a licence other than GPL3 in said project without the express permission of CSCS.

Other than that (and the year in LICENSE.txt) the PR looks good to me.

LICENSE.txt Outdated
<https://www.gnu.org/licenses/why-not-lgpl.html>.
GT4Py - GridTools Framework

Copyright (c) 2014-2023, ETH Zurich
Copy link
Contributor

Choose a reason for hiding this comment

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

2024?

LICENSE.txt Outdated Show resolved Hide resolved
@havogt
Copy link
Contributor Author

havogt commented Aug 13, 2024

Thanks, Edoardo. I left the reference in the GDP as it is a design document from the time where this statement was relevant. (Actually, I changed it first yesterday, but then changed it back.)

Copy link

@mbianco mbianco left a comment

Choose a reason for hiding this comment

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

Seems good to go!

@havogt havogt merged commit f18e4d6 into GridTools:main Aug 13, 2024
54 of 55 checks passed
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.

3 participants