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

false positive when checking fsf_wrong_address #85

Closed
subpop opened this issue Dec 2, 2016 · 4 comments
Closed

false positive when checking fsf_wrong_address #85

subpop opened this issue Dec 2, 2016 · 4 comments

Comments

@subpop
Copy link

subpop commented Dec 2, 2016

I ran rpmlint on a package and it printed the incorrect-fsf-address warning on the COPYING file. I looked into the file, and it appears to have the current address.

When I run the regex against the file manually using grep and it does not return a match:

 grep -E '(675\s+Mass\s+Ave|59\s+Temple\s+Place|Franklin\s+Steet|02139|02111-1307)' src-tarball/default/COPYING

Is this a false positive?

@scop
Copy link
Member

scop commented Dec 4, 2016

Hard to tell without access to the actual package being checked. Is it available somewhere online?

@subpop
Copy link
Author

subpop commented Dec 4, 2016 via email

@scop
Copy link
Member

scop commented Dec 5, 2016

The COPYING file you linked to is not the one included in the package; the one in it is an old version of the LGPL 2.1 which does contain an old FSF address.

$ curl -O https://kojipkgs.fedoraproject.org//packages/freeorion/0.4.6/4.fc25/x86_64/freeorion-0.4.6-4.fc25.x86_64.rpm
$ rpmdev-extract freeorion-0.4.6-4.fc25.x86_64.rpm
$ grep -E '(675\s+Mass\s+Ave|59\s+Temple\s+Place|Franklin\s+Steet|02139|02111-1307)' freeorion-0.4.6-4.fc25.x86_64/usr/share/licenses/freeorion/COPYING
     59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

@scop scop closed this as completed Dec 5, 2016
@subpop
Copy link
Author

subpop commented Dec 6, 2016

I found that. There are two COPYING files that I was including as the %license in the spec file. One of them does have the old address. Thanks for pointing that out!

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

2 participants