Closed Bug 1307995 Opened 9 years ago Closed 9 years ago

Drop Belarusian (be) from Firefox for Android in both maemo-locales and all-locales. Add Nepali (ne-NP), Bulgarian (bg) and Kabyle (kab) to all-locales

Categories

(Firefox Build System :: Android Studio and Gradle Integration, defect)

defect
Not set
normal

Tracking

(relnote-firefox 51+, firefox51 fixed)

RESOLVED FIXED
Tracking Status
relnote-firefox --- 51+
firefox51 --- fixed

People

(Reporter: delphine, Assigned: delphine)

References

Details

Attachments

(1 file)

As commented by :flod in Bug 1304743, Belarusian hasn't see any activity in almost 2 years. We will want to drop it from both maemo-locales and all-locales.
Assignee: mozilla_by → lebedel.delphine
For the sake of simplicity, changing the title of this bug to work on multiple locales here. We will also want to add Kabyle (kab), Bulgarian (bg) and Nepali (ne-NP) to single-locale builds.
Summary: Drop Belarusian (be) from Firefox for Android in both maemo-locales and all-locales → Drop Belarusian (be) from Firefox for Android in both maemo-locales and all-locales. Add Nepali (ne-NP), Bulgarian (bg) and Kabyle (kab) to single-locales
Summary: Drop Belarusian (be) from Firefox for Android in both maemo-locales and all-locales. Add Nepali (ne-NP), Bulgarian (bg) and Kabyle (kab) to single-locales → Drop Belarusian (be) from Firefox for Android in both maemo-locales and all-locales. Add Nepali (ne-NP), Bulgarian (bg) and Kabyle (kab) to all-locales
bg and kab are new Fennec locales. Please add to dashboards
Attached patch bug1307995.patchSplinter Review
My comment above about dashboards is wrong, I forgot one locale. Please add bg, ne-NP and kab to dashboards as they are new fennec locales. Please drop be from dashboards. thanks (and sorry for all this noise...)
Attachment #8800461 - Flags: review?(francesco.lodolo)
Comment on attachment 8800461 [details] [diff] [review] bug1307995.patch Review of attachment 8800461 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. I'm pretty sure dashboards will get the locales automatically as soon as this lands in mozilla-aurora.
Attachment #8800461 - Flags: review?(francesco.lodolo) → review+
Correction: On the other hand, we'll need to NI Pike to remove Belarusian from the dashboard when this lands. I'll leave setting flags to you for the attachments ;-)
Moving over to Fx Android Build Config, as that is what this bug is about now.
Component: be / Belarusian → Build Config & IDE Support
Product: Mozilla Localizations → Firefox for Android
Comment on attachment 8800461 [details] [diff] [review] bug1307995.patch This is a change that needs to land only in Mozilla Aurora. Also for relnotes: this adds 3 locales (ne-NP, bg and kab) and drops one (be)
Attachment #8800461 - Flags: approval-mozilla-aurora?
I'm sorry, for the added locales we don't need relnotes since they are added to single-locale builds! However since be locale is dropped from maemo-locales, this should probably be renoted. Thanks!
Comment on attachment 8800461 [details] [diff] [review] bug1307995.patch Locale changes. Take it in 51 aurora.
Attachment #8800461 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Depends on: 1311693
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
No longer depends on: 1311693
Product: Firefox for Android → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: