AIX Compilation error for trunk build source pixman-compose.c : mozilla/gfx/cairo/libpixman/src/pixman-private.h", line 32.9: 1506-213 (S) Macro name DEBUG cannot be redefined.

RESOLVED FIXED

Status

()

Core
Graphics
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Shailen, Assigned: Shailen)

Tracking

Trunk
Other
AIX
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

877 bytes, patch
Stuart Parmenter
: review+
Details | Diff | Splinter Review
(Assignee)

Description

11 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.11) Gecko/20071127 Firefox/2.0.0.11
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.11) Gecko/20071127 Firefox/2.0.0.11

On AIX, while compiling trunk build source mozilla/gfx/cairo/libpixman/src/pixman-compose.c, getting below compilation error -
mozilla/gfx/cairo/libpixman/src/pixman-private.h", line 32.9: 1506-213 (S) Macro name DEBUG cannot be redefined.

Reproducible: Always

Steps to Reproduce:
1.Build Mozilla Firefox Trunk build on AIX with Gnome 2.12 RPMS of 64 bit
version
2.
3.
Actual Results:  
Build fails with the above errors.


Expected Results:  
No build failure
(Assignee)

Updated

11 years ago
OS: Other → AIX
(Assignee)

Comment 1

11 years ago
Created attachment 295208 [details] [diff] [review]
Patch V1

Using #ifndef to ensure DEBUG is defined only once.
Attachment #295208 - Flags: review?(pavlov)
Component: Build Config → GFX: Thebes
Product: Firefox → Core
QA Contact: build.config → thebes
Version: unspecified → Trunk
Assignee: nobody → shailen.n.jain

Comment 2

11 years ago
Comment on attachment 295208 [details] [diff] [review]
Patch V1

this should be pushed upstream.
Attachment #295208 - Flags: review?(vladimir)
Attachment #295208 - Flags: review?(pavlov)
Attachment #295208 - Flags: review+
(Assignee)

Comment 3

11 years ago
Sent an email to http://cairographics.org/contact/ to include the patch for
cairo source.
upstream pixman already has:

#undef DEBUG
#define DEBUG 1

in there, as does our in-tree version.  So I'm not sure what's needed here...
(er, #define DEBUG 0, I meant)

Going to resolve this as fixed.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.