Closed
Bug 863672
Opened 11 years ago
Closed 11 years ago
Port configure.in changes to comm-aurora
Categories
(MailNews Core :: Build Config, defect)
Tracking
(thunderbird22 fixed)
RESOLVED
FIXED
Thunderbird 22.0
Tracking | Status | |
---|---|---|
thunderbird22 | --- | fixed |
People
(Reporter: mcsmurf, Assigned: mcsmurf)
References
Details
Attachments
(1 file, 1 obsolete file)
2.92 KB,
patch
|
jcranmer
:
review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
We need to port a few changes from mozilla-aurora configure to configure.in one last time (basically, Bug 846540 fixed most cases where configure.in changes had to be ported). Basically SeaMonkey needs the MOZ_GAMEPAD and the MOZ_WEBSPEECH changes from Bug Bug 604039 and 650295 to fix quite a few tests (maybe this bug should go to the SeaMonkey product). See http://hg.mozilla.org/mozilla-central/annotate/f8d27fe5d7c0//configure.in#l5921 for MOZ_GAMEPAD and http://hg.mozilla.org/mozilla-central/annotate/f8d27fe5d7c0//configure.in#l5354 for MOZ_WEBSPEECH.
Assignee | ||
Comment 1•11 years ago
|
||
This seems to fix the issue (it's still building, I'm not really a configure.in expert :)
Assignee | ||
Comment 2•11 years ago
|
||
Forgot to hg qrefresh
Attachment #739548 -
Attachment is obsolete: true
Assignee | ||
Comment 3•11 years ago
|
||
Thunderbird comm-aurora Try build worked fine (build only, no tests)
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 739549 [details] [diff] [review] Patch This patch copies over the required changes from mozilla/configure.in. I only changed this line AC_CHECK_HEADERS([linux/joystick.h]) from MOZ_CHECK_HEADER([linux/joystick.h]) (comm-* does not know the MOZ_CHECK_HEADERS macro) Note: This is a comm-aurora patch, comm-central does not need this patch (anymore)
Attachment #739549 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 5•11 years ago
|
||
I meant: "(comm-* does not know the MOZ_CHECK_HEADER macro)" (without the S)
Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 739549 [details] [diff] [review] Patch This patch is needed for comm-aurora only, comm-central does not need this (anymore).
Attachment #739549 -
Flags: review?(bugspam.Callek) → review?(Pidgeot18)
Comment 7•11 years ago
|
||
Comment on attachment 739549 [details] [diff] [review] Patch Review of attachment 739549 [details] [diff] [review]: ----------------------------------------------------------------- ::: configure.in @@ +4404,5 @@ > + MOZ_WEBSPEECH=, > + MOZ_WEBSPEECH=1) > + > +if test -n "$MOZ_WEBSPEECH"; then > + AC_DEFINE(MOZ_WEBSPEECH) It looks like in mozilla-aurora, MOZ_WEBSPEECH also implies MOZ_MEDIA.
Attachment #739549 -
Flags: review?(Pidgeot18) → review+
Assignee | ||
Comment 8•11 years ago
|
||
Comment on attachment 739549 [details] [diff] [review] Patch [Approval Request Comment] Regression caused by (bug #): - User impact if declined: SeaMonkey browser tests fail, maybe also has unknown user impact (not known if there are bugs that are caused by this) Testing completed (on c-c, etc.): Patch cannot be tested on comm-central as it applies to comm-aurora only Risk to taking this patch (and alternatives if risky): Few risk I would say, the variables enabled/disabled by this patch get only used in the comm-central build system. Whether or not the core code in mozilla/ (which those configure vars apply to) gets built is "decided" by mozilla/configure.in and not by the comm-central configure.in. The configure changes here are only needed to fix the installer package manifest (because the manifest uses ifdefs).
Attachment #739549 -
Flags: approval-comm-aurora?
Updated•11 years ago
|
Attachment #739549 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee | ||
Comment 9•11 years ago
|
||
Pushed: https://hg.mozilla.org/releases/comm-aurora/rev/5a3559594fce
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 22.0
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → bugzilla
Updated•11 years ago
|
status-thunderbird22:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•