[Bug 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-10 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-10 09:50 EST ---
python-demjson-1.3-2.fc7.1 has been pushed to the Fedora 7 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, 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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-10 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-10 09:53 EST ---
python-demjson-1.3-2.fc8.1 has been pushed to the Fedora 8 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, 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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-10 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |CURRENTRELEASE
   Fixed In Version||1.3-2.fc7.1




-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-05 05:46 EST ---
Bodhi would have auto-closed this bug after moving the F-7 and F-8 packages 
from updates-testing to updates-stable.

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-05 06:01 EST ---
But then I am not sure how long to keep open this review request as review
already passed and packages successfully built for all requested branches.
Its almost one month I can see updates submitted as per comments #9 and #10 but
not pushed to stable repository.
Feel free to open this review as anyway bodhi will going to change resolution to
Errata (IIRC) once updates gets pushed to stable repo.

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-04 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-05 00:12 EST ---
ok then we can close this review as packages built successfully and are in
respective repositories.

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 03:30 EST ---
(In reply to comment #12)
 is this pushed to rawhide?

Not sure I understand this question. Afaict there's a python-demjson package
available in f9 as well as in devel.

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-27 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-04-27 06:27 EST ---
is this pushed to 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, 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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Additional Comments From [EMAIL PROTECTED]  2008-04-09 01:16 EST ---
python-demjson-1.3-2.fc8 has been pushed to the Fedora 8 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 python-demjson'.  You can 
provide feedback for this update here: 
http://admin.fedoraproject.org/updates/F8/FEDORA-2008-2942

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-01 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-04-01 02:53 EST ---
Thanks for reviewing!

New Package CVS Request
===
Package Name: python-demjson
Short Description: Python JSON module and lint checker
Owners: thm
Branches: F-7 F-8
InitialCC: none
Cvsextras Commits: yes

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-01 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-04-01 15:25 EST ---
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, 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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-01 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-04-01 17:11 EST ---
python-demjson-1.3-2.fc8 has been submitted as an update for Fedora 8

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-04-01 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-04-01 17:14 EST ---
python-demjson-1.3-2.fc7 has been submitted as an update for Fedora 7

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-31 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-31 02:31 EST ---
will you please post updated SRPM link ?

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-31 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-31 03:01 EST ---
Spec URL: http://thm.fedorapeople.org/python-demjson.spec
SRPM URL: http://thm.fedorapeople.org/python-demjson-1.3-2.fc8.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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-31 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-04-01 00:00 EST ---
Review:
+ package builds in mock.
 Koji build =http://koji.fedoraproject.org/koji/taskinfo?taskID=539324
  http://koji.fedoraproject.org/koji/taskinfo?taskID=539326
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
5989d22248cce8641c986ed189e4e601  demjson-1.3.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Not a GUI app.

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, 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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-27 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-demjson -  Python JSON module and lint checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-27 07:38 EST ---
Not needed now following as per python packaging guidelines
%if 0%{?fedora} = 8
BuildRequires: python-setuptools-devel
%else
BuildRequires: python-setuptools
%endif

But add only
BuildRequires: python-setuptools-devel

also, drop INSTALL.txt from %doc

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-27 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-27 08:18 EST ---
(In reply to comment #1)
 Not needed now following as per python packaging guidelines
 %if 0%{?fedora} = 8
 BuildRequires: python-setuptools-devel
 %else
 BuildRequires: python-setuptools
 %endif

Isn't this necessary to successfully build for F-7?

-- 
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 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-03-27 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-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-27 19:19 EST ---
(In reply to comment #1)
 Not needed now following as per python packaging guidelines

Ok, seems indeed not to be necessary anymore.

 But add only
 BuildRequires: python-setuptools-devel

Done.

 also, drop INSTALL.txt from %doc

Done.

Spec file and srpm updated.

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