[Bug 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|django-threadedcomments-0.5 |django-threadedcomments-0.5
   |.3-2.el5|.3-2.fc14

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

--- Comment #14 from Fedora Update System  
2010-12-20 17:01:01 EST ---
django-threadedcomments-0.5.3-2.fc14 has been pushed to the Fedora 14 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|django-threadedcomments-0.5 |django-threadedcomments-0.5
   |.3-2.fc14   |.3-2.fc13

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

--- Comment #15 from Fedora Update System  
2010-12-20 17:04:37 EST ---
django-threadedcomments-0.5.3-2.fc13 has been pushed to the Fedora 13 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||django-threadedcomments-0.5
   ||.3-2.el5
 Resolution||ERRATA
Last Closed||2010-12-20 12:29:24

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-20 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=645761

--- Comment #13 from Fedora Update System  
2010-12-20 12:29:19 EST ---
django-threadedcomments-0.5.3-2.el5 has been pushed to the Fedora EPEL 5 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-02 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=645761

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2010-12-02 11:33:43 EST ---
django-threadedcomments-0.5.3-2.el5 has been pushed to the Fedora EPEL 5
testing repository.  If problems still persist, please make note of it in this
bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update django-threadedcomments'.  You
can provide feedback for this update here:
https://admin.fedoraproject.org/updates/django-threadedcomments-0.5.3-2.el5

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-01 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=645761

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-01 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=645761

--- Comment #9 from Fedora Update System  2010-12-01 
15:31:16 EST ---
django-threadedcomments-0.5.3-2.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/django-threadedcomments-0.5.3-2.fc14

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-01 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=645761

--- Comment #10 from Fedora Update System  
2010-12-01 15:32:38 EST ---
django-threadedcomments-0.5.3-2.fc13 has been submitted as an update for Fedora
13.
https://admin.fedoraproject.org/updates/django-threadedcomments-0.5.3-2.fc13

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-12-01 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=645761

--- Comment #11 from Fedora Update System  
2010-12-01 15:34:00 EST ---
django-threadedcomments-0.5.3-2.el5 has been submitted as an update for Fedora
EPEL 5.
https://admin.fedoraproject.org/updates/django-threadedcomments-0.5.3-2.el5

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-11 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=645761

--- Comment #8 from Jason Tibbitts  2010-11-11 10:54:33 EST 
---
Git done (by process-git-requests).

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-10 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=645761

Domingo Becker  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from Domingo Becker  2010-11-10 
10:59:03 EST ---
New Package SCM Request
===
Package Name: django-threadedcomments
Short Description: A simple yet flexible threaded commenting system for Django
Owners: beckerde
Branches: f13 f14 el5 el6
InitialCC: glezos diegobz

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-10 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=645761

Gianluca Sforna  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Gianluca Sforna  2010-11-10 10:46:18 EST 
---
Just one minor thing. The new doc section includes INSTALL.txt; since
installation is covered by rpm we don't usually package those files.

Anyway, I think we're done here. 

Full review:

* Package name is correct
* spec file name correct
* spec file is legible and in english
* License field is correct and acceptable in Fedora
* License text is included in package

* sources match upstream
* macros used consistently

* Builds correctly in mock for F14
* rpmlint is silent

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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-08 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=645761

--- Comment #5 from Domingo Becker  2010-11-08 
09:22:46 EST ---
Removed the warnings mentioned above.

Bumped new release number.

Spec URL:
http://beckerde.fedorapeople.org/transifex/django-threadedcomments.spec
SRPM URL:
http://beckerde.fedorapeople.org/transifex/django-threadedcomments-0.5.3-2.fc13.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-04 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=645761

--- Comment #4 from Gianluca Sforna  2010-11-04 18:51:16 EDT 
---
Ok. This is mosty in a good shape. please fix these rpmlint complaints and I
think I can approve it:


django-threadedcomments.noarch: W: summary-not-capitalized C a simple yet
flexible threaded commenting system for Django
django-threadedcomments.noarch: E: description-line-too-long C
django-threadedcomments is a simple yet flexible threaded commenting system for
Django.
django-threadedcomments.noarch: W: incoherent-version-in-changelog 1.1.4-1
['0.5.3-1.fc14', '0.5.3-1']
django-threadedcomments.noarch: W: hidden-file-or-dir
/usr/share/doc/django-threadedcomments-0.5.3/docs/._pinax.txt


Always remember to run rpmlint both on source and binary packages, reviewers
are going to do it anyway and ask you to fix warning/errors

It's also easier for the reviewer if you always increment the release number at
each iteration, e.g. you sould make a 0.5.3-2, adding a changelog line.

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-04 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=645761

--- Comment #3 from Domingo Becker  2010-11-04 
09:10:58 EDT ---
(In reply to comment #1)
> Most of my comments in bug 645764 also apply here:
> 
> * Full URL is needed on Source

Fixed.

> * License should be BSD

Fixed.

> * License Text should be included in the package

Fixed.

> * Summary should be shortened (A simple yet... is enough)

Please, check it.

> * There are couple more files for the %doc section (Changelog.txt and
> contributors.txt)

Fixed.

> * Version advised in changlog does not match package

Changelog seems to be outdated.
I will communicate it upstream.

> * Did not check a mock build, but chances are Django is needed as BR:

Added as BuildRequires.

Same spec and srpm urls as in first comment.

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-11-03 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=645761

Gianluca Sforna  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 Blocks|177841(FE-NEEDSPONSOR)  |
 AssignedTo|nob...@fedoraproject.org|gia...@gmail.com
   Flag||fedora-review?

--- Comment #2 from Gianluca Sforna  2010-11-03 19:45:24 EDT 
---
Updating fields for review, looks like you have been sponsored :)

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-10-30 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=645761

Gianluca Sforna  changed:

   What|Removed |Added

 CC||gia...@gmail.com

--- Comment #1 from Gianluca Sforna  2010-10-30 17:01:03 EDT 
---
Most of my comments in bug 645764 also apply here:

* Full URL is needed on Source
* License should be BSD
* License Text should be included in the package
* Summary should be shortened (A simple yet... is enough)
* There are couple more files for the %doc section (Changelog.txt and
contributors.txt)
* Version advised in changlog does not match package
* Did not check a mock build, but chances are Django is needed as BR:

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-10-22 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=645761

Domingo Becker  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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 645761] Review Request: django-threadedcomments - a simple yet flexible threaded commenting system for Django

2010-10-22 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=645761

Thomas Spura  changed:

   What|Removed |Added

 Blocks||645752

-- 
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