Closed
Bug 486111
Opened 16 years ago
Closed 16 years ago
compare-locales support for comparing source repo forests
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Pike, Assigned: Pike)
Details
(Whiteboard: [l10n])
Attachments
(2 files)
|
4.31 KB,
patch
|
coop
:
review+
|
Details | Diff | Splinter Review |
|
2.14 KB,
patch
|
coop
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
I'm adding support for compare-locales to run tests just on the source repos.
That is, instead of doing a mobile checkout inside a mozilla-central for fennec and a releases/mozilla-1.9.1 checkout inside comm-central for thunderbird, I'm just using the hg repos as they live on the hg server, and compare them as such.
That works without touching any of the real comparison code, I just need to specialize the app enumeration, and to feed that baby, the l10n.ini parser.
The patch is running successfully locally on what I dub l10n master v2.
I'm using a up-front patch to the compare-locales repo as this is a slightly bigger change that might benefit from some review comments.
Attachment #370190 -
Flags: review?(ccooper)
| Assignee | ||
Updated•16 years ago
|
Whiteboard: [l10n]
| Assignee | ||
Comment 1•16 years ago
|
||
I'd like to piggy-back this patch to buildbotcustom with this bug. It's the same downstream feature, and it's really just a small nit that I need for it.
Attachment #370873 -
Flags: review?(ccooper)
Updated•16 years ago
|
Attachment #370873 -
Flags: review?(ccooper) → review+
| Assignee | ||
Comment 2•16 years ago
|
||
Comment on attachment 370873 [details] [diff] [review]
piggy-back patch for buildbotcustom
This one needs landing on buildbotcustom.
Attachment #370873 -
Flags: checked‑in? checked‑in?
Comment 3•16 years ago
|
||
(In reply to comment #2)
> (From update of attachment 370873 [details] [diff] [review])
> This one needs landing on buildbotcustom.
This is checked in:
changeset: 247:7c763b51f7ac
(sorry, can't set the flag for some reason)
Updated•16 years ago
|
Attachment #370873 -
Flags: checked‑in? checked‑in? → checked‑in+ checked‑in+
Updated•16 years ago
|
Attachment #370190 -
Flags: review?(ccooper) → review+
| Assignee | ||
Comment 4•16 years ago
|
||
http://hg.mozilla.org/build/compare-locales/rev/737781045a35, landed attachment 370190 [details] [diff] [review] on the compare-locales repo, too.
FIXED.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•