[Bug 226189] Merge Review: neon

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: neon


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226189





------- Additional Comments From tibbs@xxxxxxxxxxx  2007-02-18 01:40 EST -------
Interestingly, once the package is installed, rpmlint produces the following
additional warnings:

W: neon unused-direct-shlib-dependency /usr/lib64/libneon.so.25.0.5
/usr/lib64/libkrb5.so.3
W: neon unused-direct-shlib-dependency /usr/lib64/libneon.so.25.0.5
/usr/lib64/libk5crypto.so.3
W: neon unused-direct-shlib-dependency /usr/lib64/libneon.so.25.0.5
/lib64/libcom_err.so.2
W: neon unused-direct-shlib-dependency /usr/lib64/libneon.so.25.0.5
/lib64/libresolv.so.2
W: neon unused-direct-shlib-dependency /usr/lib64/libneon.so.25.0.5
/lib64/libdl.so.2

I will admit to not having a clue as to what might cause this or how to get rid
of it, as I've never seen this warning from rpmlint before.

I note there are a few Conflicts with an extremely old version of subversion. 
Even the subversion in FC1 was newer than the problem version, so there's really
no reason for the Conflicts bits to be there these days.

There seems to be a test suite and according to %changelog it was run at some
point but doesn't seem to be run now.  I added a %check section and got things
to run until:

./uri-tests: error while loading shared libraries: libz.so.1: failed to map
segment from shared object: Cannot allocate memory

Not sure what's up there, so probably best not to try to run the test suite.

The -devel package includes both a static library and a .la file.  According to
the guidelines, neither of these should be there except in exceptional
circumstances.

Review:
* source files match upstream:
   b5513f88cb54c5f11e4c8348ee6c7ace9767b45c263c3a3ba8a5ce4e2b40a07a
   neon-0.25.5.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
O dist tag is not present (not required)
* build root is correct.
* license field matches the actual license.
* license is open source-compatible (LGPL)
* License text included in package.
* latest version is being packaged (latest in the 0.25 series)
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
? rpmlint has complaints, but I'm not sure what they mean.
* final provides and requires are sane:
  neon-0.25.5-6.x86_64.rpm
   libneon.so.25()(64bit)
   neon = 0.25.5-6
  =
   /sbin/ldconfig
   libcom_err.so.2()(64bit)
   libcrypto.so.6()(64bit)
   libexpat.so.0()(64bit)
   libgssapi_krb5.so.2()(64bit)
   libgssapi_krb5.so.2(gssapi_krb5_2_MIT)(64bit)
   libk5crypto.so.3()(64bit)
   libkrb5.so.3()(64bit)
   libneon.so.25()(64bit)
   libssl.so.6()(64bit)
   libz.so.1()(64bit)

  neon-devel-0.25.5-6.x86_64.rpm
   neon-devel = 0.25.5-6
  =
   /bin/sh
   expat-devel
   libneon.so.25()(64bit)
   neon = 0.25.5-6
   openssl-devel
   pkgconfig
   zlib-devel

* %check is not present; the included test suite doesn't actually run.
* shared libraries present; ldconfig is called properly.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (ldconfig)
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* headers are in the -devel subpackage.
* a pkgconfig file is present and in the -devel package; pkgconfig is a
  dependency.
X a .la file is present, as well as a static library.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]