Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435017
--- Comment #12 from Tim Fenn <fenn(a)stanford.edu> 2008-11-10 13:26:45 EDT ---
(In reply to comment #11)
For -8:
* License
- The license tag should be LGPLv2+.
Fixed.
* BuildRequires
- This package won't build without "BuildRequires: mmdb-devel".
Fixed.
* configure option
------------------------------------------------------------
55 ++ pkg-config --variable=prefix mmdb
56 + ./configure --build=i386-redhat-linux-gnu
--host=i386-redhat-linux-gnu --target=i386-redhat-linux-gnu --program-prefix=
--prefix=/usr --exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin
--sysconfdir=/etc --datadir=/usr/share --includedir=/usr/include
--libdir=/usr/lib --libexecdir=/usr/libexec --localstatedir=/var
--sharedstatedir=/usr/com --mandir=/usr/share/man --infodir=/usr/share/info
--with-mmdb=/usr --disable-static
57 configure: WARNING: unrecognized options: --with-mmdb
------------------------------------------------------------
- Is the option "--with-mmdb=foo" useful?
I forgot to include a patch to one of the makefiles to go along with the
configure patch - fixed.
* Timestamps
------------------------------------------------------------
make install DESTDIR=$RPM_BUILD_ROOT install='install -p'
------------------------------------------------------------
- The last argument should be "INSTALL='install -p'".
Fixed.
* Document
- There is no need that -devel subpackage should own the same
document files in main package.
Removed.
* rpmlint
** undefined-non-weak-symbol
- "$ rpmlint ssm" (please try rpmlint for 'installed' ssm) shows
that libssm.so.0 contains many undefined non-weak symbols.
! You can check this also by $ ldd -r %_libdir/libssm.so.0
For packages providing -devel subpackage this cannot be
allowed because this causes linkage error.
I guess making libssm.so linked against libmmdb.so will fix this
issue
(because
LD_PRELOAD=%_libdir/libmmdb.so.0.0.0 ldd -r %_libdir/libssm.so.0.0.0
shows no undefined non-weak symbols)
Yeah, this all gets cleaned up with proper linking to libmmdb.
new spec:
http://www.stanford.edu/~fenn/packs/ssm.spec
new srpm:
http://www.stanford.edu/~fenn/packs/ssm-0.1-9.f8.src.rpm
--
Configure bugmail:
https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.