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

Automatically insert a MPL LICENSE file #4734

Closed
davidhedlund opened this issue Sep 4, 2017 · 9 comments
Closed

Automatically insert a MPL LICENSE file #4734

davidhedlund opened this issue Sep 4, 2017 · 9 comments

Comments

@davidhedlund
Copy link

davidhedlund commented Sep 4, 2017

Describe the problem and steps to reproduce it:

The most popular license on AMO is MPL 2.0.

What happened?

Most of them do not ship with a LICENSE at all.

What did you expect to happen?

Anything else we should know?

The Free Software Directory (https://directory.fsf.org/wiki/Free_Software_Directory:IceCat_WebExtensions_(proposed)) don't approve add-ons that are said to be distributed under MPL on AMO but doesn't ship with a copy of the license.

@jvillalobos
Copy link

Automatically changing files in add-ons is error-prone, so we don't do it unless strictly necessary, which isn't the case here.

Adding a linter warning about the LICENCE file if the add-on has one of the preset open source licenses could be useful, though. @wagnerand, @kewisch thoughts?

As for exposing the presence of the file in the API, I think that's too specific.

@diox
Copy link
Member

diox commented Sep 4, 2017

Adding a linter warning about the LICENCE file if the add-on has one of the preset open source licenses could be useful, though. @wagnerand, @kewisch thoughts?

It'd be nice, but the licence choice happens after validation in the submission flow (both for new submissions and updates) so the only warning we can add really is just a generic "Your add-on does not contain a LICENCE file" at validation time, without knowing what licence will be chosen later in the process.

@jvillalobos
Copy link

For updates we at least know what was chosen for the previous version, which will most likely remain the same for the new version. I'm not opposed to a generic notice, though, since most add-ons claim to be open-source.

@wagnerand
Copy link
Member

I mostly agree with @jvillalobos although I don't believe it warrants a validation warning. We have to be careful to not overload developers with warnings, otherwise we risk that they start ignoring them.

https://developer.mozilla.org/en-US/Add-ons/Listing already mentions a license in general, we could add a note about including LICENSE files in the add-on.

@davidhedlund
Copy link
Author

davidhedlund commented Sep 5, 2017

@wagnerand

https://developer.mozilla.org/en-US/Add-ons/Listing already mentions a license in general, we could add a note about including LICENSE files in the add-on.

Yes, that is a good idea. Can you add it (The GPL licenses recommends a COPYING file.)?

@jvillalobos
Copy link

Okay, let's not bother with the validation warning now. Nothing left to do in code, then.

@davidhedlund
Copy link
Author

davidhedlund commented Sep 7, 2017

@jvillalobos Its been very painful to spend endless of hours to verify if a copy of the license file exists in each add-on for the add-on repository I'm building for GNU IceCat.

Can you please at least make a recommendation message display? If your server don't find a copy of a license file then you could send a message to the owner of the add-on during the upload process. So for example, for add-ons released under your most popular license, Mozilla Public License (MPL) 2.0, you could show this message.

We recommend that save https://www.mozilla.org/media/MPL/2.0/index.txt as file name LICENSE in the root directory.

Again, I've been working hundreds of hours on IceCats add-on list and I'm tired of contacting the add-on owners each time. Its extremely impractical to force other people to do it when you can encourage the add-on owners to do this properly.

Also, regarding #4734

https://developer.mozilla.org/en-US/Add-ons/Listing already mentions a license in general, we could add a note about including LICENSE files in the add-on.

Can you please do that?

@jvillalobos
Copy link

Again, I've been working hundreds of hours on IceCats add-on list and I'm tired of contacting the add-on owners each time.

I'm sorry you have to do that, but we need to balance various factors and decided this isn't worth doing. Adding too much messaging during the submission process makes developers less likely to pay attention to the most critical ones.

Can you please do that?

It's a wiki, so anyone can do that. I'm pretty busy myself, so I can't help you with that.

@davidhedlund
Copy link
Author

I would like Mozilla to add a license file to all 256 add-ons in https://addons.mozilla.org/en-US/firefox/user/mozilla/#my-submissions

@KevinMind KevinMind transferred this issue from mozilla/addons-server May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants