Summary: | Licenses statements are missing from source files and tarball | ||
---|---|---|---|
Product: | XCB | Reporter: | Michal Nowak <mnowak> |
Component: | Utils | Assignee: | Jamey Sharp <jamey> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | major | ||
Priority: | high | CC: | julien, landijk-dev |
Version: | unspecified | Keywords: | licence |
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | adds copyright statements and licenses to .c files which need them |
Description
Michal Nowak
2008-08-11 08:15:43 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 In v3 it is still a problem. Now it blocked my package review: https://bugzilla.redhat.com/show_bug.cgi?id=458784#c8 Created attachment 20480 [details] [review] adds copyright statements and licenses to .c files which need them made using 'git diff' 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.