Bug 198247 - Review Request: libpng10
Summary: Review Request: libpng10
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Michael J Knox
QA Contact: Fedora Package Reviews List
Depends On:
Blocks: FE-ACCEPT 195858 198245
TreeView+ depends on / blocked
Reported: 2006-07-10 15:31 UTC by Paul Howarth
Modified: 2010-05-11 04:33 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2006-08-02 13:07:07 UTC
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Paul Howarth 2006-07-10 15:31:49 UTC
Spec URL: http://www.city-fan.org/~paul/extras/Gnome-1/libpng10.spec
SRPM URL: http://www.city-fan.org/~paul/extras/Gnome-1/libpng10-1.0.20-1.src.rpm


This package was removed from Core for FC6 and I am submitting it to Extras to support the people that still need it for legacy applications. I pulled the package from Core CVS and then tweaked it for Extras and to suit my own cosmetic preferences. I also updated it to the latest upstream version and updated the configuration patch so that it would apply against the new version. Reviewers may consider using rpmdiff to compare the built packages with the versions in Fedora Core 5.

Comment 1 Rex Dieter 2006-07-10 16:13:35 UTC
Since -devel has files in %{_libdir}/pkgconfig , to avoid possible unowned 
dirs, it should:
Requires: pkgconfig

Comment 2 Paul Howarth 2006-07-10 16:24:57 UTC
(In reply to comment #1)
> Since -devel has files in %{_libdir}/pkgconfig , to avoid possible unowned 
> dirs, it should:
> Requires: pkgconfig

It already does:

%package devel
Group:		Development/Libraries
Summary:	Development tools for version 1.0 of libpng
Requires:	libpng10 = %{version}-%{release}, zlib-devel, pkgconfig

Comment 3 Rex Dieter 2006-07-10 16:28:41 UTC
doh, sorry for the noise (I'm used to only looking at one Requires: item per 

Comment 4 Michael J Knox 2006-07-19 21:41:31 UTC
Hi, I will review this one for you. 

Review for release 1:

* RPM name is OK
* Source libpng-1.0.20.tar.bz2 is the same as upstream
* Builds fine in mock
* rpmlint of libpng10 looks OK
* rpmlint of libpng10-devel looks OK
* File list of libpng10 looks OK
* File list of libpng10-devel looks OK


Looks good. I am happy, consider it APPROVED. Please remember to close the bug
report once this has been imported into CVS. 

Comment 5 Paul Howarth 2006-07-20 15:42:51 UTC
I need some help with this. It's failing to build on x86_64:


At first I thought it was out-of-order building due to a broken Makefile and the
use of %{?_smp_mflags}, but that wasn't it. I now suspect it may be to do with
the use of assembly code. The strange thing is that it builds fine on x86_64
RHEL3 (the only x86_64 box I have access to).

It could probably be fixed by using the configure script instead of the old
Makefiles, but doing it that way I can't find any way to make it produce shared
libraries with an soname (libpng.so. that lines up with what's in
previous Fedora/Red Hat releases.

Comment 6 Michael J Knox 2006-07-21 01:28:57 UTC
I have a x86-64 machine at work. I will try and load up FC5 on it and take a look. 

Comment 7 Paul Howarth 2006-07-21 06:44:53 UTC
Thanks. I plan to replace my home machine with a 64-bit one next month but I'd
like to sort this out sooner :-)

Comment 8 Michael J Knox 2006-07-25 22:07:24 UTC
argh... bab memory makes for very interesting errors not relating to libpng10. 

Hopefully be able to sort out this hardware by weeks end.

Comment 9 Michael J Knox 2006-07-25 22:31:03 UTC
Hey Paul, just curious as to why we don't use the configure script in this package? 

I added 


to the spec and it builds on x86-64 now. Though there are files that are not
listed in %files

Comment 10 Paul Howarth 2006-07-26 06:56:09 UTC
I'd love to use the configure script. However, I can't find any way to tweak it
to get it to produce the right soname. For instance, the libpng10-1.0.18 package
in FC5 has libpng.so.; it follows that this libpng10-1.0.20 package
should have libpng.so. for compatibility (which is the main reason for
the existence of this package) but I can't get the configure script, or more
exactly, libtool, to generate an object with that soname.

I think the configure script output could be useful though, as it should produce
the correct set of defines to be used when building the package using the older

Comment 11 Paul Howarth 2006-08-01 16:25:29 UTC
OK, I've managed to persuade, with the aid of a patch, the configure script to
generate the correct soname. Can you give this one a try on rawhide x86_64?

SPEC: http://www.city-fan.org/~paul/extras/Gnome-1/libpng10.spec
SRPM: http://www.city-fan.org/~paul/extras/Gnome-1/libpng10-1.0.20-3.src.rpm

Comment 12 Michael J Knox 2006-08-01 22:26:46 UTC
Build on x86-64 now. Nice work fixing that. 

Comment 13 Paul Howarth 2006-08-02 13:07:07 UTC
 13589 (libpng10): Build on target fedora-development-extras succeeded.
     Build logs may be found at

Thanks for the review and the help.

Comment 14 Paul Howarth 2010-05-10 12:10:38 UTC
Package Change Request
Package Name: libpng10
New Branches: EL-6
Owners: pghmcfc

libpng10 has been dropped from EL-6 (it was present in previous EL releases).

Comment 15 Kevin Fenzi 2010-05-11 04:33:41 UTC
cvs done.

Note You need to log in before you can comment on or make changes to this bug.