Closed
Bug 494376
Opened 17 years ago
Closed 17 years ago
Update l10n_revision in l10n repacks to match l10n revision
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Pike, Assigned: Pike)
Details
(Whiteboard: [l10n])
Attachments
(2 files)
|
1.92 KB,
patch
|
coop
:
review+
armenzg
:
review+
coop
:
checked-in+
|
Details | Diff | Splinter Review |
|
1.02 KB,
patch
|
coop
:
review+
mozilla
:
review+
coop
:
checked-in+
|
Details | Diff | Splinter Review |
After updating the l10n hg repo, we should have a SetProperty step to update l10n_revision to actually specify the hg ident -i.
| Assignee | ||
Updated•17 years ago
|
Whiteboard: [l10n]
| Assignee | ||
Comment 1•17 years ago
|
||
This should be added to updateSources. Sadly, this will require changing both the nightly and the release repack factory. I don't expect this to have regression potential for comm-central.
| Assignee | ||
Comment 2•17 years ago
|
||
Taking. Adding hg ident -i to both impls of updateSources, right after the hg update for the l10n repo.
Assignee: nobody → l10n
Status: NEW → ASSIGNED
Attachment #380828 -
Flags: review?(ccooper)
Attachment #380828 -
Flags: review?(armenzg)
Updated•17 years ago
|
Attachment #380828 -
Flags: review?(armenzg) → review+
Updated•17 years ago
|
Attachment #380828 -
Flags: review?(ccooper) → review+
| Assignee | ||
Comment 3•17 years ago
|
||
Can you do the landing/reconfig dance for me?
Keywords: checkin-needed
Comment 4•17 years ago
|
||
Comment on attachment 380828 [details] [diff] [review]
hg ident in updateSource for l10n_revision
changeset: 324:18da9917b4a1
Attachment #380828 -
Flags: checked‑in+
Updated•17 years ago
|
Keywords: checkin-needed
| Assignee | ||
Comment 5•17 years ago
|
||
The same dance for fennec, too.
Attachment #383457 -
Flags: review?(ccooper)
Attachment #383457 -
Flags: review?(aki)
Updated•17 years ago
|
Attachment #383457 -
Flags: review?(ccooper) → review+
Updated•17 years ago
|
Attachment #383457 -
Flags: review?(aki) → review+
| Assignee | ||
Comment 6•17 years ago
|
||
Coop, can you land this?
| Assignee | ||
Comment 7•17 years ago
|
||
aki, coop, can one of you get this into production?
Comment 8•17 years ago
|
||
Comment on attachment 383457 [details] [diff] [review]
patch MobileNightlyRepackFactory, too
http://hg.mozilla.org/build/buildbotcustom/rev/c225ff037969
Updated•17 years ago
|
Attachment #383457 -
Flags: checked-in+
Comment 9•17 years ago
|
||
production-master reconfig-ed.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•