Update l10n_revision in l10n repacks to match l10n revision

RESOLVED FIXED

Status

RESOLVED FIXED
10 years ago
6 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [l10n])

Attachments

(2 attachments)

(Assignee)

Description

10 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

10 years ago
Whiteboard: [l10n]
(Assignee)

Comment 1

10 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

10 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)
Attachment #380828 - Flags: review?(armenzg) → review+
Attachment #380828 - Flags: review?(ccooper) → review+
(Assignee)

Comment 3

10 years ago
Can you do the landing/reconfig dance for me?
Keywords: checkin-needed
Comment on attachment 380828 [details] [diff] [review]
hg ident in updateSource for l10n_revision

changeset:   324:18da9917b4a1
Attachment #380828 - Flags: checked‑in+
Keywords: checkin-needed
(Assignee)

Comment 5

10 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)
Attachment #383457 - Flags: review?(ccooper) → review+

Updated

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

Comment 6

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

Comment 7

10 years ago
aki, coop, can one of you get this into production?
production-master reconfig-ed.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 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.