[compare-locales] local compare-locales busted for fennec

RESOLVED FIXED

Status

Mozilla Localizations
Infrastructure
P1
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
AFAICT, I busted local compare-locales runs for fennec in bug 504819.

This is showing now as l10n-merge isn't working.

The culprit is that if we find tld, we unconditionally remove it from dirs in L10nConfigParser.onLoadConfig, but only iterate over it cleverly in the SourceTreeConfigParser.dirsIter.

Effectively, we're not running compare-locales on the mobile dir at all.

Bad. P1.

PS: Also a bad sign that for about half a year, nobody noticed that l10n merge on fennec was apparently just a lie.
(Assignee)

Updated

8 years ago
Blocks: 560844
(Assignee)

Comment 1

8 years ago
OK, this wasn't broken in production since September, because we never actually bumped the RELEASE_AUTOMATION tag to use that.

It was broken in the source though. Not sure if I had a broken release on pypi.

I confirmed that it was actually http://hg.mozilla.org/build/compare-locales/rev/35b7774af82e that regressed this locally, aka bug 504819.
Blocks: 504819
(Assignee)

Updated

8 years ago
Assignee: nobody → l10n
Status: NEW → ASSIGNED
(Assignee)

Comment 2

8 years ago
Created attachment 440586 [details] [diff] [review]
move the generation of the base path tuple into method and overload

coop, can you review this?

The basic gist of it is this:

directories() returns a list of l10n module and a tuple of path segments, where the latter is used to construct the reference code in en-US.

For regular local builds, that should return (basepath, 'mobile') and for src-tree work it'd just be (basepath). src-tree here means that you have your sources on disk as on hg, and not mobile-browser as mobile inside central or 1.9.2.

I tested this with both compare-locales, and langpacker (which uses plain src), with and without merge.

I'd take an r+ to include bumping the RELEASE_AUTOMATION tag to a revision with this patch, i.e., including the changesets http://hg.mozilla.org/build/compare-locales/graph/22?revcount=4. None of them are functional, though, just for reference that we don't make the "let's just bump" mistake again.
Attachment #440586 - Flags: review?(ccooper)

Comment 3

8 years ago
Comment on attachment 440586 [details] [diff] [review]
move the generation of the base path tuple into method and overload

(In reply to comment #2) 
> I'd take an r+ to include bumping the RELEASE_AUTOMATION tag to a revision with
> this patch, i.e., including the changesets
> http://hg.mozilla.org/build/compare-locales/graph/22?revcount=4. None of them
> are functional, though, just for reference that we don't make the "let's just
> bump" mistake again.

Patch looks fine. Who's going to bump the tag?
Attachment #440586 - Flags: review?(ccooper) → review+
(Assignee)

Comment 4

8 years ago
Fixed, http://hg.mozilla.org/build/compare-locales/rev/bfb8fb7970df.

Sorry for the trouble.

RELEASE_AUTOMATION tag moved, too, http://hg.mozilla.org/build/compare-locales/rev/fdb241a03df5, and released 0.8.2 to pypi.
(Assignee)

Updated

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.