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.

Sign up for GitHub

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

Jump to bottom

License only covering Proj.4? #2415

Closed
danme74 opened this issue Nov 5, 2020 · 2 comments · Fixed by #2417
Closed

License only covering Proj.4? #2415

danme74 opened this issue Nov 5, 2020 · 2 comments · Fixed by #2417
Labels
bug

Comments

@danme74
Copy link

danme74 commented Nov 5, 2020

The file COPYING contains information about licenses. As stated in master right now (2020-11-05) it only applies to Proj.4.

I assume license applies to the current code base and should be updated to reflect that.

I also suggest renaming COPYING to something more obvious such as LICENSE or LICENSE.TXT, but that's up to you. ("License" feels like a broader and more correct term than "copying".)

I would also prefer if the license information was part of the installation, similar to how GDAL (and other libraries) handle it:
/usr/share/doc/gdal-libs-1.11.4/LICENSE.TXT
@danme74 danme74 added the bug label Nov 5, 2020
rouault added a commit to rouault/PROJ that referenced this issue Nov 6, 2020
@rouault
COPYING: update
fb88313 
Fixes OSGeo#2415

- Change mention of PROJ.4 to PROJ
- Change 'I' pronoun to Frank Warmerdam
- Update copyright holder names and years with content from
  https://metadata.ftp-master.debian.org/changelogs//main/p/proj/proj_7.2.0-1_copyright
rouault added a commit to rouault/PROJ that referenced this issue Nov 6, 2020
@rouault
COPYING: update
029ef50 
Fixes OSGeo#2415

- Change mention of PROJ.4 to PROJ
- Change 'I' pronoun to Frank Warmerdam
- Update copyright holder names and years with content from
  https://metadata.ftp-master.debian.org/changelogs//main/p/proj/proj_7.2.0-1_copyright
@rouault rouault mentioned this issue Nov 6, 2020
Merged
@rouault
Copy link
Member

rouault commented Nov 6, 2020

I also suggest renaming COPYING to something more obvious such as LICENSE or LICENSE.TXT

COPYING is a standard filename for a autoconf/automake project. See https://www.gnu.org/software/automake/manual/automake.html#Gnits

@hobu
Copy link
Contributor

hobu commented Nov 6, 2020

I also suggest renaming COPYING to something more obvious such as LICENSE or LICENSE.TXT,

I disagree that COPYING is any more obvious than LICENSE. It's just the GNU way.

rouault added a commit to rouault/PROJ that referenced this issue Nov 6, 2020
@rouault
COPYING: update
b56f39d 
Fixes OSGeo#2415

- Change mention of PROJ.4 to PROJ
- Change 'I' pronoun to Frank Warmerdam
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

COPYING: update and install COPYING, NEWS and AUTHORS rouault/PROJ
3 participants
@hobu @rouault @danme74