Build failure on Beta in non-unified mode

RESOLVED FIXED in Firefox 37

Status

RESOLVED FIXED
4 years ago
3 months ago

People

(Reporter: jya, Assigned: jya)

Tracking

37 Branch
mozilla39
x86
macOS
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox37 fixed, firefox38 fixed, firefox39 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
widget/cocoa/TextInputHandler.mm doesn't compile on beta with non-unified build
(Assignee)

Comment 1

4 years ago
fix non-unified build.
Attachment #8571667 - Flags: review?(ryanvm)
(Assignee)

Updated

4 years ago
Assignee: nobody → jyavenard
Status: NEW → ASSIGNED
Attachment #8571667 - Flags: review?(ryanvm) → review?(smichaud)
Comment on attachment 8571667 [details] [diff] [review]
Fix non-unified compilation in beta

Nice catch.
Attachment #8571667 - Flags: review?(smichaud) → review+
(Assignee)

Comment 4

4 years ago
Comment on attachment 8571667 [details] [diff] [review]
Fix non-unified compilation in beta

Approval Request Comment
[Feature/regressing bug #]: NA
[User impact if declined]: Can't compile beta on mac with unified build disabled.
[Describe test coverage new/current, TreeHerder]: None.
[Risks and why]: None, only adding missing header.
[String/UUID change made/needed]: None
Attachment #8571667 - Flags: approval-mozilla-beta?
(Assignee)

Comment 5

4 years ago
As per IRC discussion...
Flags: needinfo?(ryanvm)
Comment on attachment 8571667 [details] [diff] [review]
Fix non-unified compilation in beta

I'm just going to land this a=bustage
Flags: needinfo?(ryanvm)
Attachment #8571667 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/releases/mozilla-aurora/rev/c2ffff195e60
status-firefox38: --- → fixed
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/839e567bda26
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
Component: Build Config → General
Product: Firefox → Firefox Build System
Target Milestone: Firefox 39 → mozilla39
You need to log in before you can comment on or make changes to this bug.