Closed
Bug 1307358
Opened 8 years ago
Closed 8 years ago
beta doesnt build using --with-system-icu
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox50 fixed, firefox51 fixed, firefox52 fixed)
VERIFIED
FIXED
mozilla52
People
(Reporter: gaston, Assigned: xidorn)
References
Details
Attachments
(1 file, 1 obsolete file)
58 bytes,
text/x-review-board-request
|
glandium
:
review+
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details |
On OpenBSD, using --with-system-icu, beta (1 and 3) fail to build on: In file included from /home/obj/ports/firefox-50.0beta3/build-amd64/netwerk/dns/Unified_cpp_netwerk_dns0.cpp:65: In file included from /home/obj/ports/firefox-50.0beta3/firefox-50.0b3/netwerk/dns/nsIDNService.cpp:6: /home/obj/ports/firefox-50.0beta3/firefox-50.0b3/netwerk/dns/nsIDNService.h:16:10: fatal error: 'unicode/uidna.h' file not found #include "unicode/uidna.h" unicode/uidna.h is in /usr/local/include but -I/usr/local/include is not present in the build flags for Unified_cpp_netwerk_dns0.cpp - so probably a missing MOZ_ICU_CFLAGS somewhere. Here's what i have in config.status: "MOZ_ICU_CFLAGS": [ "-I/usr/local/include" ], "MOZ_ICU_LIBS": [ "-L/usr/local/lib", "-licui18n", "-licuuc", "-licudata" ],
Reporter | ||
Comment 1•8 years ago
|
||
Another possibility is that uidna.h isnt created under system_wrappers/unicode which should also wrap it ?
Flags: needinfo?(mh+mozilla)
Comment 2•8 years ago
|
||
This might be regression by bug 1284406.
Reporter | ||
Comment 3•8 years ago
|
||
Trying with unicode/uidna.h added to config/system-headers...
Reporter | ||
Comment 4•8 years ago
|
||
(In reply to Makoto Kato [:m_kato] from comment #2) > This might be regression by bug 1284406. Oh, probably then - thanks, hadnt found this bug.
Updated•8 years ago
|
Flags: needinfo?(mh+mozilla)
Reporter | ||
Comment 5•8 years ago
|
||
Fails also with unicode/uidna.h added to config/system-headers: /home/obj/ports/firefox-50.0beta3/build-amd64/dist/system_wrappers/unicode/uidna.h:3:15: fatal error: 'unicode/uidna.h' file not found #include_next <unicode/uidna.h>
Blocks: 1284406
Assignee | ||
Comment 6•8 years ago
|
||
Could you test if this patch works for you?
Flags: needinfo?(landry)
Attachment #8797448 -
Flags: review?(mh+mozilla)
Reporter | ||
Comment 7•8 years ago
|
||
Yep, works for me - once commited, would need backporting to aurora and beta, ideally.....
Flags: needinfo?(landry)
Comment 8•8 years ago
|
||
Comment on attachment 8797448 [details] [diff] [review] patch Review of attachment 8797448 [details] [diff] [review]: ----------------------------------------------------------------- ::: old-configure.in @@ +5783,5 @@ > +if test "$MOZ_WIDGET_TOOLKIT" = "cocoa"; then > + USE_ICU=1 > +fi > + > +MOZ_CONFIG_ICU() It would probably be better moved above https://dxr.mozilla.org/mozilla-central/rev/42c95d88aaaa7c2eca1d278399421d437441ac4d/old-configure.in#5736 Likewise in js/src/old-configure.in.
Attachment #8797448 -
Flags: review?(mh+mozilla) → feedback+
Comment hidden (mozreview-request) |
Comment 10•8 years ago
|
||
mozreview-review |
Comment on attachment 8797919 [details] Bug 1307358 - Config ICU before assigning OS_*FLAGS. https://reviewboard.mozilla.org/r/83516/#review82128
Attachment #8797919 -
Flags: review?(mh+mozilla) → review+
Assignee | ||
Updated•8 years ago
|
Attachment #8797448 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → xidorn+moz
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 11•8 years ago
|
||
Comment on attachment 8797919 [details] Bug 1307358 - Config ICU before assigning OS_*FLAGS. Approval Request Comment [Feature/regressing bug #]: bug 1284406 [User impact if declined]: Package maintainers for distributions may not be able to build Firefox [Describe test coverage new/current, TreeHerder]: Build system only change, should be fine as far as it doesn't bust on TreeHerder [Risks and why]: low risk as far as it builds [String/UUID change made/needed]: n/a
Attachment #8797919 -
Flags: approval-mozilla-beta?
Attachment #8797919 -
Flags: approval-mozilla-aurora?
Updated•8 years ago
|
Comment 12•8 years ago
|
||
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/890556340d6e Config ICU before assigning OS_*FLAGS. r=glandium
Keywords: checkin-needed
Comment on attachment 8797919 [details] Bug 1307358 - Config ICU before assigning OS_*FLAGS. Fix to support distro builds, Aurora51+, Beta50+
Attachment #8797919 -
Flags: approval-mozilla-beta?
Attachment #8797919 -
Flags: approval-mozilla-beta+
Attachment #8797919 -
Flags: approval-mozilla-aurora?
Attachment #8797919 -
Flags: approval-mozilla-aurora+
Comment 14•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/e899c491933d
Comment 15•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/02929136e12c
Comment 16•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/890556340d6e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Reporter | ||
Comment 17•8 years ago
|
||
50beta5 built fine using --with-system-icu.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•