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.

Sign up for GitHub

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

Jump to bottom

Add MPL2 license #31

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add MPL2 license #31

wants to merge 2 commits into from

Conversation

micaeljtoliveira
Copy link
Contributor

@micaeljtoliveira micaeljtoliveira commented May 17, 2023 •
edited
Loading

No description provided.

@micaeljtoliveira micaeljtoliveira force-pushed the micaeljtoliveira-patch-1 branch from b827904 to a3123ea Compare May 17, 2023 03:59
@micaeljtoliveira micaeljtoliveira changed the title Add LICENSE Add MPL2 license May 17, 2023
@micaeljtoliveira
Copy link
Contributor Author

@aekiss I decided to add a license to these tools. Would you agree with the choice?

@micaeljtoliveira micaeljtoliveira requested a review from aekiss May 17, 2023 04:02
@aekiss
Copy link
Contributor

aekiss commented May 17, 2023

I'm happy to go with whatever license you think best

@aekiss
Copy link
Contributor

aekiss commented May 17, 2023

should check with @russfiedler though - he wrote most of these I believe.

Except kdtree2, which already has its own license - is that a problem? https://github.com/COSIMA/domain-tools/blob/d4dcf41222c4b9c474fa85490c17d0cf68b114d4/src/kdtree2.f90#L4

@micaeljtoliveira
Copy link
Contributor Author

should check with @russfiedler though - he wrote most of these I believe.

Yes, you are absolutely right! This needs to be approved by @russfiedler

Except kdtree2, which already has its own license - is that a problem?

No. As far as I can tell, the licenses are compatible.

@micaeljtoliveira micaeljtoliveira force-pushed the micaeljtoliveira-patch-1 branch from c90d2d0 to 5b27819 Compare May 17, 2023 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aekiss aekiss Awaiting requested review from aekiss

@russfiedler russfiedler Awaiting requested review from russfiedler

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@micaeljtoliveira @aekiss