716554 - MPL 2 upgrade: DOM Inspector

Status: RESOLVED FIXED

(mozilla.org :: Licensing, task)

Description

[Gervase Markham [:gerv]]

This bug tracks the MPL 2 upgrade for the project named in the subject line.
The repo is here: http://hg.mozilla.org/dom-inspector/

Gerv

Comment 1

[Gervase Markham [:gerv]]

sevenspade: is there any particular time which is good for you?

Gerv

Comment 2

[Colby Russell :crussell (no longer Mozilla-ing)]

I can review a patch containing the relicensing changes as soon as you're ready, if that's what comment 1 is referring to.

Comment 3

[Gervase Markham [:gerv]]

Attachment: Patch v.1

Here you go :-)

Gerv

Comment 4

[Colby Russell :crussell (no longer Mozilla-ing)]

Comment on attachment 587751 [details] [diff] [review]
Patch v.1

Review of attachment 587751 [details] [diff] [review]:
-----------------------------------------------------------------

::: resources/locale/ca/viewers/boxModel.dtd
@@ +1,3 @@
> +<!-- This Source Code Form is subject to the terms of the Mozilla Public
> +- License, v. 2.0. If a copy of the MPL was not distributed with this
> +- file, You can obtain one at http://mozilla.org/MPL/2.0/. -->

The indentation here is messed up for some reason.  This occurs in the following files:

/resources/locale/@AB_CD@/viewers/accessibleProps.dtd
/resources/locale/@AB_CD@/viewers/accessibleTree.dtd
/resources/locale/@AB_CD@/viewers/accessibleTreeEvalJSDialog.dtd
/resources/locale/@AB_CD@/viewers/boxModel.dtd
/resources/locale/@AB_CD@/viewers/computedStyle.dtd
/resources/locale/@AB_CD@/viewers/dom.dtd
/resources/locale/@AB_CD@/viewers/domNode.dtd
/resources/locale/@AB_CD@/viewers/jsObject.dtd
/resources/locale/@AB_CD@/viewers/styleRules.dtd
/resources/locale/@AB_CD@/viewers/stylesheets.dtd
/resources/locale/@AB_CD@/viewers/xblBindings.dtd


Where @AB_CD@ is one of {ca, cs, da, de, en-US, fi, fr, ga-IE, hu, nb-NO, pl, pt-BR, ru, sk, sv-SE}.  (Note that not all filenames that could be generated from this pattern correspond to files that exist.)

Comment 5

[Gervase Markham [:gerv]]

Thank you; bug found and fixed.

Gerv

Comment 6

[Gervase Markham [:gerv]]

Attachment: Patch v.2

Try this :-)

Gerv

Comment 7

[Colby Russell :crussell (no longer Mozilla-ing)]

Comment on attachment 590753 [details] [diff] [review]
Patch v.2

Review of attachment 590753 [details] [diff] [review]:
-----------------------------------------------------------------

I wonder if you're using an out-of-date repo?  Neither the first patch nor this one touches anything in
dom-inspector/resources/locale/en-GB/

The en-GB localization was added a few days before this bug was opened.

I also noticed that even though the first patch did, this one doesn't touch the (non-)license block in 
dom-inspector/resources/locale/sv-SE/viewers/accessibleTreeEvalJSDialog.dtd

Comment 8

[Gervase Markham [:gerv]]

Attachment: Patch v.3

Try this; tree definitely updated, and that odd file relicensed by hand.

Gerv

Comment 9

[Colby Russell :crussell (no longer Mozilla-ing)]

Comment on attachment 601962 [details] [diff] [review]
Patch v.3

Review of attachment 601962 [details] [diff] [review]:
-----------------------------------------------------------------

There are lots of occurrences in this patch where a blank line is inserted after the license block in cases where one already exists, but they're all ones where the existing "blank line" is really a line containing only spaces.  Those need to be dealt with anyway, so whenever that happens, we'll take any extra new lines out along with them.

Comment 10

[Gervase Markham [:gerv]]

http://hg.mozilla.org/dom-inspector/rev/6c425834882f

Gerv