Bug 227198 (jpgalleg)

This is my first review, so this is kind of unofficial.  So here goes:

- rpmlint -i is silent on the src.rpm.
- Name of spec file matches name of package which matches base part of source
code name
- License in License tag seems to match the license in the source code, but
would it be possible to be more specific that zlib/libpng?  (zlib has a BSD
license, libpng has an "OSI certified license" that looks kind of BSD-ish.)
- source file matches that given in the URL.
- spec file successfuly builds jpgalleg, jpgalleg-devel, and jpgalleg-debuginfo
RPMS on i386.  (I don't have access to any other architectures.)
- BuildRequires seems good.
- calls ldconfig in %post and %postun as it should.
- premissions look good.
- Requires: for the -devel subpackage look good.

I haven't verified that it builds in mock yet, but everything else except for
the license issue above looks quite good.  Even the license thing is minor.

(In reply to comment #1)
> This is my first review, so this is kind of unofficial.  So here goes:
> 
> - License in License tag seems to match the license in the source code, but
> would it be possible to be more specific that zlib/libpng?  (zlib has a BSD
> license, libpng has an "OSI certified license" that looks kind of BSD-ish.)

zlib/libpng License is an often used term, its even in the list of licenses
rpmlint recognises as valid, also this is how upstream refers to the license for
this package, thus I believe this best describes the license.

My apologies.  I was not aware of that.  Thanks for the info.  (As you can tell,
I'm *really* new at this.  Hence, why I didn't make this an official review.)

==== REVIEW GUIDELINES ====
- rpmlint output:
W: jpgalleg-devel no-documentation
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_cx_r
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_cx_w
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_get_cpuid_info
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_486
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5
_i_is_cpuid_supported
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_cyrix
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_fpu

Hans: can you please investigate these rpmlint warnings before I continue with
the review?

Those are "normal" for an allegro extension library. These symbols are defined
in /usr/lib/liballeg_unsharable.a, against which each allegro using app should
be linked (allegro-config --libs).

As you can see in the .spec file for libraries build against allegro like
libjpeg, liballeg_unsharable.a is stripped from the allegro-config --libs
output, as including this would make the library non pic, which is why these
routines are in the .a file and are linked into the binary, not into any libs.

==== REVIEW GUIDELINES ====
- rpmlint output:
W: jpgalleg-devel no-documentation
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_cx_r
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_cx_w
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_get_cpuid_info
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_486
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5
_i_is_cpuid_supported
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_cyrix
W: jpgalleg undefined-non-weak-symbol /usr/lib64/libjpgal.so.2.5 _i_is_fpu

Okay, see comment #5

- package named according to package naming guidelines
- spec file name matches %{name}
- package meets packaging guidelines
- licensed with open source compatible license
- license field matches actual license
- license text included in %doc
- spec written in American english
- spec legible
- sources match upstream 4bdd12fc3c1afb06f4ec23b4cb200559
- successfully compiles and builds on FC-6 x86_64
- all build dependencies listed in BR
- no locales
- ldconfig called in %post/un sections
- package is not relocatable
- package owns all directories it creates
- directories it does not owned belong to filesystem
- no duplicates in %files
- file permissions set properly
- contains proper %clean
- macro usage is consistent
- contains code
- no large documentation
- files in %doc do not affect runtime
- header files located in devel subpackage
- no static libraries
- no pkgconfig files
- .so file in devel subpackage
- devel have fully versioned dependency
- no libtool archive files
- not a GUI application
- package does not own files or directories owned by other packages

*** APPROVED ***

New Package CVS Request
=======================
Package Name:       jpgalleg
Short Description:  JPEG library for the Allegro game library
Owners:             j.w.r.degoede
Branches:           FC-6 devel
InitialCC:          <empty>

Thanks for the review!

Imported and build, closing.