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

License issues #89

Closed
davidhedlund opened this issue Apr 26, 2016 · 20 comments
Closed

License issues #89

davidhedlund opened this issue Apr 26, 2016 · 20 comments

Comments

@davidhedlund
Copy link

There are no Mozilla License Headers (see https://www.mozilla.org/en-US/MPL/headers/) in your source files. Can you please add them?

@davidhedlund davidhedlund changed the title License notes in source files missing Add license headers Apr 26, 2016
@Quicksaver
Copy link
Owner

Those license headers are only required for files uploaded to Mozilla repos. The add-on already includes the full license at the root of the package, which extends to all files within it. The headers are not necessary as I understand it.

@davidhedlund
Copy link
Author

Sorry, I didn't told you my purpose with this issue.

You need to add finish this issue so this page can be approved: https://directory.fsf.org/wiki/Puzzle_Toolbar

The Free Software Directory guidelines require license headers. Only when you have added them your entry can be approved by a moderator like me.

Can you please do this?

@Quicksaver
Copy link
Owner

I don't understand why that would be necessary. If you're distributing the add-on in the website, as long as its current licensing info is kept (the "LICENSE" file at the root), that should be enough. If you're not distributing it, then it shouldn't make a difference.

I don't want to add more licensing info than is absolutely necessary, and I don't know this website. Why is this necessary and what does it do for the add-on?

@davidhedlund
Copy link
Author

davidhedlund commented Apr 27, 2016

"Best practices for maintaining license information

License information can also be maintained in individual files, in a central location, or in some combination of both. Most projects use a hybrid approach, placing the primary license in a top-level COPYING or LICENSE file, and also including some license information in each file’s header." - https://softwarefreedom.org/resources/2012/ManagingCopyrightInformation.html

License headers are absolutely necessary (see https://www.gnu.org/licenses/gpl-howto.en.html for example of GPL).

"grep for 'copyright' to see what some of the program headers say. A typical header will reference the license and may mention how you're allowed combine it with other files and if you may upgrade the license. If the headers reference non-free licenses then we have a problem." - https://directory.fsf.org/wiki/Free_Software_Directory:Requirements

https://directory.fsf.org/wiki/Puzzle_Toolbar will automatically be listed for the latest GNU IceCat add-on repository (see https://directory.fsf.org/wiki/Free_Software_Directory:IceCat_extensions_(proposed) once done.

@Quicksaver
Copy link
Owner

My only issues are I don't want to spend a whole day attaching headers to over 1k files (I don't have just this add-on and I prefer consistency over anything else), and I'm against it myself because license headers only increase the file sizes to add info that, in this case, is redundant because the licensing is obvious from the single file included in the package and from the info in the add-on's homepage.

But if you want, you're welcome to submit a PR with the headers yourself. ;)

BTW it is already listed at https://directory.fsf.org/wiki/IceCat/Puzzle_Toolbars, although the name and version are severely outdated.

@Quicksaver Quicksaver reopened this Apr 27, 2016
@davidhedlund
Copy link
Author

What is the current name of the add-on then?

@Quicksaver
Copy link
Owner

@davidhedlund
Copy link
Author

davidhedlund commented Apr 28, 2016

@Quicksaver
Copy link
Owner

Changing urls doesn't redirect from the old to the new ones. So bookmarks would be lost, and users using older versions of the add-on (with the old urls) wouldn't be able to reach the right place. I avoid changing urls whenever possible.

@yfdyh000
Copy link

Why "urls doesn't redirect from the old to the new ones."? https://help.github.com/articles/renaming-a-repository/

@Quicksaver
Copy link
Owner

For instance, if I changed to https://github.com/Quicksaver/Puzzle-Bars, the current https://github.com/Quicksaver/Puzzle-Toolbars would no longer work.

@davidhedlund
Copy link
Author

This is a separate issue.

This was referenced Sep 20, 2016
@Quicksaver
Copy link
Owner

I'm sorry this took so long.

@davidhedlund
Copy link
Author

No worries, you did it a last.

@davidhedlund
Copy link
Author

davidhedlund commented Sep 29, 2016

@Quicksaver Said

Releasing new versions on Addons.Mozilla.Org with the updated files with
the headers is another matter. I have several more things to add and
bugs to fix to the add-ons, so that will definitely take a while. But if
you like I can still notify you when I update those as well.

Richard responded

Please reply when that happens.

This is a long-term issue, not a super hurry.

Can you please re-open this issue until then?

@Quicksaver
Copy link
Owner

Sure.

@Quicksaver Quicksaver reopened this Sep 29, 2016
@davidhedlund
Copy link
Author

@Quicksaver Thank you.

@davidhedlund davidhedlund changed the title Add license headers License issues Oct 25, 2016
@Quicksaver
Copy link
Owner

@davidhedlund replied to the e-mail as well. Done. :)

@davidhedlund
Copy link
Author

@Quicksaver Thanks.

@davidhedlund
Copy link
Author

davidhedlund commented Sep 4, 2017

Previously I recommended a modification of the GPL guidelines which was wrong. Now I've had a fresh look on the actual license.

From https://www.mozilla.org/en-US/MPL/2.0/

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.

With other words, you can choose to use LICENSE, but no license notices in the source files.

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

No branches or pull requests

3 participants