Bug 17078 - Licenses statements are missing from source files and tarball
Summary: Licenses statements are missing from source files and tarball
Alias: None
Product: XCB
Classification: Unclassified
Component: Utils (show other bugs)
Version: unspecified
Hardware: Other All
: high major
Assignee: Jamey Sharp
QA Contact:
Keywords: licence
Depends on:
Reported: 2008-08-11 08:15 UTC by Michal Nowak
Modified: 2008-11-21 01:11 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

adds copyright statements and licenses to .c files which need them (22.06 KB, patch)
2008-11-20 15:56 UTC, Jonathan Landis
Details | Splinter Review

Description Michal Nowak 2008-08-11 08:15:43 UTC
I am missing some clearer license statement in xcb-util tarball and source files. Four or five of the .c files, from ~15, have some sort of BSD license in header, but dozen of them does not. 

In Red Hat BZ https://bugzilla.redhat.com/show_bug.cgi?id=452427 there's ongoing package review for awesome v2, but I wish to update it to v3, which triggers xcb-util. 

I feel it will be problem at time of package review of xcb-util (as a new dep for awesome) in case the license is not clear enough.

(Cc-ing Julien -- awesome maintainer/creator and publisher of xcb-0.2.1)
Comment 1 Julien Danjou 2008-08-11 08:22:10 UTC
I already put this subject on the table[1] in March, but no further action had been taken at that time.

In the mean time, I got my fdo account and XCB repository access so I might review my patches and apply (some of) them.

[1] http://lists.freedesktop.org/archives/xcb/2008-March/003403.html
Comment 2 Michal Nowak 2008-10-13 01:37:14 UTC
In v3 it is still a problem. Now it blocked my package review:

Comment 3 Jonathan Landis 2008-11-20 15:56:52 UTC
Created attachment 20480 [details] [review]
adds copyright statements and licenses to .c files which need them

made using 'git diff'
Comment 4 Jonathan Landis 2008-11-20 15:58:55 UTC
Attached please find a patch for this bug.  I'm planning to use xcb in a project of mine, and I'd like to contribute something back to xcb.  Below are some notes clarifying how the patch was made.

As stated in the original bug report, some files contain license blocks and some don't.  Of those that contain the license blocks, the licenses appear to be equivalent.  Hence the only problem is figuring out who the authors are for those files without copyright statements.  The determination of authorship was made using 'git blame' against a clone of the current master.

The license body was taken from the file iccm/icccm.c.  There is no particular reason for this except that iccm.c was first on the list of files I looked at.  Licenses were only added to files with extension .c, as it appears these are the only files of concern.  Any existing license on a file was left alone (except possibly for the copyright character as described below), and the list of authors was not modified on any file which already had a copyright statement.

Note that reply_formats.c originally had only a license "stub" with no body-- it referred to a non-existent COPYING file.  The reference to the COPYING file was replaced with the actual license body, and the list of authors was taken from the original copyright stub.  That is, the copyright statement was unchanged.

The phrase "Copyright (C|c)" was manually replaced with "Copyright ©", even on files with existing copyright statements and licenses.  I believed at the time the appearance of the special symbol actually mattered, but looking back now I think actually it does not.  For the case of US law, see http://www.law.cornell.edu/uscode/17/401.html.  In any case the change doesn't hurt anything, and it brings the files into closer uniformity, as some already had the proper copyright character.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.