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 a LICENSE file (BSD 2 clause) #207

Merged
merged 3 commits into from
Oct 18, 2024
Merged

add a LICENSE file (BSD 2 clause) #207

merged 3 commits into from
Oct 18, 2024

Conversation

hannesm
Copy link
Member

@hannesm hannesm commented Oct 17, 2024

as suggested in #206

I propose the ISC license, and it would be amazing if all contributors would agree to that:

I used the data from https://github.com/mirage/qubes-mirage-firewall/graphs/contributors as a baseline, in case I forgot anyone, please raise your voice.

To agree with the ISC license, please comment on this PR or approve this pull request. Thanks a lot!

@hannesm hannesm mentioned this pull request Oct 17, 2024
remove the LICENSE section from the README
@hannesm
Copy link
Member Author

hannesm commented Oct 17, 2024

Sorry to bother you all again, as @palainp noticed, there's a LICENSE section in the README - which is a 2 clause BSD license.

So, I revised this PR to single out the LICENSE from the README into a separate file (LICENSE.md). Approvals are not required, since there was already a license present when you contributed your code.

@hannesm hannesm changed the title add a LICENSE file (ISC) add a LICENSE file (BSD 2 clause) Oct 17, 2024
@palainp
Copy link
Member

palainp commented Oct 18, 2024

Thank you @hannesm I suppose I can merge this PR now (without any further approvals as the license has not changed)?

@hannesm hannesm merged commit 86ee78d into mirage:main Oct 18, 2024
2 checks passed
@hannesm hannesm deleted the license branch October 18, 2024 12:54
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