Closed Bug 1421122 Opened 7 years ago Closed 7 years ago

Poll ONLY locales that SeaMonkey supports (as in listed in all-locales)

Categories

(SeaMonkey :: Release Engineering, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ewong, Assigned: ewong)

References

Details

Attachments

(2 files, 6 obsolete files)

Currently (and this has been going on for some time) we have been polling
the whole list of locales from l10n-central's repo.  

We need to poll *only* the locales that we support.. i.e. from
all-locales.
Attached patch [custom] proposed patch (obsolete) — Splinter Review
Instead of polling *all* locales (which we've been doing in the past),
poll only those listed in all-locales.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #8932311 - Flags: review?(frgrahl)
Summary: Poll ONLY locales that we support → Poll ONLY locales that SeaMonkey supports (as in listed in all-locales)
Attached patch [custom] proposed patch (obsolete) — Splinter Review
Attachment #8932311 - Attachment is obsolete: true
Attachment #8932311 - Flags: review?(frgrahl)
Attachment #8932312 - Flags: review?(frgrahl)
Attached patch [custom] proposed patch (v3) (obsolete) — Splinter Review
use comm-central's all-locales list.
Attachment #8932312 - Attachment is obsolete: true
Attachment #8932312 - Flags: review?(frgrahl)
Attachment #8932323 - Flags: review?(frgrahl)
Attachment #8932325 - Flags: review?(frgrahl)
Attachment #8932323 - Attachment is obsolete: true
Attachment #8932323 - Flags: review?(frgrahl)
Attachment #8932327 - Flags: review?(frgrahl)
Attached patch [custom] proposed patch (v4) (obsolete) — Splinter Review
hgUrl needs to have the / truncated from the end if it exists.
Attachment #8932327 - Attachment is obsolete: true
Attachment #8932327 - Flags: review?(frgrahl)
Attachment #8932349 - Flags: review?(frgrahl)
Attachment #8932349 - Attachment is obsolete: true
Attachment #8932349 - Flags: review?(frgrahl)
Attachment #8932357 - Flags: review?(frgrahl)
Attachment #8932325 - Attachment is obsolete: true
Attachment #8932325 - Flags: review?(frgrahl)
Attachment #8932358 - Flags: review?(frgrahl)
Attachment #8932358 - Flags: review?(frgrahl) → review+
Comment on attachment 8932357 [details] [diff] [review]
[custom] use the branch's own suite/locales/all-locales instead of c-c's [checked-in]

Looks good.
Attachment #8932357 - Flags: review?(frgrahl) → review+
Attachment #8932357 - Attachment description: [custom] use the branch's own suite/locales/all-locales instead of c-c's → [custom] use the branch's own suite/locales/all-locales instead of c-c's [checked-in]
Attachment #8932358 - Attachment description: [configs] just use all_locales_file → [configs] just use all_locales_file [checked-in]
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: