After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 656596 - Outdated FSF Address in code files
Outdated FSF Address in code files
Status: RESOLVED FIXED
Product: ghex
Classification: Applications
Component: general
unspecified
Other Linux
: Normal trivial
: ---
Assigned To: GHex maintainers
GHex maintainers
Depends on:
Blocks:
 
 
Reported: 2011-08-15 17:48 UTC by Dominique Leuenberger
Modified: 2011-08-17 12:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Replace the FSF addresses in the header files (24.65 KB, patch)
2011-08-16 09:01 UTC, Dominique Leuenberger
committed Details | Review

Description Dominique Leuenberger 2011-08-15 17:48:39 UTC
While building ghex on the Open Build Service (OBS), this minor warning is spit out at the end of the build:

ghex-devel.x86_64: W: incorrect-fsf-address /usr/include/gtkhex-3/hex-document.h
ghex-devel.x86_64: W: incorrect-fsf-address /usr/include/gtkhex-3/gtkhex.h
ghex.x86_64: W: incorrect-fsf-address /usr/share/doc/packages/ghex/COPYING-DOCS
ghex.x86_64: W: incorrect-fsf-address /usr/share/doc/packages/ghex/README
ghex.x86_64: W: incorrect-fsf-address /usr/share/doc/packages/ghex/COPYING
The Free Software Foundation address in this file seems to be outdated or
misspelled.  Ask upstream to update the address, or if this is a license file,
possibly the entire file with a new copy available from the FSF.

Updating COPYING and the various headers with the current FSF provided ones should resolve the issues.
Comment 1 Kalev Lember 2011-08-16 05:32:20 UTC
Thanks for pointing this out. Do you have a patch for fixing the addresses?
Comment 2 Dominique Leuenberger 2011-08-16 06:06:17 UTC
Will create one in the upcoming 12 hours if that helps.
Comment 3 Kalev Lember 2011-08-16 06:36:24 UTC
Please do.
Comment 4 Dominique Leuenberger 2011-08-16 09:01:07 UTC
Created attachment 193918 [details] [review]
Replace the FSF addresses in the header files

I hope to have caught all of them without being too intrusive.

I used 'licenscheck' to identify the files with the wrong addresses.
COPYING and COPYING-DOCS have been fully copied from 
GPL: http://www.gnu.org/licenses/gpl-2.0.txt
resp
FDL: http://www.gnu.org/licenses/fdl-1.1.txt
Comment 5 Kalev Lember 2011-08-17 12:14:28 UTC
Review of attachment 193918 [details] [review]:

Thanks, pushed to master as 2e0d7f75.

I found some very minor issues with the patch and and amended it before pushing:
 - git-am complained about 'new blank line at EOF.' for both of the COPYING files and sure enough, when compared to the downloaded FSF documents there was an extra newline in both files.
 - in the README file there was a trivial copy-paste error:
+   If not, write to the Free Software Foundation, Inc., 51 Franklin St,
+   Fifth Floor, Boston, MA  02110-1301  USA if not, write to the Free Software