Closed Bug 760009 Opened 11 years ago Closed 10 years ago
MPL 2 upgrade: comm-central (cleanup)
We have upgraded most of comm-central in bug 757018; this bug is for dealing with any mistakes or missed files. Gerv
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
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
See also bug 759816. Gerv
File new bugs for any additional problems. Gerv
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Has comment 2 here already been addressed?
Ah. Probably not :-) Gerv
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
: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
(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.
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
: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: 10 years ago → 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.