1368444 - [compare-locales] sort missing and obsolete entries by file order

Status: RESOLVED FIXED

(Localization Infrastructure and Tools :: compare-locales, enhancement)

Description

[Axel Hecht [:Pike]]

Right now, compare-locales shows missing and obsolete entries in alphabetical order of keys, which makes it harder to make use of it.

Let's keep entries in file order, somewhat.

I'll be using file order in the reference file, instead of trying to use the order in the l10n file. If they're similar, that's the same thing, if they're not, who knows how the target file seems to be ordered.

Comment 1

[Axel Hecht [:Pike]]

Attachment: bug 1368444, sort entries by file order,

I had to revert to sorted entries for left and right again,
as otherwise the obsolete entries get order wildly.

Review commit: https://reviewboard.mozilla.org/r/144118/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/144118/

Comment 2

[Matjaz Horvat [:mathjazz]]

Comment on attachment 8872586 [details]
bug 1368444, sort entries by file order,

https://reviewboard.mozilla.org/r/144118/#review150402

LGTM

Comment 3

[Axel Hecht [:Pike]]

Attachment: no bug, make tree output be hierarchical for modules, rs=foopy

This is a bit of a follow-up to the configuration landing,
making the json of both variants match for modules with
'/' in them. So devtools/client and devtools/shared get
a shared parent devtools, with client and shared being
children.

Review commit: https://reviewboard.mozilla.org/r/146600/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/146600/

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 8872586 [details]
bug 1368444, sort entries by file order,

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/144118/diff/1-2/

Comment 5

[Axel Hecht [:Pike]]

https://hg.mozilla.org/l10n/compare-locales/rev/fd159b984acf395badabb7da0b252c45c6cfcdfe, landed. Sorry for the noise about the intermediate landing :-/