Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 190129 - Review Request: python-krbV
Summary: Review Request: python-krbV
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2006-04-27 20:22 UTC by Mike Bonnet
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2006-05-21 21:05:01 UTC
Type: ---
wtogami: fedora-cvs+

Attachments (Terms of Use)

Description Mike Bonnet 2006-04-27 20:22:16 UTC
Spec URL:
Description: python-krbV allows python programs to use Kerberos 5 authentication/security

Comment 1 Mike Bonnet 2006-05-19 18:27:50 UTC
This has been waiting around for a review for a while now.  Any chance of
getting it reviewed?

Comment 2 Jason Tibbitts 2006-05-19 21:36:54 UTC
I'm no Python expert but I'll have a go.

Comment 3 Jason Tibbitts 2006-05-19 22:52:55 UTC
A few issues:

specfile in srpm is mode 0600 and source tarball in srpm is mode 0664.  Both
should be 644.  These probably don't matter once things are in CVS, but just in
case I would fix them before checking in.

There's no reason to BuildRequires: python; python-devel will pull it in (not a

You use %{optflags} instead of $RPM_OPT_FLAGS (which is fine), but still use
$RPM_BUILD_ROOT instead of %{buildroot}.  The guidelines explicitly discourage

* package meets naming and packaging guidelines.
X specfile is properly named and is cleanly written but does not use macros
consistently (see $RPM_BUILD_ROOT comment above).
* license field matches the actual license.
* license is open source-compatible; license text is included in the package.
* source files match upstream (makes sense; you are the upstream):
   b79db9912efd76ab9a88441e455455d4  python-krbV-1.0.12.tar.gz
   b79db9912efd76ab9a88441e455455d4  python-krbV-1.0.12.tar.gz-srpm
* latest version is being packaged.
O BuildRequires are proper (BR: python not needed but not a blocker).
* package builds in mock (development, x86_64).
O rpmlint is silent (it does warn about inappropriate permissions in the SRPM).
* final provides and requires are sane:
   python-krbV = 1.0.12-2.fc6
   python(abi) = 2.4
   python-abi = 2.4
   rpmlib(CompressedFileNames) <= 3.0.4-1
   rpmlib(PayloadFilesHavePrefix) <= 4.0-1
* shared libraries are present, but internal to Python so there's no need to run
* no .pyo files to %ghost.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
O file permissions are appropriate, except in the srpm.
* %clean is present.
O %check is not present; no test suite upstream.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.

Comment 4 Mike Bonnet 2006-05-21 18:13:05 UTC
New specfile and srpm posted:

Fixed .spec permissions (rpmbuild -ts was doing something funny).
Removed BuildRequires: python
Replaced $RPM_BUILD_ROOT with %{buildroot}

Comment 5 Jason Tibbitts 2006-05-21 18:38:38 UTC
Odd that the tarball is still mode 664, but this definitely won't matter after
you import.  Otherwise, everything looks good.


Comment 6 Mike Bonnet 2006-05-21 21:05:01 UTC
Thanks, built in devel.

Comment 7 Mike Bonnet 2007-05-07 18:34:34 UTC
Package Change Request
Package Name: python-krbV
New Branches: EL-4 EL-5 FC-5

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