[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-21 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=521716

Bug 521716 depends on bug 560457, which changed state.

Bug 560457 Summary: Review Request: pyutil - A collection of mature utilities 
for Python programmers
https://bugzilla.redhat.com/show_bug.cgi?id=560457

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-21 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=521716

Ruben Kerkhof ru...@rubenkerkhof.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-19 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=521716

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2010-02-19 13:56:07 EST 
---
CVS done (by process-cvs-requests.py).

Also added an F-13 branch.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-16 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=521716

--- Comment #6 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-02-16 
11:44:51 EST ---
FYI I approved pyutil and python-zbase.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-16 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=521716

Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #8 from Thomas Spura toms...@fedoraproject.org 2010-02-16 
12:02:21 EST ---
(In reply to comment #6)
 FYI I approved pyutil and python-zbase.

Thank you.


#

APPROVED

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-02-16 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=521716

Ruben Kerkhof ru...@rubenkerkhof.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Ruben Kerkhof ru...@rubenkerkhof.com 2010-02-16 15:57:20 
EST ---
New Package CVS Request
===
Package Name: python-zfec
Short Description: A fast erasure codec with python bindings
Owners: ruben
Branches: F-12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-01-31 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=521716

Ruben Kerkhof ru...@rubenkerkhof.com changed:

   What|Removed |Added

 Depends on||560457

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-01-31 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=521716

--- Comment #4 from Thomas Spura toms...@fedoraproject.org 2010-01-31 
16:34:27 EST ---
- sources matches upstream: 542957786c0b5de3ff76094349d0cf0d


Only SHOULDS left:

(In reply to comment #3)
  - Query upstream to add license headers.
 Why, the license is included

I remember, it's negative to not have headers, but can't find it in the
guidelines. Only in COPYING.GPL it's just suggested in 'How to Apply These
Terms to Your New Programs' too. Because you seem to be in contact with
upstream, you could ask them to add the headers.
But this is a should anyway... ;)


- Deleting the tests would be good:
  Then there would be installed just what is needed.
  Here it doesn't matter much, because the tests are small...


##


I'd approve this now, but I want to wait for the dependencies.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-01-31 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=521716

--- Comment #5 from Ruben Kerkhof ru...@rubenkerkhof.com 2010-01-31 17:09:00 
EST ---
Hi Thomas,

Ok, I'll delete the tests, and I've asked upstream to add license headers.

Would you mind taking a look at pyutil and python-zbase32 as well?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings

2010-01-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=521716

--- Comment #2 from Ruben Kerkhof ru...@rubenkerkhof.com 2010-01-24 18:09:47 
EST ---
Hi Thomas, thanks for the review!

Upstream released 1.4.6, which includes a few fixes for the issues you
mentioned.
I asked them to push the new version to pypi, when that's done I'll create a
new version. Should be ready in a few days.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review