[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-06-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #15 from Parag AN(पराग) panem...@gmail.com ---
any plans for el6 and epel7 builds?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-06-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #16 from Kashyap Chamarthy kcham...@redhat.com ---
Parag, no plans for EL6 or EPEL7 unless someone requests for it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-06-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-06-27 10:12:24



--- Comment #17 from Parag AN(पराग) panem...@gmail.com ---
Thanks. I will close this review then as this package is already built for
required branches.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

  Flags||needinfo?(kchamart@redhat.c
   ||om)



--- Comment #13 from Parag AN(पराग) panem...@gmail.com ---
ping is this built already?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Kashyap Chamarthy kcham...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
  Flags|needinfo?(kchamart@redhat.c |
   |om) |



--- Comment #14 from Kashyap Chamarthy kcham...@redhat.com ---
Yes it is. (That's what I meant in my previous reply. Sorry for being not
clear.)

$ koji latest-build f21 python-gerrit-view
Build Tag   Built by
   

python-gerrit-view-0.3.2-2.fc21   f21   ausil
$

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-04-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Kashyap Chamarthy kcham...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(kchamart@redhat.c |
   |om) |



--- Comment #12 from Kashyap Chamarthy kcham...@redhat.com ---
Yes, sorry for the delay in response, was away on vacation.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-04-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||kcham...@redhat.com
  Flags||needinfo?(kchamart@redhat.c
   ||om)



--- Comment #11 from Parag AN(पराग) panem...@gmail.com ---
Can this be closed?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #10 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Kashyap Chamarthy kcham...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Kashyap Chamarthy kcham...@redhat.com ---
New Package SCM Request
===
Package Name: python-gerrit-view
Short Description: A set of tools to query/view Gerrit patch reviews and their
Zuul status
Owners: kashyapc
Branches: f20 el6 epel7
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #1 from Kashyap Chamarthy kcham...@redhat.com ---
Correct SPEC:

http://kashyapc.fedorapeople.org/spec-files/python-gerrit-view.spec

(Thanks to Parag Nemade for packaging advice.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
   Assignee|nob...@fedoraproject.org|panem...@gmail.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #2 from Kashyap Chamarthy kcham...@redhat.com ---
Successful Koji scratch build:


$ koji build --scratch rawhide ../SRPMS/python-gerrit-view-0.3.0-1.fc20.src.rpm 
Uploading srpm: ../SRPMS/python-gerrit-view-0.3.0-1.fc20.src.rpm
[] 100% 00:00:01  25.15 KiB  13.17 KiB/sec
Created task: 6568385
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=6568385
Watching tasks (this may be safely interrupted)...
6568385 build (rawhide, python-gerrit-view-0.3.0-1.fc20.src.rpm): open
(arm02-builder06.arm.fedoraproject.org)
  6568386 buildArch (python-gerrit-view-0.3.0-1.fc20.src.rpm, noarch): open
(arm02-builder19.arm.fedoraproject.org)
6568385 build (rawhide, python-gerrit-view-0.3.0-1.fc20.src.rpm): open
(arm02-builder06.arm.fedoraproject.org) - closed
  0 free  1 open  1 done  0 failed
  6568386 buildArch (python-gerrit-view-0.3.0-1.fc20.src.rpm, noarch): open
(arm02-builder19.arm.fedoraproject.org) - closed
  0 free  0 open  2 done  0 failed

6568385 build (rawhide, python-gerrit-view-0.3.0-1.fc20.src.rpm) completed
successfully

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #3 from Parag AN(पराग) panem...@gmail.com ---
Review:-

+ Package build successfully in rawhide mock

+ Source verified with upstream as (sha256sum)
srpm tarball :12c8272c7caf2e9d63ef4cf4d312be1b3ead8562dc9052a6490a2548b0b8dad6
upstream tarball :
12c8272c7caf2e9d63ef4cf4d312be1b3ead8562dc9052a6490a2548b0b8dad6

+ rpmlint on generated rpms gave output
python-gerrit-view.noarch: E: explicit-lib-dependency python-gerritlib
python-gerrit-view.noarch: W: no-manual-page-for-binary czuul
python-gerrit-view.noarch: W: no-manual-page-for-binary qgerrit
python-gerrit-view.noarch: W: no-manual-page-for-binary cgerrit
python-gerrit-view.src: W: spelling-error %description -l en_US qgerrit -
Merritt
python-gerrit-view.src: W: spelling-error %description -l en_US cgerrit -
Merritt
python-gerrit-view.src: W: spelling-error %description -l en_US czuul -
Cozumel
2 packages and 0 specfiles checked; 1 errors, 6 warnings.
== Error can be ignored

+ License of this package is ASL 2.0 which is given in source header.

- Package contains duplicate files


Suggestions:
1) build.log showed messages like
warning: File listed twice:
/usr/lib/python2.7/site-packages/gerrit_view-0.3.0-py2.7.egg-info

your %files should be
%files
%doc README.rst
%{_bindir}/*
%{python2_sitelib}/gerrit_view-%{version}*

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #4 from Kashyap Chamarthy kcham...@redhat.com ---
Address the above comments. Updated SPEC, regenerated SRPM:

http://kashyapc.fedorapeople.org/spec-files/python-gerrit-view.spec
http://kashyapc.fedorapeople.org/srpms/python-gerrit-view-0.3.0-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Parag AN(पराग) panem...@gmail.com ---
Looks good now.

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #6 from Parag AN(पराग) panem...@gmail.com ---
and make sure you will have no Requires: in duplicate like this one 
Requires: python-urwid

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #7 from Kashyap Chamarthy kcham...@redhat.com ---
Thanks a lot for the review, Parag. 

Fixed duplicate entry of 'python-urwid' in Requires. Updated SPEC, and
regenerated SRPM:

http://kashyapc.fedorapeople.org/spec-files/python-gerrit-view.spec
http://kashyapc.fedorapeople.org/srpms/python-gerrit-view-0.3.0-3.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069566] Review Request: python-gerrit-view -- A set of tools to query/view Gerrit patch reviews and their Zuul status

2014-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069566



--- Comment #8 from Parag AN(पराग) panem...@gmail.com ---
Thanks for the update. The SPEC file looks corrected now.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review