Skip to content

Include Full Apache-2.0 License text#1010

Open
bohemian-miser wants to merge 1 commit into
mozilla:mainfrom
bohemian-miser:patch-2
Open

Include Full Apache-2.0 License text#1010
bohemian-miser wants to merge 1 commit into
mozilla:mainfrom
bohemian-miser:patch-2

Conversation

@bohemian-miser
Copy link
Copy Markdown

Section 4.a. of the Apache 2.0 license, states that a full copy of the license needs to travel with the software whenever it is redistributed.

I pointed this out in the previous pull request, but the PR was already closed and ignored #1009

The confusion came from the appendix regarding license headers in source code and was misread as applying to the project.

Section 4.a. of the Apache 2.0 license, states that a full copy of the license needs to travel with the software whenever it is redistributed.

I pointed this out in the previous pull request, but the PR was already closed and ignored mozilla#1009 

The confusion came from the appendix regarding license headers in source code and was misread as applying to the project.
Copy link
Copy Markdown
Contributor

@gijsk gijsk left a comment

Choose a reason for hiding this comment

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

Thanks for the renewed PR.

For context, I did not intentionally ignore your PR or comments, I have simply been busy and didn't get around to yours yet.

My understanding here is that if we're updating things, ideally the LICENSE file should contain the contents of https://www.apache.org/licenses/LICENSE-2.0.txt verbatim with no changes, and a NOTICE file should be used to include copyrights (instead of leaving them at the top of the license file).

Would you mind updating the PR accordingly?

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.

2 participants