411112 - AIX Compilation error for trunk build source nsPNGEncoder.cpp : mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp", line 159.14: 1540-0217 (S) "jmpbuf" is not a member of "struct png_struct_def".

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Shailen]

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/modules/libpr0n/encoders/png/nsPNGEncoder.cpp etting below compilation
error -
"/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp", line 159.14: 1540-0217 (S) "jmpbuf" is not a member of "struct png_struct_def".
"/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp", line 232.14: 1540-0217 (S) "jmpbuf" is not a member of "struct png_struct_def".
"/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp", line 311.14: 1540-0217 (S) "jmpbuf" is not a member of "struct png_struct_def".


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 error.

Expected Results:  
No build failure

Comment 1

[Shailen]

Attachment: Patch V1

Looks like reverting back the fix for the bug 308126 resolves the issue.

Comment 2

[Stuart Parmenter]

do we need this or not?  *confused*

Comment 3

[Shailen]

I suppose we do not need this and removing the fix for Bug 308126 would resove the above compilation error.

Comment 4

[Stuart Parmenter]

bug 308126 added this code to fix a compilation error for AIX.  Are we going to break whoever if we check this in?

Comment 5

[Shailen]

I believe we are not breaking the code as I have tested many times. 

Comment 6

[timeless]

so, how is your version of aix different from that other version of aix?

Comment 7

[Shailen]

The AIX version that is used for testing is 5.2 

Comment 8

[Stuart Parmenter]

Comment on attachment 295750 [details] [diff] [review]
Patch V1

i guess since this is the latest bug filed we'll back out the other bug.

Comment 9

[Reed Loden [:reed]]

Checking in modules/libpr0n/encoders/png/nsPNGEncoder.cpp;
/cvsroot/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp,v  <--  nsPNGEncoder.cpp
new revision: 1.8; previous revision: 1.7
done