This repository was archived by the owner on Nov 8, 2023. It is now read-only.
Resolve licence technicality: Add explicit MPL-2.0 licence notice "Exhibit A"#56
Merged
fitzgen merged 1 commit intofitzgen:masterfrom May 18, 2023
Merged
Resolve licence technicality: Add explicit MPL-2.0 licence notice "Exhibit A"#56fitzgen merged 1 commit intofitzgen:masterfrom
fitzgen merged 1 commit intofitzgen:masterfrom
Conversation
fitzgen
approved these changes
May 9, 2023
Owner
|
This seems to have conflicts that need resolution/rebasing. |
Unlike most other Free Software licences, the MPL explicitly says it
applies only to files with a specific notice:
1.4. "Covered Software"
means Source Code Form to which the initial Contributor has attached
the notice in Exhibit A, the Executable Form of such Source Code
Form, and Modifications of such Source Code Form, in each case
including portions thereof.
Unfortunately, this means that copying the MPL text into tree, and
setting the "license" Cargo option, leaves an ambiguous situation.
One might presume that the intent was to actually *use* the MPL for
the whole project, but the legal licence text explicitly rejects that.
Thankfully, according to the licence text, it is not actually
necessary to add the notice to *every* file:
If it is not possible or desirable to put the notice in a particular
file, then You may include the notice in a location (such as a LICENSE
file in a relevant directory) where a recipient would be likely to look
for such a notice.
So clarify this situation by adding one central explicit copy of the
MPL "Exhibit A" text, and declare it to apply to everything.
Putting it in Cargo.toml puts it next to the "license =" tag. Another
possibility would be `README.md` but IMO this legal technicality
doesn't really warrant such exposure.
(Putting it in the LICENCE file, as the text itself suggests, would
mean either (i) renaming the verbatim copy of the MPL 2.0 and writing
a new file or (ii) adding it as a rubric to the top of the MPL 2.0
text in LICENCE - resulting in a LICENCE file which is not identical
to the usual MPL-2.0 text file. Neither of those seem desirable.)
Contributor
Author
|
Thanks, have rebased. |
fitzgen
approved these changes
May 18, 2023
Contributor
Author
|
Thanks :-). Do you expect to make a release with this in any time soon? Currently we (Tor, Arti) have a bodge in our tree but we'd like to adopt |
Owner
|
Published. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Unlike most other Free Software licences, the MPL explicitly says it applies only to files with a specific notice:
Unfortunately, this means that copying the MPL text into tree, and setting the "license" Cargo option, leaves an ambiguous situation. One might presume that the intent was to actually use the MPL for the whole project, but the legal licence text explicitly rejects that.
Thankfully, according to the licence text, it is not actually necessary to add the notice to every file:
So clarify this situation by adding one central explicit copy of the MPL "Exhibit A" text, and declare it to apply to everything.
Putting it in Cargo.toml puts it next to the "license =" tag. Another possibility would be
README.mdbut IMO this legal technicality doesn't really warrant such exposure.(Putting it in the LICENCE file, as the text itself suggests, would mean either (i) renaming the verbatim copy of the MPL 2.0 and writing a new file or (ii) adding it as a rubric to the top of the MPL 2.0 text in LICENCE - resulting in a LICENCE file which is not identical to the usual MPL-2.0 text file. Neither of those seem desirable.)