Closed
Bug 232131
Opened 22 years ago
Closed 21 years ago
Relicense xpinstall to tri-license
Categories
(mozilla.org :: Miscellaneous, task)
mozilla.org
Miscellaneous
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerv, Assigned: gerv)
Details
Attachments
(1 file, 1 obsolete file)
35.28 KB,
application/x-tar
|
Details |
We are relicensing the entire Mozilla codebase to MPL/LGPL/GPL. For no
particular reason, other than it's an average directory, I've decided to start
with xpinstall.
This bug is to allow interested parties to review the performance of the
relicensing script. I will be attaching the patch here. Further relicensings are
expected to proceed without a bug (or all under the same bug number.)
jst had particular concerns about whitespace; CCing him. It seems that the new
text does not have trailing whitespace, but the bare-line comment characters do.
i.e. you get
+ * the terms of any one of the MPL, the GPL or the LGPL.<no whitespace here>
+ * <one whitespace here>
+ * ***** END LICENSE BLOCK ***** */<no whitespace here>
Is that an issue? If so, I'm sure a fix could be found.
CCing also the two XPInstall owner accounts, in case anyone is watching them.
Gerv
Assignee | ||
Comment 1•22 years ago
|
||
Here's the gzipped diff (36k gzipped, 1.1Mb raw).
Gerv
Assignee | ||
Comment 2•22 years ago
|
||
Oops; forgot to CC jst.
Gerv
Comment 3•22 years ago
|
||
I'd rather see this go in w/o any trailing whitespace in the lisence at all. Can
I help with looking at the script that does these changes?
Assignee | ||
Comment 4•22 years ago
|
||
Trent is normally really responsive; I'll just drop him a line.
Gerv
Comment 5•22 years ago
|
||
I have send an updated relic.py (0.5.7) to Gerv to fix the trailing-whitespace
problem.
Assignee | ||
Comment 6•22 years ago
|
||
Here's a diff from the latest version of relic, which suppresses the spaces JST
was concerned about. Could anyone with further issues please raise them? Ta :-)
Gerv
Assignee | ||
Updated•22 years ago
|
Attachment #139866 -
Attachment is obsolete: true
Comment 7•22 years ago
|
||
Looks goot to me, fwiw.
Comment 8•22 years ago
|
||
xpinstall/packager/unix/deliver.pl lost its usage comment block describing the
script arguments -- Please restore. Other than that it looks great, thanks for
the hard work and perseverance!
Assignee | ||
Comment 9•21 years ago
|
||
Fixed. Checkin log omitted for brevity :-)
Gerv
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•