Closed
Bug 760009
Opened 12 years ago
Closed 12 years ago
MPL 2 upgrade: comm-central (cleanup)
Categories
(mozilla.org :: Licensing, task)
mozilla.org
Licensing
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerv, Assigned: gerv)
References
Details
We have upgraded most of comm-central in bug 757018; this bug is for dealing with any mistakes or missed files. Gerv
Assignee | ||
Comment 1•12 years ago
|
||
Florian writes in bug 759808: > There's a comment at the top of [installer/windows/nsis/updater_append.ini] stating that > it's important that this file starts with a line break, and this file is used to append it > to another file that already has a license header, so I think we don't want the header here. > The file exists for other Mozilla applications in mozilla-central and comm-central, and I > think you will want to revert the change to these files. Gerv
Assignee | ||
Comment 2•12 years ago
|
||
Florian also writes:
> instantbird/themes/accounts-aero.css
> instantbird/themes/blist-aero.css
> instantbird/themes/conversation-aero.css
> instantbird/themes/instantbird-aero.css
> instantbird/themes/tabbrowser-winstripe/tabbrowser-aero.css
> These css files start by %include'ing their non -aero version, so adding
> the license header in them with a /* */ comment format would duplicate the
> header in the resulting files. I think it's good to add a license header to
> these files, but it should be with a "% " prefix, so that the headers are
> preprocessed out. The same situation exists on comm-central.
Gerv
Assignee | ||
Comment 3•12 years ago
|
||
See also bug 759816. Gerv
![]() |
||
Comment 4•12 years ago
|
||
Discovered in Bug 794777 Comment 3: http://mxr.mozilla.org/comm-central/source/suite/branding/nightly/icons/svg/README#8
Assignee | ||
Comment 5•12 years ago
|
||
Comment 4 fix: https://hg.mozilla.org/comm-central/rev/544cc63da917 Gerv
Assignee | ||
Comment 6•12 years ago
|
||
File new bugs for any additional problems. Gerv
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•12 years ago
|
||
Ah. Probably not :-) Gerv
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 9•12 years ago
|
||
:florian: I can't find the files listed in comment 2 in a current checkout of comm-central. Can you check whether they still exist and, if so, tell me where they are? Thanks, Gerv
Comment 10•12 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #9) > Can you check whether they still exist and, if so, tell me > where they are? They exist both in comm-central and mozilla-central: http://mxr.mozilla.org/comm-central/find?text=&string=-aero.css The specific file names given in comment 2 are the files that live in the Instantbird repository.
Comment 11•12 years ago
|
||
A look at the more thorough MXR query http://mxr.mozilla.org/comm-central/search?string=%include&find=css shows that these 3 files may also want to have their license header preprocessed out: http://mxr.mozilla.org/comm-central/source/mail/components/im/themes/chat.css http://mxr.mozilla.org/comm-central/source/mozilla/browser/base/content/highlighter.css http://mxr.mozilla.org/comm-central/source/mozilla/mobile/xul/themes/core/tablet.css
Assignee | ||
Comment 12•12 years ago
|
||
:flo: I tried to fix this using my script, but it's a big pain in the backside because the script firstly assumes MPL2 is correct, and then if you force past that it doesn't detect the license block as one it needs to remove, and if you fix that it assumes the comment character currently in use is the correct one for any replacement license. <sigh> These files are correctly licensed. If someone wants to go through and change the comment character, I will applaud them from the sidelines :-) Gerv
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•