[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-08 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-08 14:29 EST ---
I'll take into consideration your suggestions.

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-19 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-19 12:59 EST ---
I've moved sources:

Spec URL: http://stingr.net/~ivaxer/python-tftpy/python-tftpy.spec
SRPM URL: 
http://stingr.net/~ivaxer/python-tftpy/python-tftpy-0.4.5-2.fc9.src.rpm

I need a sponsor..


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-19 Thread bugzilla
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-tftpy - Python TFTP library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||182235
  nThis||
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-19 14:31 EST ---
To spot:
Would you add Python 1.6b1-2.0,2.1 license to Fedora license list?
Currently the spec file by John uses the license tag "Python", however actually
this package uses Python 1.6b1-2.0,2.1 license (OSI seems to be using "CNRI") 
[1]
which is GPL incompatible according to [2] where current Python license is GPL
compatible.

[1] http://www.opensource.org/licenses/pythonpl.php
[2] http://www.gnu.org/licenses/license-list.html#GPLIncompatibleLicenses

To John:
I will check this package tomorrow, however
would you have another review request or have done a pre-review of other 
person's
review request?
-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
http://fedoraproject.org/PackageReviewStatus/NEW.html
(NOTE: please don't choose "Merge Review")


Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-20 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 08:11 EST ---
My pre-review in #455581 bug.

I also have almost finished package Ceph FS (http://ceph.newdream.net) for 
fedora.
But I have not yet submit to review, because sources non-stable.
Spec here:
http://stingr.net/git/?p=ivaxer/rpm/ceph;a=tree
It would be great if Ceph was aproved in fedora. May be finish and submit?

Thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-20 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 13:25 EST ---
Well,

* This package itself is okay, except for license tag
* Your pre-review seems good for initial comments

Now I will wait for the reply from spot.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-20 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 13:40 EST ---
By the way:
(In reply to comment #12)
> I also have almost finished package Ceph FS (http://ceph.newdream.net) for 
> fedora.
> But I have not yet submit to review, because sources non-stable.
> Spec here:
> http://stingr.net/git/?p=ivaxer/rpm/ceph;a=tree
> It would be great if Ceph was aproved in fedora. May be finish and submit?

Currently any new packages which requires kernel modules outside kernel tree or
external kernel modules are banned for Fedora.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-20 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-20 14:42 EST ---
(In reply to comment #13)
> Well,
> 
> * This package itself is okay, except for license tag
> * Your pre-review seems good for initial comments
> 

Thanks.

(In reply to comment #14)
> By the way:
> 
> Currently any new packages which requires kernel modules outside kernel tree 
> or
> external kernel modules are banned for Fedora.
> 

Oh, yes, I'm sorry.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #16 from John Khvatov <[EMAIL PROTECTED]>  2008-08-07 10:29:10 EDT 
---
I sent another review-request #458288.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #17 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-14 03:18:55 EDT 
---
Just leaving a comment that I am waiting for spot's license check.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #18 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-23 12:37:51 EDT 
---
To Spot:

Sorry again, however would you review the license of this package
(as written in my comment 11)?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #19 from Tom "spot" Callaway <[EMAIL PROTECTED]>  2008-08-23 
12:54:52 EDT ---
License is OK, Free but GPL incompatible. I've added it to the license list in
1.03, use:

License: CNRI

Lifting FE-Legal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Tom "spot" Callaway <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks|182235  |




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #20 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-23 13:00:11 EDT 
---
To spot: Thanks!!

To John:
Now would you change the license tag and modify anything which you think is
needed
and re-upload the srpm? Then I will recheck your srpm.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks|177841  |




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #21 from John Khvatov <[EMAIL PROTECTED]>  2008-08-24 04:17:01 EDT 
---
Fixed license tag:
Spec URL: http://stingr.net/~ivaxer/python-tftpy/python-tftpy.spec
SRPM URL:
http://stingr.net/~ivaxer/python-tftpy/python-tftpy-0.4.5-3.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #22 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-24 12:11:29 EDT 
---
--
This package (python-tftpy) is APPROVED by mtasaka
--

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #23 from John Khvatov <[EMAIL PROTECTED]>  2008-08-24 13:10:19 EDT 
---
New Package CVS Request
===
Package Name: python-tftpy
Short Description: A Pure-Python library for TFTP
Owners: ivaxer
Branches: F-9
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #24 from Kevin Fenzi <[EMAIL PROTECTED]>  2008-08-24 15:10:03 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #25 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-29 02:19:54 EDT 
---
Please submit a request on bodhi to push F-9 package to the repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #26 from Fedora Update System <[EMAIL PROTECTED]>  2008-08-29 
05:35:14 EDT ---
python-tftpy-0.4.5-3.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/python-tftpy-0.4.5-3.fc9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-08-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #27 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-29 05:52:20 EDT 
---
Thanks. Now closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-09-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #28 from Fedora Update System <[EMAIL PROTECTED]>  2008-09-10 
02:37:37 EDT ---
python-tftpy-0.4.5-3.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-03-28 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-28 14:41 EST ---
PING?!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-06-03 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-03 10:16 EST ---
Spec URL: http://spaconf.sgu.ru/rpm/python-tftpy.spec
SRPM URL: http://spaconf.sgu.ru/rpm/python-tftpy-0.4.5-1.fc9.src.rpm

Update to upstream (0.4.5) and rebuild for f9.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-06-21 Thread bugzilla
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-tftpy - Python TFTP library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-06-21 19:43 EST ---
I don't see you in the account system at all; is this your first package for 
Fedora?

I'm assuming that it is; setting things appropriately.  See
http://fedoraproject.org/wiki/PackageMaintainers/Join for more information.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-06-22 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-22 08:55 EST ---
Yes, this is my first package in Fedora.
I have registered the Fedora account(name: ivaxer), I hope that nothing is 
missed.

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-05 Thread bugzilla
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-tftpy - Python TFTP library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-05 11:42 EST ---
I am not sponsored yet. So, cannot officially review your package. But to help 
here is an:

Unofficial package review
===
rpmlint -i :
python-tftpy.src: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 4)
python-tftpy.src: E: description-line-too-long Tftpy is a TFTP library for the 
Python programming language. It includes client and server classes, with sample 
implementations. Hooks are included for
python-tftpy.src: E: description-line-too-long easy inclusion in a UI for 
populating progress indicators. It supports RFCs 1350, 2347 and 2348.

rpmlint -i 
python-tftpy.noarch: W: spurious-executable-perm /usr/share/doc/python-
tftpy-0.4.5/bin/tftpy_server.py
python-tftpy.noarch: W: doc-file-dependency /usr/share/doc/python-tftpy-0.4.5/
bin/tftpy_server.py /usr/bin/env


- You seem to have mixed tabs and spaces
- Try wrapping description to 80 columns (1 column = 1 char) a line
- bin files -- 'bin/tftpy_client.py' & 'bin/tftpy_server.py' should go to 'doc' 
but they should go to 'bin'
 you are excluding .pyc and .pyo but files are not there.  
- you need to replace env in these files with %{__python} -- may be consult 
reviewer
 
Required 
=
[x] Package naming & spec file naming 
[x] md5sum matches for source from srpm and project page
[?] License field present, but file COPYING does not look license to me -- 
 It says 'if you press accept' so essentially it is being used  while using 
this program. 
 It looks okay to me - but may be do some more research.
[x] spec file is legible and written in American English
[x] All files are in utf-8 
[x] no static libraries, no libtool, package does not own files or folders used 
by other packages.
[x] nothing used in %doc for runtime.
[x] consistently used macros
[x] package build and compile using koji 
[-] tabs and spaces mixed 
[-] dependency env warning
[-] description is okay but needs to be wrapped.

Optional 
=
[x] latest stable packaged 
[?] includes a COPYING file but needs some more research
  
-N/A, x = check, ! = Problem and ? = Not evaluated 

It is okay, just needs some quick fixs in my few.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-05 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-05 13:52 EST ---
typo*

Keys:
- = Problem, x = check, and ? = Not evaluated

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-06 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-06 04:23 EST ---
Thanks for review.

I fix problems:
rpmlint -i python-tftpy-0.4.5-2.fc9.src.rpm:
1 packages and 0 specfiles checked; 0 errors, 0 warnings

rpmlint -i python-tftpy-0.4.5-2.fc9.noarch.rpm:
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Moved 'bin/tftpy_server.py' and 'bin/tftpy_client.py' to %{_bindir}.

Spec URL: http://212.193.33.181/rpm/python-tftpy.spec
SRPM URL: http://212.193.33.181/rpm/python-tftpy-0.4.5-2.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 434861] Review Request: python-tftpy - Python TFTP library

2008-07-06 Thread bugzilla
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-tftpy - Python TFTP library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-06 05:46 EST ---
I am mixing my keys :P, but you got them right.
[-] = Problem or suggestion, [x] = Checked, ? = No idea  

Package looks okay to me.

Just few more optional suggestions
[-] Changelog message should be about what changes you did to reach this 
version. You may consider providing a better log ;-) like issues you resolved.

[-] you may consider removing PKG-INFO if you are not installing it from srpm.

Thanks



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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