Merge lp:~widelands-dev/widelands/copyright-file into lp:~widelands-dev/widelands/debian

Proposed by Hans Joachim Desserud
Status: Superseded
Proposed branch: lp:~widelands-dev/widelands/copyright-file
Merge into: lp:~widelands-dev/widelands/debian
Diff against target: 146 lines (+131/-0)
2 files modified
debian/copyright (+97/-0)
debian/copyright.todo (+34/-0)
To merge this branch: bzr merge lp:~widelands-dev/widelands/copyright-file
Reviewer Review Type Date Requested Status
Martin Quinson Pending
GunChleoc Pending
Widelands Developers Pending
Review via email: mp+245297@code.launchpad.net

This proposal has been superseded by a proposal from 2016-01-22.

Description of the change

Update the copyright file in debian packaging to document the various licenses origins of files used. Mainly to incorporate the new fonts we now bundle, but I also thought we should probably mention our third party libraries.

We have not yet documented all of these properly though, see copyright.todo for details on remaining work. There are some questions there where I'm not sure how to proceed. I'm also open for other feedback.

I think it is possible to add individual people as reviewers, so I'll add GunChleoc (since you added the fonts) and Martin Quinson since you're the Debian maintainer and probably know more about this than me. Being appointed as reviewers will presumably ping you so that you are aware of this merge proposal. Combined I believe we can answer the questions I have posted, and get the copyright file up to date. :)

To post a comment you must log in.
Revision history for this message
GunChleoc (gunchleoc) wrote :

I used diff comments to add some more info.

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

This is surprisingly hard.

I did have some progress when I discovered that I can open fonts in FontViewer on Ubuntu and it will be able to disply embedded information such as copyright information.

Revision history for this message
SirVer (sirver) wrote :

ping. hjd, what is the state of this branch?

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

>ping. hjd, what is the state of this branch?

Hello hello. Not much has happened since the previous commit/comment.

Like I talked about in https://code.launchpad.net/~hjd/widelands/tests-poc/+merge/250533, the main intent with this branch was to get the packaging in a better shape to make it easier to create the official Debian packages once we release build19 (rc). It is my understanding that the Debian policies require packages to give full copyright statement on their content, so since it will need to be done anyways we might as well get it done early.

This work is not critical for Widelands itself since we don't really have such obligations (unless specific licenses state otherwise) but is in general nice to give credit to other people's work which we include. I'd say we are doing a good job of this though, at least with the fonts/licenses there.

As I mentioned, I believe this work will need to be done for the offical Debian packages, which is why I wanted to start it straight away, while the new fonts were still fresh. This is also why I added the Debian maintainer as a reviewer here, hoping to get some feedback and possibly guidance.

I am a bit stuck on the questions in the todo, and I don't really know how to proceed with this. I'll leave this open for a while longer, but if I don't hear anything, I'll likely close it as a draft/experiment. While I would like to see this done properly, I don't know whether the current state of the branch does more good than harm, and I don't think it is that critical for the Widelands project to have this file 100% completed.

Revision history for this message
Martin Quinson (mquinson) wrote :

I'm so deep under water wrt my daytime job that I am completely unable to help (or even properly read your emails) before the summer.

Sorry and good luck
Mt

----- Mail original -----
> >ping. hjd, what is the state of this branch?
>
> Hello hello. Not much has happened since the previous commit/comment.
>
> Like I talked about in
> https://code.launchpad.net/~hjd/widelands/tests-poc/+merge/250533, the main
> intent with this branch was to get the packaging in a better shape to make
> it easier to create the official Debian packages once we release build19
> (rc). It is my understanding that the Debian policies require packages to
> give full copyright statement on their content, so since it will need to be
> done anyways we might as well get it done early.
>
> This work is not critical for Widelands itself since we don't really have
> such obligations (unless specific licenses state otherwise) but is in
> general nice to give credit to other people's work which we include. I'd say
> we are doing a good job of this though, at least with the fonts/licenses
> there.
>
> As I mentioned, I believe this work will need to be done for the offical
> Debian packages, which is why I wanted to start it straight away, while the
> new fonts were still fresh. This is also why I added the Debian maintainer
> as a reviewer here, hoping to get some feedback and possibly guidance.
>
> I am a bit stuck on the questions in the todo, and I don't really know how to
> proceed with this. I'll leave this open for a while longer, but if I don't
> hear anything, I'll likely close it as a draft/experiment. While I would
> like to see this done properly, I don't know whether the current state of
> the branch does more good than harm, and I don't think it is that critical
> for the Widelands project to have this file 100% completed.
> --
> https://code.launchpad.net/~widelands-dev/widelands/copyright-file/+merge/245297
> You are requested to review the proposed merge of
> lp:~widelands-dev/widelands/copyright-file into
> lp:~widelands-dev/widelands/debian.
>

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

No problem Martin, that's completely understandable. Thanks for letting us know. :)

In the meantime, I'll just put this branch on ice, and we'll get back to it when we have a better idea on how to proceed. I'll try to adjust the review status so that it doesn't show up in the list of active reviews.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/copyright'
2--- debian/copyright 2014-12-13 12:37:38 +0000
3+++ debian/copyright 2014-12-27 11:47:34 +0000
4@@ -22,6 +22,103 @@
5 On Debian systems, the complete text of the GNU General Public
6 License version 2 can be found in "/usr/share/common-licenses/GPL-2".
7
8+Files: src/third_party/eris/*
9+Copyright: 2013 Florian Nücke, 1994–2013 Lua.org, PUC-Rio.
10+License:
11+Eris
12+Copyright (c) 2013 Florian Nücke.
13+
14+Permission is hereby granted, free of charge, to any person obtaining a copy
15+of this software and associated documentation files (the "Software"), to deal
16+in the Software without restriction, including without limitation the rights
17+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
18+copies of the Software, and to permit persons to whom the Software is
19+furnished to do so, subject to the following conditions:
20+
21+The above copyright notice and this permission notice shall be included in
22+all copies or substantial portions of the Software.
23+
24+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
25+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
26+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
27+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
28+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
29+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
30+THE SOFTWARE.
31+
32+-----------------------------------------------------------------------------
33+
34+Lua 5.2.2 from http://www.lua.org/ with patches http://www.lua.org/bugs.html
35+slightly modified to access internal library functions for persistence.
36+
37+Copyright © 1994–2013 Lua.org, PUC-Rio.
38+
39+Permission is hereby granted, free of charge, to any person obtaining a copy
40+of this software and associated documentation files (the "Software"), to deal
41+in the Software without restriction, including without limitation the rights
42+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
43+copies of the Software, and to permit persons to whom the Software is
44+furnished to do so, subject to the following conditions:
45+
46+The above copyright notice and this permission notice shall be included in
47+all copies or substantial portions of the Software.
48+
49+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
50+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
51+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
52+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
53+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
54+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
55+THE SOFTWARE.
56+
57+Files: i18n/fonts/Culmus/*
58+Copyright: 2010 by Yoram Gnat (gyoramg@users.sourceforge.net).
59+License: This package is distributed under the terms of GNU General Public License
60+version 2 (see file GNU-GPL).
61+---------------------------------------------------------------------
62+
63+"Taamey Frank CLM" fonts are copyright (C) 2010 by Yoram Gnat
64+(gyoramg@users.sourceforge.net). All rights reserved.
65+
66+As a special exception, if you create a document which uses
67+this font, and embed this font or unaltered portions of this
68+font into the document, this font does not by itself cause
69+the resulting document to be covered by the
70+GNU General Public License. This exception does not however
71+invalidate any other reasons why the document might be covered
72+by the GNU General Public License. If you modify this font,
73+you may extend this exception to your version of the font,
74+but you are not obligated to do so. If you do not wish to do so,
75+delete this exception statement from your version.
76+
77+Files: i18n/fonts/Nakula/*
78+Copyright: Nakula Devanagari font © 2006 University of Cambridge. Designed and developed by IMRC, India. Latin and Cyrillic glyphs from the font Nimbus Roman No9 L Regular, © 1999 (URW)++ Design & Development; Cyrillic glyphs by Valek Filippov © 2001-2002.
79+License:
80+These fonts are free software; you can redistribute them and/or modify them under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version.
81+
82+These fonts are distributed in the hope that they will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details.
83+
84+Files: i18n/fonts/Sinhala/*
85+Copyright: 2004,2008 Anuradha Ratnaweera and Harshani Devadithya, 2006 Harshula Jayasuriya, 1994, The Wellcome Trust, London
86+License:
87+ LKLUG font is free software; you can redistribute it and/or modify
88+ it under the terms of the GNU General Public License, version 2, as
89+ published by the Free Software Foundation.
90+
91+ This program is distributed in the hope that it will be useful, but
92+ WITHOUT ANY WARRANTY; without even the implied warranty of
93+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
94+ General Public License, version 2, for more details.
95+
96+ The Debian packaging is (C) 2004,2008, Anuradha Ratnaweera, and is
97+ licensed under the GNU General Public License, version 2.
98+
99+ You should have received a copy of the GNU General Public License,
100+ version 2, in the file `COPYING', or on Debian based systems in the
101+ file `/usr/share/common-licenses/GPL-2'; if not, write to the Free
102+ Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
103+ MA 02110-1301 USA.
104+
105 Files: i18n/fonts/Widelands/Widelands.ttf
106 Copyright: 2006-2010, Peter Schwanemann (Peter (AT) drehatlas (DOT) de)
107 License: SIL Open Font License, Version 1.1 (http://scripts.sil.org/OFL)
108
109=== added file 'debian/copyright.todo'
110--- debian/copyright.todo 1970-01-01 00:00:00 +0000
111+++ debian/copyright.todo 2014-12-27 11:47:34 +0000
112@@ -0,0 +1,34 @@
113+Documents known missing entries in the copyright file
114+
115+src/thirdparty/minizip/*
116+We've shipped this for a while, only the fonts below are new.
117+According to the readme it originates from zlib's minizip example, though has
118+been modified in Widelands, so we may want to document this, including our
119+changes. What's best practice here, mark it based on the original license and
120+then mention any modifications done by the Widelands team?
121+
122+i18n/fonts/DejaVu/*
123+Our bundled version mentions something about glyphs which I can't find at
124+https://tracker.debian.org/pkg/fonts-dejavu. I assumed we could follow the
125+example from that package on how to document these fonts, but I don't know if
126+the glyphs doesn't need a specific mentioning or whether the Debian package has
127+stripped them out. Anyone know?
128+
129+i18n/fonts/FaKacstBook/*
130+Presumably GPLed, but doesn't detail which version nor the copyright holder.
131+The Launchpad site says GPL v2, but the tar.gz doesn't include a license file.
132+
133+i18n/fonts/MicroHei/*
134+"This font is licensed under Apache2.0 or GPLv3
135+ with font embedding exceptions (see Appendix B)."
136+Don't know how we deal with dual-licensing in this context. Do we pick one, or
137+should both be listed?
138+
139+i18n/fonts/*mmrCensus*
140+Open Font License. Again we may need some name(s).
141+>I found their project homepage: https://code.google.com/p/myanmar3source/
142+Now I'm confused. The project page mentions the following:
143+Code license
144+GNU Lesser GPL
145+Content license
146+Creative Commons 3.0 BY

Subscribers

People subscribed via source and target branches