[Bug 1614044] Review Request: python-animatplot - Making animating in Matplotlib easy

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614044

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/WZFO5PGFGBYANMDTO2WYLUYMIVKZATTR/


[Bug 1614044] Review Request: python-animatplot - Making animating in Matplotlib easy

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614044



--- Comment #6 from Fedora Update System  ---
python-animatplot-0.2.2-2.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-7b310fd69c

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/BMFIXCGMQLPVIITNATYAL2FFFU7LIVMB/


[Bug 1614929] Review Request: golang-github-fatih-set - Set data structure for Go

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614929



--- Comment #2 from redhatri...@gmail.com ---
Added the SRPM link and had an unneeded BuildRequires:

Spec URL:
https://copr-be.cloud.fedoraproject.org/results/rga/compliance-masonry/fedora-28-x86_64/00786883-golang-github-fatih-set/golang-github-fatih-set.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/rga/compliance-masonry/fedora-28-x86_64/00786883-golang-github-fatih-set/golang-github-fatih-set-0.2.1-1.fc28.src.rpm

Koji Scratch Build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=28973857

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/W2BA7W3MQ5GW4YY6CAILDBE5QEMJ4AJS/


[Bug 1614044] Review Request: python-animatplot - Making animating in Matplotlib easy

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614044



--- Comment #5 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-animatplot

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/M6B6MWK5WFZE3NDXQ4DZD7A65T27RI2G/


[Bug 1583798] Review Request: termy-qt - TermySequence terminal multiplexer client

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1583798



--- Comment #5 from Eamon Walsh  ---
Okay, a new 1.1.0 release has been made upstream. Upstream is now distributing
termy-qt as a separate tarball. Upstream is also no longer distributing
termy-emoji and termy-icon-theme tarballs. These are now bundled into termy-qt.
The V8 library is also now bundled as described in comment #3.

Spec URL: https://termysequence.io/fedora/termy-qt.spec
SRPM URL: https://termysequence.io/fedora/termy-qt-1.1.0-1.fc28.src.rpm

Available in COPR:
https://copr.fedorainfracloud.org/coprs/ewalsh/termysequence/

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/WOTS5XDEXCUIPGMP7NRBBXOJA4EATPKE/


[Bug 1582983] Review Request: termy-server - TermySequence terminal multiplexer server

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1582983



--- Comment #14 from Eamon Walsh  ---
Okay, a new 1.1.0 release has been made upstream. Upstream is now distributing
termy-server as a separate tarball.

Spec URL: https://termysequence.io/fedora/termy-server.spec
SRPM URL: https://termysequence.io/fedora/termy-server-1.1.0-1.fc28.src.rpm

Available in COPR:
https://copr.fedorainfracloud.org/coprs/ewalsh/termysequence/

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/DV3MKVGDAJWQWYG6FKL5EHZFU2R2B33I/


[Bug 1614504] Review Request: classification-banner - Displays Classification Banner for a Graphical Session

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614504



--- Comment #3 from redhatri...@gmail.com ---
(In reply to Artur Iwicki from comment #2)
> >URL:https://github.com/SecurityCentral/classification-banner
> >Source0:
> >https://github.com/SecurityCentral/classification-banner/archive/%{version}/%{name}-%{version}.tar.gz
> You can shorten this by using %{URL} as part of Source0.

Thanks! I didn't even think of that. Fixed.

> 
> >%{__install} -d -m755 ...
> >%{__install} -m644 ...
> Macro forms of system executables should not be used.
> https://fedoraproject.org/wiki/Packaging:Guidelines#Macros

Thanks! I must have missed that in reading through the Guidelines. Fixed.

Updated spec, SRPM, and ran builds.

Spec URL:
https://copr-be.cloud.fedoraproject.org/results/rga/classification-banner/epel-7-x86_64/00786861-classification-banner/classification-banner.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/rga/classification-banner/epel-7-x86_64/00786861-classification-banner/classification-banner-1.6.7-1.el7.src.rpm

Koji Scratch Build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=28973094
Copr Builds:
https://copr.fedorainfracloud.org/coprs/rga/classification-banner/build/786861/

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/XUHZ2BLBIQB4PXGXFU3PARPZ7JBAXIF7/


[Bug 1614504] Review Request: classification-banner - Displays Classification Banner for a Graphical Session

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614504

Artur Iwicki  changed:

   What|Removed |Added

 CC||fed...@svgames.pl



--- Comment #2 from Artur Iwicki  ---
>URL:https://github.com/SecurityCentral/classification-banner
>Source0:
>https://github.com/SecurityCentral/classification-banner/archive/%{version}/%{name}-%{version}.tar.gz
You can shorten this by using %{URL} as part of Source0.

>%{__install} -d -m755 ...
>%{__install} -m644 ...
Macro forms of system executables should not be used.
https://fedoraproject.org/wiki/Packaging:Guidelines#Macros

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/VK2EHTWOIJQ6G6CYZNSOF3F4RRGX24PL/


[Bug 1614929] Review Request: golang-github-fatih-set - Set data structure for Go

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614929



--- Comment #1 from redhatri...@gmail.com ---
Dependency for part of my first package
https://bugzilla.redhat.com/show_bug.cgi?id=1609038

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/2NGIMMUWDDBS5QMHAXMMHWXXFVCZV3GS/


[Bug 1609038] Review Request: compliance-masonry - Security Documentation Builder

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1609038

redhatri...@gmail.com changed:

   What|Removed |Added

 Depends On||1614929




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1614929
[Bug 1614929] Review Request: golang-github-fatih-set - Set data structure
for Go
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/FRUDGM6SL4BB3CS4JTCIDF6L4BLRABUM/


[Bug 1614929] Review Request: golang-github-fatih-set - Set data structure for Go

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614929

redhatri...@gmail.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR),
   ||1609038




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=1609038
[Bug 1609038] Review Request: compliance-masonry - Security Documentation
Builder
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/ZCRHCEYOHKDIVOFYGWT5AAQ4DCQCEYCD/


[Bug 1614929] New: Review Request: golang-github-fatih-set - Set data structure for Go

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614929

Bug ID: 1614929
   Summary: Review Request: golang-github-fatih-set - Set data
structure for Go
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: redhatri...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/rga/compliance-masonry/fedora-rawhide-x86_64/00786826-golang-github-fatih-set/golang-github-fatih-set.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/rga/compliance-masonry/fedora-rawhide-x86_64/00786826-golang-github-fatih-set/golang-github-fatih-set.spec

Koji Scratch Build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=28971307

Description: 
Set is a basic and simple, hash-based, Set data structure implementation in Go
(Golang).
Fedora Account System Username: rga

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/UH5OYYGW3N2TAEORSH5MN44MBITDDTTZ/


[Bug 1607675] Review Request: castget - A command-line podcast downloader

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1607675



--- Comment #9 from Fedora Update System  ---
castget-1.2.4-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If
problems still persist, please make note of it in this bug report.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/FHU6IDX7Z2DM4Q4B4UVZKN5EDMP3MUCS/


[Bug 1592136] Review Request: perl-GraphViz2 - GraphViz2 Perl module

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1592136

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2018-08-10 05:17:32



--- Comment #5 from Emmanuel Seyman  ---
Finally, this will be a rawhide-only package.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/HXX32RDY36JXKXRNEYEPSAVCZFJALHGZ/


[Bug 1593893] Review Request: perl-DBIx-Class-Visualizer - Visualize a DBIx::Class schema

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1593893
Bug 1593893 depends on bug 1592136, which changed state.

Bug 1592136 Summary: Review Request: perl-GraphViz2 - GraphViz2 Perl module
https://bugzilla.redhat.com/show_bug.cgi?id=1592136

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/M57JOONLCTQHSKMQ2PUV2DYU5DGNFH7L/


[Bug 1614044] Review Request: python-animatplot - Making animating in Matplotlib easy

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614044

José Matos  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|jama...@fc.up.pt
  Flags||fedora-review+



--- Comment #4 from José Matos  ---
(In reply to Elliott Sales de Andrade from comment #3)
> (In reply to José Matos from comment #2)
> > 1) why do you use as the URL the readthedocs and not the github repo?
> 
> It just seemed more interesting than the other. I suppose it doesn't really
> matter that much.

I was curious. After all the readthedocs refers the github source.

> > 2) fedora-review identifies a missing %license...
> 
> Yes, that was mistakenly put in %doc.
> 
> Spec URL: https://qulogic.fedorapeople.org/python-animatplot.spec
> SRPM URL:
> https://qulogic.fedorapeople.org/python-animatplot-0.2.2-2.fc28.src.rpm

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/GVV6EWC2WQCQPSNXQLYF6YYMQ4HXMSRE/


[Bug 1614044] Review Request: python-animatplot - Making animating in Matplotlib easy

2018-08-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1614044



--- Comment #3 from Elliott Sales de Andrade  ---
(In reply to José Matos from comment #2)
> 1) why do you use as the URL the readthedocs and not the github repo?

It just seemed more interesting than the other. I suppose it doesn't really
matter that much.

> 2) fedora-review identifies a missing %license...

Yes, that was mistakenly put in %doc.

Spec URL: https://qulogic.fedorapeople.org/python-animatplot.spec
SRPM URL:
https://qulogic.fedorapeople.org/python-animatplot-0.2.2-2.fc28.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/PX24G77FP3UWKHQJUHA25IWUSMBYHP2Y/