Bug 919590 - Review Request: python-tftpy - TFTP module for Python
Summary: Review Request: python-tftpy - TFTP module for Python
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 919592
TreeView+ depends on / blocked
 
Reported: 2013-03-08 20:06 UTC by Jeff Bastian
Modified: 2013-03-29 01:37 UTC (History)
4 users (show)

Fixed In Version: python-tftpy-0.6.0-1.fc18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-03-19 17:11:06 UTC
Type: ---
Embargoed:
kevin: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Jeff Bastian 2013-03-08 20:06:16 UTC
Spec URL: http://people.redhat.com/~jbastian/cxmanage/specs/python-tftpy.spec
SRPM URL: http://people.redhat.com/~jbastian/cxmanage/python-tftpy-0.6.0-1.fc18.src.rpm
Description:
    Tftpy is a TFTP library for the Python programming language. It includes
    client and server classes, with sample implementations. Hooks are included
    for easy inclusion in a UI for populating progress indicators. It supports
    RFCs 1350, 2347, 2348 and the tsize option from RFC 2349.
Fedora Account System Username: jbastian

Note: python-tftpy was in Fedora a few years ago, but it seems to have disappeared after Fedora 12.  See bug 434861.

Comment 1 Jeff Bastian 2013-03-08 20:22:11 UTC
This is my first (of three, also see bug 919591 and bug 919592) packages so I need a sponsor.

Comment 2 Kevin Fenzi 2013-03-09 21:00:18 UTC
Hey Jeff. I'd love to see these in, so I am happy to try and work on reviewing and sponsoring you. 

Will try and get you a review of this package later today...

Comment 3 Kevin Fenzi 2013-03-10 18:02:19 UTC
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License (MIT)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
d8de06a543989b910e46d2793f96fb9c5cbd7f99ab014982945b58ae60289893  tftpy-0.6.0.tar.gz
d8de06a543989b910e46d2793f96fb9c5cbd7f99ab014982945b58ae60289893  tftpy-0.6.0.tar.gz.orig

OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good. 
OK - Package is code or permissible content. 

OK - Package compiles and builds on at least one arch. 
OK - Package has no duplicate files in %files. 
OK - Package doesn't own any directories other packages own. 
OK - Package owns all the directories it creates. 
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output. 
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock. 
OK - Should build on all supported archs
OK - Should function as described. 
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin

Issues: 

1. rpmlint says: 

python-tftpy.noarch: W: spelling-error %description -l en_US tsize -> size, t size
python-tftpy.noarch: W: no-manual-page-for-binary tftpy_client.py
python-tftpy.noarch: W: no-manual-page-for-binary tftpy_server.py
python-tftpy.src: W: spelling-error %description -l en_US tsize -> size, t size
2 packages and 0 specfiles checked; 0 errors, 4 warnings.

All those can be ignored. 

I don't see any bockers here, so this package is APPROVED. 

Let me go ahead and review your other two and see about sponsoring you after those are done.

Comment 4 Kevin Fenzi 2013-03-10 18:15:26 UTC
Since the other two are waiting on that ipmitool bug to get fixed, I'll go ahead and sponsor you and you can get this one imported and done.

Comment 5 Jeff Bastian 2013-03-13 17:17:35 UTC
New Package SCM Request
=======================
Package Name: python-tftpy
Short Description: TFTPy is a pure Python implementation of the Trivial FTP protocol
Owners: jbastian
Branches: f18 f19
InitialCC:

Comment 6 Jeff Bastian 2013-03-13 17:50:58 UTC
I see python-tftpy is already in git, but it is retired.  Maybe it just needs to be un-retired for Fedora 18?
http://pkgs.fedoraproject.org/cgit/python-tftpy.git/

Comment 7 Gwyn Ciesla 2013-03-13 18:09:17 UTC
Un-retired for devel, file a Package Change request for f19, f18, etc.

Comment 8 Jeff Bastian 2013-03-13 18:26:13 UTC
Package Change Request
======================
Package Name: python-tftpy
New Branches: f18 f19
Owners: jbastian

Comment 9 Gwyn Ciesla 2013-03-13 18:29:45 UTC
Git done (by process-git-requests).

Comment 10 Jeff Bastian 2013-03-13 19:36:27 UTC
I'm waiting for the package to be unblocked before I can build.
https://fedorahosted.org/rel-eng/ticket/5531

Comment 11 Jeff Bastian 2013-03-19 17:11:06 UTC
It worked:
http://koji.fedoraproject.org/koji/buildinfo?buildID=403825

Comment 12 Fedora Update System 2013-03-19 17:16:07 UTC
python-tftpy-0.6.0-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/python-tftpy-0.6.0-1.fc18

Comment 13 Fedora Update System 2013-03-29 01:37:26 UTC
python-tftpy-0.6.0-1.fc18 has been pushed to the Fedora 18 stable repository.


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