makefile for bundled libpng has a decoder file with the encoder files

RESOLVED FIXED

Status

()

RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: zeev.tarantov, Assigned: glennrp+bmo)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

(blocking2.0 -)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2a1pre) Gecko/20090306 Minefield/3.2a1pre
Build Identifier: 

... which prevents the png decoder from building without the png encoder.

Reproducible: Always

Steps to Reproduce:
1. try to build with --enable-image-encoders not containing png
2. observe build failure
3. profit!
Actual Results:  
../decoders/png/libimgpng_s.a(nsPNGDecoder.o): In function `PNGGetColorProfile(png_struct_def*, png_info_struct
*, int, unsigned int*, unsigned int*)':
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:444: undefined reference to `MOZ_PNG_set_gray_to_rgb'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:425: undefined reference to `MOZ_PNG_set_gray_to_rgb'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:481: undefined reference to `MOZ_PNG_set_gray_to_rgb'
../decoders/png/libimgpng_s.a(nsPNGDecoder.o): In function `info_callback(png_struct_def*, png_info_struct*)':
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:532: undefined reference to `MOZ_PNG_set_expand'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:589: undefined reference to `MOZ_PNG_set_gray_to_rgb'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:392: undefined reference to `MOZ_PNG_set_gamma'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:553: undefined reference to `MOZ_PNG_set_expand'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:529: undefined reference to `MOZ_PNG_set_expand'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:557: undefined reference to `MOZ_PNG_set_strip_16'
/home/wolf/mozilla/src/modules/libpr0n/decoders/png/nsPNGDecoder.cpp:395: undefined reference to `MOZ_PNG_set_gamma'


Expected Results:  
build success.

diff -r b0df372b9192 modules/libimg/png/Makefile.in
--- a/modules/libimg/png/Makefile.in	Sat Mar 07 13:40:08 2009 -0800
+++ b/modules/libimg/png/Makefile.in	Sun Mar 08 14:12:27 2009 +0200
@@ -74,6 +74,7 @@
 		pngpread.c \
 		pngread.c \
 		pngrio.c \
+		pngrtran.c \
 		pngrutil.c \
 		$(NULL)
 
@@ -86,7 +87,6 @@
 		pngwio.c \
 		pngwrite.c \
 		pngwtran.c \
-		pngrtran.c \
 		pngwutil.c \
 		$(NULL)
(Reporter)

Comment 1

10 years ago
Created attachment 367476 [details] [diff] [review]
move source file from encoder to decoder
Attachment #367476 - Flags: review?(pavlov)
(Reporter)

Updated

10 years ago
Attachment #367476 - Attachment description: what the message says → move source file from encoder to decoder
(Assignee)

Updated

9 years ago
Assignee: nobody → glennrp+bmo
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
(Assignee)

Updated

9 years ago
Attachment #367476 - Flags: review?(joe)
Attachment #367476 - Flags: review?(pavlov)
Attachment #367476 - Flags: review?(joe)
Attachment #367476 - Flags: review+
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
(Assignee)

Comment 2

9 years ago
This bug was apparently introduced by check-in of a patch from bug #386585 in November 2007.  I've verified that it prevents building a read-only PNG codec as Zeev reported and that Zeev's patch fixes the problem.  As far as I can tell, the bug exists in all 1.9.x branches.
blocking2.0: --- → ?
(In reply to comment #1)
> Created an attachment (id=367476) [details]
> what the message says

Pushed as: http://hg.mozilla.org/mozilla-central/rev/ab685c3e46eb
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
blocking2.0: ? → -
You need to log in before you can comment on or make changes to this bug.