Update l10n_revision in l10n repacks to match l10n revision

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
9 years ago
5 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [l10n])

Attachments

(2 attachments)

(Assignee)

Description

9 years ago
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

9 years ago
Whiteboard: [l10n]
(Assignee)

Comment 1

9 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

9 years ago
Created attachment 380828 [details] [diff] [review]
hg ident in updateSource for l10n_revision

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

9 years ago
Attachment #380828 - Flags: review?(armenzg) → review+

Updated

9 years ago
Attachment #380828 - Flags: review?(ccooper) → review+
(Assignee)

Comment 3

9 years ago
Can you do the landing/reconfig dance for me?
Keywords: checkin-needed

Comment 4

9 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

9 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

9 years ago
Created attachment 383457 [details] [diff] [review]
patch MobileNightlyRepackFactory, too

The same dance for fennec, too.
Attachment #383457 - Flags: review?(ccooper)
Attachment #383457 - Flags: review?(aki)

Updated

9 years ago
Attachment #383457 - Flags: review?(ccooper) → review+

Updated

9 years ago
Attachment #383457 - Flags: review?(aki) → review+
(Assignee)

Comment 6

9 years ago
Coop, can you land this?
(Assignee)

Comment 7

9 years ago
aki, coop, can one of you get this into production?

Updated

9 years ago
Attachment #383457 - Flags: checked-in+

Comment 9

9 years ago
production-master reconfig-ed.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.