Skip to content

Add a file with the MIT license for behave/i18n.py#1039

Open
musicinmybrain wants to merge 1 commit intobehave:mainfrom
musicinmybrain:i18n-mit-license
Open

Add a file with the MIT license for behave/i18n.py#1039
musicinmybrain wants to merge 1 commit intobehave:mainfrom
musicinmybrain:i18n-mit-license

Conversation

@musicinmybrain
Copy link
Copy Markdown

@bittner
Copy link
Copy Markdown
Member

bittner commented Jun 22, 2022

Two questions:

  • Is "being generated" sufficient to require including a license from the original project?
  • As behave's license is already of the same type, could we simply extend the copyright notice in the project license instead of adding another file?

@musicinmybrain
Copy link
Copy Markdown
Author

Two questions:

* Is "being generated" sufficient to require including a license from the original project?

The usual disclaimer (I’m not a lawyer, this isn’t legal advice) applies. In the US, copyright does not protect titles, names, and short phrases, nor does it protect things like ingredient lists in recipes. However, the overall compilation of translations in the i18n file seems like it might represent sufficient creativity to qualify as a copyrightable work of authorship. If so, then

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

would apply.

It’s helpful to distributors, and most robust against accidental removal, if this is in the form of a license file, although a source code comment in i18n.py could otherwise be adequate since it would still be present in both sdists and bdists/wheels.

* As behave's license is already of the same type, could we simply extend the copyright notice in the project license instead of adding another file?

You could combine the two license files into a single file, but I don’t think you could combine the two notices. Besides the question of whether or not combining copyright notices is a correct approach for two projects with identical MIT licenses, behave’s license appears to be a 2-clause BSD license, with similar requirements but altogether different text.

@jenisys jenisys force-pushed the main branch 3 times, most recently from 7ec93d2 to 46ad983 Compare November 6, 2022 14:11
@jenisys jenisys force-pushed the main branch 4 times, most recently from fe1ca4d to fcfe5af Compare April 22, 2023 17:20
@jenisys jenisys force-pushed the main branch 2 times, most recently from 0a4d73b to 2c11d2e Compare May 14, 2024 22:39
@jenisys jenisys force-pushed the main branch 2 times, most recently from 3e51dda to c6ab01c Compare May 26, 2024 15:00
@jenisys jenisys force-pushed the main branch 3 times, most recently from eccf022 to 93e1218 Compare June 16, 2024 16:10
@jenisys jenisys force-pushed the main branch 2 times, most recently from cc16ac0 to 22569f4 Compare October 14, 2024 20:09
@jenisys jenisys force-pushed the main branch 2 times, most recently from 01407cf to 42e64a9 Compare October 24, 2024 23:28
@jenisys jenisys force-pushed the main branch 8 times, most recently from e12ed4d to 002e07a Compare July 15, 2025 18:48
@jenisys jenisys force-pushed the main branch 4 times, most recently from 612d6d1 to 5796057 Compare August 4, 2025 06:58
@jenisys jenisys force-pushed the main branch 6 times, most recently from 370ce68 to cba3c4f Compare August 31, 2025 15:00
@bittner
Copy link
Copy Markdown
Member

bittner commented Jan 17, 2026

I believe, we should strive to become REUSE compliant. REUSE introduces a LICENSES folder in a repository that maintains all licenses used in the project. In addition, all (source) files must maintain a short header with license information.

It might feel a bit cumbersome to go the REUSE route with their tooling, because it makes the code base a bit more noisy - just for adding licensing information -, but for cases like this one, for projects that have more than just one license, the pros outshine the cons, IMHO.

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