Skip to content

Change license text to LGPL-2.1#511

Merged
brianmay merged 1 commit intosshuttle:masterfrom
Rylan12:license-update
Aug 26, 2020
Merged

Change license text to LGPL-2.1#511
brianmay merged 1 commit intosshuttle:masterfrom
Rylan12:license-update

Conversation

@Rylan12
Copy link
Contributor

@Rylan12 Rylan12 commented Aug 26, 2020

Hi there, Homebrew maintainer here.

While trying to add license information about sshuttle to Homebrew (see Homebrew/homebrew-core#60154), we ran into some confusion about sshuttle's license.

As seen in setup.py, LGPL-2.1-or-later seems to be the correct license:

sshuttle/setup.py

Lines 7 to 10 in 73eb3b6

# sshuttle is free software: you can redistribute it and/or modify
# it under the terms of the GNU Lesser General Public License as
# published by the Free Software Foundation; either version 2.1 of
# the License, or (at your option) any later version.

However, the LICENSE file still contains the text for LGPL-2.0.

This PR changes the license text to reflect the LGPL-2.1 license. If it's the other way around (meaning that this should be LGPL-2.0 and the setup.py header is incorrect, I would be happy to have this PR make those changes instead).

@brianmay
Copy link
Member

Good question. For reference this:

It looks like there is a bit of confusion of the copyright in the upstream project. In fact one of the copyright headers looks like a copy and paste from on of my projects (including the project name!). https://github.com/apenwarr/sshuttle/blob/master/setup.py

Having said that, I really don't think anybody is going to be upset with this proposed changed. It already has been LGPL2.1+ for almost 3 years in setup.py.

Will merge. Thanks.

@brianmay brianmay merged commit e8f3b53 into sshuttle:master Aug 26, 2020
@Rylan12 Rylan12 deleted the license-update branch August 26, 2020 22:06
@Rylan12
Copy link
Contributor Author

Rylan12 commented Aug 26, 2020

Thanks, @brianmay!

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