Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
Summary: Review Request: python-krbV Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: bugzilla-sink@leemhuis.info ReportedBy: mikeb@redhat.com QAContact: fedora-package-review@redhat.com
Spec URL: http://people.redhat.com/mikeb/python-krbV/python-krbV.spec SRPM URL: http://people.redhat.com/mikeb/python-krbV/python-krbV-1.0.12-2.src.rpm Description: python-krbV allows python programs to use Kerberos 5 authentication/security
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
------- Additional Comments From mikeb@redhat.com 2006-05-19 14:27 EST ------- This has been waiting around for a review for a while now. Any chance of getting it reviewed?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|bugzilla-sink@leemhuis.info |tibbs@math.uh.edu OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From tibbs@math.uh.edu 2006-05-19 17:36 EST ------- I'm no Python expert but I'll have a go.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
------- Additional Comments From tibbs@math.uh.edu 2006-05-19 18:52 EST ------- 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 blocker).
You use %{optflags} instead of $RPM_OPT_FLAGS (which is fine), but still use $RPM_BUILD_ROOT instead of %{buildroot}. The guidelines explicitly discourage this: http://fedoraproject.org/wiki/Packaging/Guidelines#head-f3d77b27a5d29dfc1f56...
Review: * 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: krbVmodule.so()(64bit) python-krbV = 1.0.12-2.fc6 - libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.4)(64bit) libkrb5.so.3()(64bit) libkrb5.so.3(krb5_3_MIT)(64bit) 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 ldconfig. * 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
------- Additional Comments From mikeb@redhat.com 2006-05-21 14:13 EST ------- New specfile and srpm posted:
http://people.redhat.com/mikeb/python-krbV/python-krbV.spec http://people.redhat.com/mikeb/python-krbV/python-krbV-1.0.12-3.src.rpm
Fixed .spec permissions (rpmbuild -ts was doing something funny). Removed BuildRequires: python Replaced $RPM_BUILD_ROOT with %{buildroot}
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From tibbs@math.uh.edu 2006-05-21 14:38 EST ------- Odd that the tarball is still mode 664, but this definitely won't matter after you import. Otherwise, everything looks good.
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
mikeb@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From mikeb@redhat.com 2006-05-21 17:05 EST ------- Thanks, built in devel.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium
mikeb@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From mikeb@redhat.com 2007-05-07 14:34 EST ------- Package Change Request ====================== Package Name: python-krbV New Branches: EL-4 EL-5 FC-5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129
wtogami@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: python-krbV
https://bugzilla.redhat.com/show_bug.cgi?id=190129
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Product|Fedora Extras |Fedora Version|devel |rawhide
package-review@lists.fedoraproject.org