The default bug view has changed. See this FAQ.

Status

()

Core
Graphics
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: espindola, Assigned: espindola)

Tracking

unspecified
mozilla13
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 593453 [details] [diff] [review]
fix build

See also bug 723114.

This patch is just a quick fix. Clang warns on GetImageFromSourceSurface via -Wreturn-type and we build with -Werror=return-type.
Attachment #593453 - Flags: review?(joe)
Attachment #593453 - Flags: review?(joe) → review?(jmuizelaar)
Attachment #593453 - Flags: review?(jmuizelaar) → review+
Duplicate of this bug: 723187
Duplicate of this bug: 723284
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=5a80e90839fd
merge! merge!
Status: NEW → ASSIGNED
And the next one :(
In file included from ../../../dist/include/nsCoreAnimationSupport.h:49:
../../../dist/include/nsIOSurface.h:48:7: error: redefinition of 'NSOpenGLContext' as different kind of symbol
That's bug 723618.
(Please can you set assignee when attaching patches, or if using bzexport, file a bug against it :-))

https://hg.mozilla.org/mozilla-central/rev/5a80e90839fd
Assignee: nobody → respindola
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13

Comment 8

5 years ago
Is it possible to get this into mozilla-beta? Today I've tried several times to build Thunderbird 12 with Clang, but it failed constantly with the error from Bug 723187. It takes me a hour or so to find out what the problem is and to find this Bug. Thanks.
(In reply to Nomis101 from comment #8)
> Is it possible to get this into mozilla-beta? Today I've tried several times
> to build Thunderbird 12 with Clang, but it failed constantly with the error
> from Bug 723187. It takes me a hour or so to find out what the problem is
> and to find this Bug. Thanks.

You could just export "-Wreturn-type" to turn the error into a warning and let the build finish.

Comment 10

5 years ago
I've applied this patch to fix the error, maybe your workaround would also help. But I think more about all the other users who are trying to build Firefox/Thunderbird 12 with Clang and run into the same error.
You need to log in before you can comment on or make changes to this bug.