Bug 250150 - Review Request: rarian - a replacement for scrollkeeper
Review Request: rarian - a replacement for scrollkeeper
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Matthias Clasen
Fedora Extras Quality Assurance
:
: 226401 (view as bug list)
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-07-30 14:22 EDT by Matthew Barnes
Modified: 2008-01-23 07:26 EST (History)
6 users (show)

See Also:
Fixed In Version: rarian-0.5.6-2.fc8
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-08-02 13:44:50 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mclasen: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Matthew Barnes 2007-07-30 14:22:27 EDT
Spec URL:
http://baloo.boston.redhat.com/~mbarnes/rarian/rarian.spec

SRPM URL:
http://baloo.boston.redhat.com/~mbarnes/rarian/rarian-0.5.4-1.fc7.src.rpm

Description:

Rarian is a documentation meta-data library, designed as a replacement for Scrollkeeper.

I've split this into a main package and two subpackages:

rarian         Contains the Rarian library ("librarian").
rarian-devel   Contains a pkg-config file and some header files.
rarian-compat  Provides all the old scrollkeeper commands.
Comment 1 Matthias Clasen 2007-07-31 10:13:11 EDT
Trivia: 
- please use a full source url

- devel package needs to require pkgconfig

- compat needs to Requires(post): rarian, I think

- mock build fails with

make[3]: Entering directory `/builddir/build/BUILD/rarian-0.5.4/util'
Moving Files if needed
mkdir: cannot create directory `/var/lib/rarian': Permission denied
make[3]: *** [install-exec-hook] Error 1

I guess there is some DESTDIR missing somewhere
Comment 2 Matthew Barnes 2007-07-31 13:44:39 EDT
Posted an update with your suggestions.

SRPM URL:
http://baloo.boston.redhat.com/~mbarnes/rarian/rarian-0.5.4-2.fc7.src.rpm

Still investigating what's causing the mock failure.
Comment 3 Matthew Barnes 2007-08-01 12:00:42 EDT
Updated to 0.5.6.  This one builds successfully in mock.

SRPM URL:
http://baloo.boston.redhat.com/~mbarnes/rarian/rarian-0.5.6-1.fc7.src.rpm
Comment 4 Matthias Clasen 2007-08-01 13:01:02 EDT
rpmlint output:

[mclasen@dhcp83-186 Desktop]$ rpmlint rarian-0.5.6-1.fc8.i386.rpm 
E: rarian zero-length /usr/share/doc/rarian-0.5.6/AUTHORS

This one should be fixed upstream, I guess. I poked Don.
 
E: rarian explicit-lib-dependency libxslt

This is ok, because it is actually requiring /usr/bin/xsltproc here
Might want to add a comment next to the Requires: to explain that

W: rarian obsolete-not-provided scrollkeeper

I think the way you handle the Obsoletes/Provides here makes sense.

 rpmlint rarian-devel-0.5.6-1.fc8.i386.rpm 
W: rarian-devel no-documentation

Ignorable

 rpmlint rarian-compat-0.5.6-1.fc8.i386.rpm 
W: rarian-compat no-documentation

Ignorable

W: rarian-compat devel-file-in-non-devel-package /usr/bin/rarian-sk-config

rpmlint is just wrong here

W: rarian-compat one-line-command-in-%post /usr/bin/rarian-sk-update

I think that is fine, and using -p here would be pretty odd


Detailed review:

package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field: 
  - I think the -compat package should have a License: GPL,
    since the utils are GPL, not LGPL
  - if you want to preemptively adopt the new license field rules, the
    license tags should be LGPLv2+ and GPLv2+, respectively
license file: should include COPYING.LIB and COPYING.UTILS 
spec language: ok
spec legibility: ok
upstream sources: ok
buildable: ok
excludearch: n/a
build deps: ok
ldconfig: ok
relocatable: n/a
directory ownership: ok
file list dupes: ok
file permissions: ok
%clean: ok
macro use: ok
content: permissible
large docs: n/a
%doc: ok
headers: ok
static libs: n/a
pc files: ok
shared lib symlinks: ok
devel package: ok
la files: ok
gui apps: n/a
file ownership: ok
%install ok
utf8 filenames: ok
Comment 5 Matthew Barnes 2007-08-01 13:23:49 EDT
(In reply to comment #4)
> E: rarian explicit-lib-dependency libxslt
> 
> This is ok, because it is actually requiring /usr/bin/xsltproc here
> Might want to add a comment next to the Requires: to explain that

I figured rpmlint was just getting confused by the fact that the package itself
is named like a library.  Added a comment just the same.

Updated SRPM:
http://baloo.boston.redhat.com/~mbarnes/rarian/rarian-0.5.6-2.fc7.src.rpm
Comment 6 Matthias Clasen 2007-08-01 13:49:51 EDT
looking fine now. Approved.
Comment 7 Matthew Barnes 2007-08-02 12:27:33 EDT
New Package CVS Request
=======================
Package Name: rarian
Short Description: a documentation meta-data library, replaces scrollkeeper
Owners: mbarnes@redhat.com
Branches: (just devel)
InitialCC: mclasen@redhat.com
Comment 8 Kevin Fenzi 2007-08-02 12:46:00 EDT
cvs done.
Comment 9 Matthew Barnes 2007-08-02 13:44:50 EDT
Cool, thanks Kevin.  I guess we can close this now.
Comment 10 Bill Nottingham 2007-08-02 16:49:27 EDT
The obsoletes in this aren't going to work right.

As it stands, rarian will obsolete rarian-compat because of the Provides. What
you probably want is:

rarian: Obsoletes: scrollkeeper <= (last scrollkeeper release)
rarian-compat: Provides: scrollkeeper = <some random # higher than last
scrollkeeper release>
Comment 11 Matthew Barnes 2007-08-02 19:54:06 EDT
That makes sense.  Should be fixed in rarian-0.5.6-3.fc8.
Comment 12 Yijun Yuan 2007-08-04 09:12:09 EDT
Still not fixed:


[root@mstar packages]# rpm -Uvh rarian-0.5.6-3.fc8.i386.rpm --nodeps
Preparing...                ########################################### [100%]
   1:rarian                 ########################################### [100%]
[root@mstar packages]# rpm -q --provides rarian
librarian.so.0  
rarian = 0.5.6-3.fc8
Comment 13 Yijun Yuan 2007-08-04 09:15:09 EDT
@_@ /me just found that rarian-compat is another package

but yum install rarian will not bring in rarian-compat thus fails. yum-3.2.2-3.fc8
Comment 14 Mamoru TASAKA 2008-01-23 07:26:48 EST
*** Bug 226401 has been marked as a duplicate of this bug. ***

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