Bug 16432

Summary: [GTK] Update license headers
Product: WebKit Reporter: Alp Toker <alp>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: christian, mjs, xan.lopez
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Update license headers for stub files mjs: review+

Alp Toker
Reported 2007-12-13 21:28:15 PST
This patch is the result of an informal audit of contributions using SVN history. Files are re-licensed to LGPL where the consent of all listed copyright holders has been given. Copyright header entries that were introduced accidentally during stubbing have been removed. Copyright header entries for stubs-only files have been removed and can be considered empty templates until they are implemented. Where SVN history is ambiguous the code (nearly all of which is stub code) has been re-written, whilst maintaining the original behaviour. The only case this was necessary was in FrameLoaderClientGtk.cpp. This patch requires additional authorization from these copyright holders before it can be landed: Christian Dywan <christian@twotoasts.de> Xan Lopez <xan@gnome.org> No authorization is required from Apple Inc. since files containing non-trivial content (more than a couple of lines of non-stub code) by Apple are not modified by this patch. (Minor coding style fixes are included for the kinds of issues that you only notice when you have to scrutinize the entire code-base of a port for days on end -- typically whitespace fixes or removal of stdio.h) Only GTK+ port files are modified. (No modifications to curl or cairo backends.)
Attachments
Update license headers for stub files (60.78 KB, patch)
2007-12-13 21:36 PST, Alp Toker 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alp Toker
Comment 1 2007-12-13 21:36:33 PST
Created attachment 17888 [details] Update license headers for stub files This is the fix discussed in the bug report.
Xan Lopez
Comment 2 2007-12-14 01:44:16 PST
Ok with me.
Christian Dywan
Comment 3 2007-12-14 01:55:25 PST
You have my full agreement to proceed with the license change to the LGPL.
Alp Toker
Comment 4 2007-12-14 03:15:41 PST
Comment on attachment 17888 [details] Update license headers for stub files The two remaining acknowledgements have come from Xan and Christian for the re-licensing to go ahead. Maciej, r?
Maciej Stachowiak
Comment 5 2007-12-14 15:33:54 PST
Comment on attachment 17888 [details] Update license headers for stub files r=me A few spots here added extra spaces.
Alp Toker
Comment 6 2007-12-14 17:06:26 PST
Landed in r28728. (The whitespace was added to match the formatting of the standard LGPL boilerplate.)
Note You need to log in before you can comment on or make changes to this bug.