Closed Bug 1402234 Opened 7 years ago Closed 7 years ago

Remove ENABLE_INTL_API=no code from intl/locale

Categories

(Core :: Internationalization, enhancement)

Product:
Core
Component:
Internationalization
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla58
Tracking Flags:
Tracking Status
firefox58 --- fixed

People

(Reporter: zbraniecki, Assigned: zbraniecki)

References

Details

Attachments

(1 file)

Bug 1402234 - Remove the ENABLE_INTL_API=no code from intl/locale.
(deleted), text/x-review-board-request
jfkthame
: review+
Details
Assignee: nobody → gandalf
Blocks: 1308359
Status: NEW → ASSIGNED
Blocks: 1387332
No longer blocks: 1308359
Rather than the mega-patch here, let's remove various parts of this in separate bugs for the individual functional units; then let's see what (if anything) remains. So holding off on this one for now....
Attachment #8911071 - Flags: review?(jfkthame)
There are patches in separate bugs (see dependencies of bug 1387332) to handle most of this except for the intl/locale subdirectory. I would suggest morphing this bug to focus just on intl/locale/ and updating the patch accordingly; that'll be a nice clean unit of work to review&land.
Summary: Remove ENABLE_INTL_API=no code from intl/ → Remove ENABLE_INTL_API=no code from intl/locale
Sounds good! Updated the patch to cover only intl/locale directory.
Comment on attachment 8911071 [details] Bug 1402234 - Remove the ENABLE_INTL_API=no code from intl/locale. https://reviewboard.mozilla.org/r/182538/#review187886 LGTM, thanks!
Attachment #8911071 - Flags: review?(jfkthame) → review+
Well, assuming tryserver is now happy with it, that is. :)
Pushed by zbraniecki@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7b47963e73f8 Remove the ENABLE_INTL_API=no code from intl/locale. r=jfkthame
https://hg.mozilla.org/mozilla-central/rev/7b47963e73f8
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: