Closed Bug 415867 Opened 13 years ago Closed 13 years ago
AIX Compilation error for trunk build source cairo-gstate
.c : line 45 .43: 1506-294 (S) Syntax error in expression on #if directive .
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:220.127.116.11) Gecko/20071127 Firefox/18.104.22.168 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:22.214.171.124) Gecko/20071127 Firefox/126.96.36.199 While compiling trunk build source mozilla/gfx/cairo/cairo/src/cairo-gstate.c on AIX, getting below compilation error - "mozilla/gfx/cairo/cairo/src/cairo-gstate.c", line 45.43: 1506-294 (S) Syntax error in expression on #if directive. gmake: *** [cairo-gstate.o] Error 1 "mozilla/gfx/cairo/cairo/src/cairo-matrix.c", line 41.43: 1506-294 (S) Syntax error in expression on #if directive. gmake: *** [cairo-matrix.o] Error 1 Reproducible: Always Steps to Reproduce: 1. Build Mozilla Firefox Trunk build on AIX with Gnome RPMS of 64 bit version. 2. 3. Actual Results: Build fails with the above errors. Expected Results: No build failure
Severity: normal → major
OS: Other → AIX
Version: unspecified → Trunk
Modified the code to replace _ISOC99_SOURCE with defined(_ISOC99_SOURCE) in #if statement
Comment on attachment 301629 [details] [diff] [review] Patch V1 Please send this upstream to cairo.
Status: UNCONFIRMED → NEW
Component: Build Config → GFX: Thebes
Ever confirmed: true
Product: Firefox → Core
QA Contact: build.config → thebes
Checking in gfx/cairo/cairo/src/cairo-gstate.c; /cvsroot/mozilla/gfx/cairo/cairo/src/cairo-gstate.c,v <-- cairo-gstate.c new revision: 1.29; previous revision: 1.28 done Checking in gfx/cairo/cairo/src/cairo-matrix.c; /cvsroot/mozilla/gfx/cairo/cairo/src/cairo-matrix.c,v <-- cairo-matrix.c new revision: 1.25; previous revision: 1.24 done
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta4
(Unless this patch makes its way into gfx/cairo as a patch file, along with an entry in the README file, this'll get blown away the next time I do a cairo update.)
sjain said on IRC that he was going to send this upstream to Cairo.
Sent an email to http://cairographics.org/contact/ to include the patch for cairo source.
You need to log in before you can comment on or make changes to this bug.