[Bug 895077] Review Request: python-docopt - Pythonic argument parser, that will make you smile

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=895077

Martin Sivák msi...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #11 from Martin Sivák msi...@redhat.com ---
Package Change Request
==
Package Name: python-docopt
New Branches: el6
Owners: msivak miminar
InitialCC: 

There is a request for docopt on CentOS 6 -
https://bugzilla.redhat.com/show_bug.cgi?id=1194598

I think it is not a bad idea.

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Petr Šabata psab...@redhat.com ---
You shouldn't need File::Find, Socket, or warnings::register BRs.  These appear
to be only used by scripts which aren't tested -- find.pl and finddepth.pl for
File::Find, and syslog.pl for Socket and warnings::register.

Suggestion: Follow English rules when sorting stuff.

No review blockers, approving.

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Petr Pisar ppi...@redhat.com ---
New Package SCM Request
===
Package Name: perl-Perl4-CoreLibs
Short Description: Libraries historically supplied with Perl 4
Upstream URL: http://search.cpan.org/dist/Perl4-CoreLibs/
Owners: ppisar jplesnik psabata
Branches: f20 f21 f22
InitialCC: perl-sig

-- 
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 1194576] New: Review Request:python-docx - Lib for creating/updating docx files [FE-NEEDSPONSOR]

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194576

Bug ID: 1194576
   Summary: Review Request:python-docx - Lib for creating/updating
docx files [FE-NEEDSPONSOR]
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: kushal...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://kushal124.fedorapeople.org/python-docx.spec
SRPM URL: https://kushal124.fedorapeople.org/python-docx-0.8.2-1.fc20.src.rpm
Description: Python library for creating and updating Microsoft Word (.docx)
files.
Fedora Account System Username: kushal124

Koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=9003516

This is my first package and I need sponsor.

-- 
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 902086] Review request: Elasticsearch

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=902086



--- Comment #128 from jiri vanek jva...@redhat.com ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #127)
 Created attachment 993789 [details]
 elasticsearch.service
 
 - move Description to the right section
 - add Documentation, ProtectHome, ProtectSystem, User, LimitNOFILE

It seems to have missing
User=elasticsearch
Group=elasticsearch

I will add it to 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 1194032] Review Request: perl-Math-Int64 - Manipulate 64 bits integers in Perl

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194032

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Petr Šabata psab...@redhat.com ---
Ack, approving.

-- 
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 1194187] Review Request: libuecc - Very small Elliptic Curve Cryptography library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194187

Felix Kaechele fe...@fetzig.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Felix Kaechele fe...@fetzig.org ---
New Package SCM Request
===
Package Name: libuecc
Short Description: Very small Elliptic Curve Cryptography library
Upstream URL: http://git.universe-factory.net/libuecc
Owners: heffer
Branches: f20 f21 f22 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 1194187] Review Request: libuecc - Very small Elliptic Curve Cryptography library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194187



--- Comment #2 from Felix Kaechele fe...@fetzig.org ---
Thank your for this review. I have changed the %doc tag to %license here:

SPEC: https://heffer.fedorapeople.org/review/libuecc/libuecc.spec
SRPM: https://heffer.fedorapeople.org/review/libuecc/libuecc-5-2.fc21.src.rpm

This is the version I will commit then.

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.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 1191212] Review Request: nodejs-cli-table - Pretty unicode tables for the CLI

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191212



--- Comment #4 from Piotr Popieluch piotr1...@gmail.com ---
Hi Ralph,

npm(expresso) is provided by expresso:

$ dnf whatprovides 'npm(expresso)'
expresso-0.9.2-6.fc21.noarch

The npm download does not contain the tests, to enable the test you could fetch
the source from github. (I actually use github as Source0 by default now for
node modules as many npm downloads miss license files and tests).

Github sources:
Source0:
https://github.com/Automattic/%{barename}/archive/%{version}/%{barename}-%{version}.tar.gz
%setup -q -n %{barename}-%{version}


The tests fail:
 uncaught undefined: Error: Cannot find module 'colors/safe'

It seems that colors/save.js appeared in colors 1.0.0 for the first time. This
module will not work with nodejs-colors-0.6.2 which is in Fedora.

To have this module working in Fedora we would need to upgrade nodejs-colors or
multiversion nodejs-colors:

https://fedoraproject.org/wiki/Packaging:Node.js?rd=Node.js/Packagers#Handling_Multiple_Version_Scenarios

-- 
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 1190728] Review Request: forbidden-apis - Checks Java byte code against a list of forbidden API signatures.

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190728

zzamb...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2015-02-20 07:09:27



-- 
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 1190733] Review Request: mycila-licenses - Licenses used in Mycila projects

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190733

zzamb...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2015-02-20 07:09:56



-- 
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 902086] Review request: Elasticsearch

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=902086



--- Comment #130 from jiri vanek jva...@redhat.com ---
Also note, deadline is 24th Feb, so we should push and build today, or in
Monday as latest

-- 
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 722807] Review Request: python-bottle - Fast and simple WSGI-framework for small web-applications

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=722807

Michel Samia msa...@netsuite.com changed:

   What|Removed |Added

 CC||msa...@netsuite.com



--- Comment #4 from Michel Samia msa...@netsuite.com ---
Package Change Request
==
Package Name: python-bottle
New Branches: el6
Owners: stardust85
InitialCC: 

Hi, Rahul Sundaram agreed that I can maintain el6 branch for his package. So
please create the branch.

Thanks,
Michel

-- 
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 722807] Review Request: python-bottle - Fast and simple WSGI-framework for small web-applications

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=722807

Miroslav Suchý msu...@redhat.com changed:

   What|Removed |Added

 CC||msu...@redhat.com
  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 1194576] Review Request:python-docx - Lib for creating/updating docx files [FE-NEEDSPONSOR]

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194576

Kushal kushal...@gmail.com changed:

   What|Removed |Added

 Blocks||100 (ML-SIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=100
[Bug 100] Machine Learning SIG - review tracker
-- 
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 1194628] New: Review Request: perl-Module-Metadata-Changes - Manage a module's machine-readable Changes/CHANGES file

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194628

Bug ID: 1194628
   Summary: Review Request: perl-Module-Metadata-Changes - Manage
a module's machine-readable Changes/CHANGES file
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ppi...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ppisar.fedorapeople.org/perl-Module-Metadata-Changes/perl-Module-Metadata-Changes.spec
SRPM URL:
https://ppisar.fedorapeople.org/perl-Module-Metadata-Changes/perl-Module-Metadata-Changes-2.05-1.fc23.src.rpm
Description:
Module::Metadata::Changes is a pure Perl module. It allows you to convert
old-style Changes/CHANGES files, and to read and write Changelog.ini files.

Fedora Account System Username: ppisar

-- 
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 902086] Review request: Elasticsearch

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=902086



--- Comment #129 from jiri vanek jva...@redhat.com ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #126)
 So... this is starting to look good.
 
 Add BR and R: java-base64 = 2.3.8-7
 
 Why this:
   Requires(post):   chkconfig
   Requires(preun):  chkconfig

removed. I found in in some example..
 
 Change systemd-units to systemd in the requirements.

changed.
 
 Still valid:
   - %description needs to be extended to say what this package does.
  The new text seems to have been pasted from a website. It's a promotional
  blurb. It even ends with Learn more.
 Please add something more technical.

Improved by paragraph from wikipedia

 
 s/distributive search and analytic/distributed search and analytics/
 

done
 The daemon creates /data/elasticsearch. It should use /var/lib/elasticsearch
 instead.

used [1]
 
 The daemon should run under its own user. See
 http://fedoraproject.org/wiki/Packaging:UsersAndGroups#Dynamic_allocation
 for the setup. /var/lib/elasticsearch has to be owned by this user.
 
good catch!
done. The user is never removed?
directory also owned.

 Updated service file in attachment.

used with small sync from upstream service
(https://jvanek.fedorapeople.org/elasticsearch/review/v6/elasticsearch.service.in)


[1] This needed some rework. I had to get rid of %jpackage_script and use
elasticsearch.in to 
This script is redirecting ES work to /var/lib/elasticsearch (in case of
service)
and to XDG_CONFIG_HOME in case od regular user's run

My motivation is, that users data definitely should be separate from service's
data. If one can use global data as default, he can always symlink.
see: https://jvanek.fedorapeople.org/elasticsearch/review/v6/elasticsearch.in



SPEC:
https://jvanek.fedorapeople.org/elasticsearch/review/v6/elasticsearch.spec
SRPM:
https://jvanek.fedorapeople.org/elasticsearch/review/v6/elasticsearch-1.4.3-0.fc21.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 1194628] Review Request: perl-Module-Metadata-Changes - Manage a module's machine-readable Changes/CHANGES file

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194628

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.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 458011] Review Request: ms-sys - Create DOS/MS-compatible boot records

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=458011



--- Comment #21 from Kevin Kofler ke...@tigcc.ticalc.org ---
Actually, is ms-sys-free really Free (or even legally redistributable)? I'm not
convinced, seeing how e.g.:
http://sourceforge.net/p/ms-sys-free/code/ci/master/tree/inc/br_fat32fd_0x52.h
says:
 /* br_fat32_0x52.h
 //
 // ANI
 // substring gmbh/tw 14.9.04 
 // modified bootstrap code 0x052 to support FreeDOS
so it looks like this doesn't actually come from the FreeDOS project, but is a
derivative work of the proprietary hexdump.

-- 
You are receiving this mail because:
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 458011] Review Request: ms-sys - Create DOS/MS-compatible boot records

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=458011

Kevin Kofler ke...@tigcc.ticalc.org changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |CANTFIX



-- 
You are receiving this mail because:
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 1189460] Review Request: dock - Improved builder for Docker images

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189460

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1189460] Review Request: dock - Improved builder for Docker images

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189460



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
dock-1.1.0-1.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/dock-1.1.0-1.fc21

-- 
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 902086] Review request: Elasticsearch

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=902086

jiri vanek jva...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|POST



-- 
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 1190733] Review Request: mycila-licenses - Licenses used in Mycila projects

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190733

zzamb...@redhat.com changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190728] Review Request: forbidden-apis - Checks Java byte code against a list of forbidden API signatures.

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190728

zzamb...@redhat.com changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 895077] Review Request: python-docopt - Pythonic argument parser, that will make you smile

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=895077

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 895077] Review Request: python-docopt - Pythonic argument parser, that will make you smile

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=895077



--- Comment #12 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 1174974] Review Request: python-mox3 - Mock object framework for Python

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174974

Miro Hrončok mhron...@redhat.com changed:

   What|Removed |Added

 Whiteboard||NotReady



--- Comment #4 from Miro Hrončok mhron...@redhat.com ---
Thanks.

As it seems, I might not need this package after all (meybe). So I'm changing
this to NotReady and will eventually either close it or continue by fixing 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 722807] Review Request: python-bottle - Fast and simple WSGI-framework for small web-applications

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=722807

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 722807] Review Request: python-bottle - Fast and simple WSGI-framework for small web-applications

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=722807



--- Comment #5 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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

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 1194187] Review Request: libuecc - Very small Elliptic Curve Cryptography library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194187



--- Comment #4 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 1194187] Review Request: libuecc - Very small Elliptic Curve Cryptography library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194187

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 1194628] Review Request: perl-Module-Metadata-Changes - Manage a module's machine-readable Changes/CHANGES file

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194628



--- Comment #1 from Petr Šabata psab...@redhat.com ---
Note: All the patched files are read-only.  I assume the patching could fail,
even though it currently doesn't.  Perhaps making them writable before patching
would be a good idea.

Ad Patch1 -- according to Web Assets guidelines, Fedora webservers provide
%{_webassetdir} in `/_sysassets', not `/.sysassets'.  Please, correct the
patch.

Consider bumping the minimum required version of EU::MM to 6.76 and using
NO_PACKLIST to simplify the SPEC.

The convert.t test doesn't actually convert anything.  Maybe you could enhance
it a bit?  However, that's beyond the scope of this 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 1194225] Review Request: python-XStatic-termjs - term.js (XStatic packaging standard)

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194225

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
python-XStatic-termjs-0.0.4.2-1.el7 has been pushed to the Fedora EPEL 7
testing repository.

-- 
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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038



--- Comment #9 from Taylor Braun-Jones tay...@braun-jones.org ---
Thanks, Rex. Do you mind if I list you as a secondary owner for this 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038



--- Comment #10 from Rex Dieter rdie...@math.unl.edu ---
Please do, thanks!

-- 
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 1194911] New: Review Request: nodejs-grunt-util-args - Grunt util for extracting arguments

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194911

Bug ID: 1194911
   Summary: Review Request: nodejs-grunt-util-args - Grunt util
for extracting arguments
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-grunt-util-args.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-grunt-util-args-0.0.3-1.fc21.src.rpm
Description: Grunt util for extracting arguments
Fedora Account System Username: piotrp

-- 
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 1194916] New: Review Request: nodejs-grunt-util-process - Grunt util for processing values in an object

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194916

Bug ID: 1194916
   Summary: Review Request: nodejs-grunt-util-process - Grunt util
for processing values in an object
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-grunt-util-process.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-grunt-util-process-0.0.1-1.fc21.src.rpm
Description: Grunt util for processing values in an object
Fedora Account System Username: piotrp

-- 
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 1107800] Review Request: dl_poly - General purpose classical molecular dynamics (MD) simulation

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1107800

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
dl_poly-1.9.20140324-9.el6 has been pushed to the Fedora EPEL 6 testing
repository.

-- 
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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |



--- Comment #8 from Rex Dieter rdie...@math.unl.edu ---
sponsored, please let me know if you need anything else.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 1194923] Review Request: nodejs-ava - Simple test runner

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194923

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Blocks||1194925




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194925
[Bug 1194925] Review Request: nodejs-log-symbols - Colored symbols for
various log levels
-- 
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 1194925] Review Request: nodejs-log-symbols - Colored symbols for various log levels

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194925

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Depends On||1194923




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194923
[Bug 1194923] Review Request: nodejs-ava - Simple test runner
-- 
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 1194926] New: Review Request: nodejs-set-immediate-shim - Simple setImmediate shim

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194926

Bug ID: 1194926
   Summary: Review Request: nodejs-set-immediate-shim - Simple
setImmediate shim
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-set-immediate-shim.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-set-immediate-shim-1.0.0-1.fc21.src.rpm
Description: Simple setImmediate shim
Fedora Account System Username: piotrp

-- 
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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038

Taylor Braun-Jones tay...@braun-jones.org changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 1186503] Review Request: openclonk - Fast-paced 2d genre mix

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1186503



--- Comment #7 from MartinKG mgans...@alice.de ---
new rpm package:

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/openclonk.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/openclonk-5.5.1-3.fc21.src.rpm

%changelog
* Fri Feb 20 2015 Martin Gansser marti...@fedoraproject.org - 5.5.1-3
- corrected requirements to pkgconfig(xml2po)
- deleted Requieres from sub package
- added Requires to main package
- added html desktop files
- replaced tabs by space
- added RR xdg-utils to docs package
- added %%{?_smp_mflags} flag to html make
- added png file to devel-docs 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1194923] New: Review Request: nodejs-ava - Simple test runner

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194923

Bug ID: 1194923
   Summary: Review Request: nodejs-ava - Simple test runner
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-ava.spec
SRPM URL: https://piotrp.fedorapeople.org/nodejs-ava-0.0.4-1.fc21.src.rpm
Description: Simple test runner
Fedora Account System Username: piotrp

nodejs-ava, nodejs-log-symbols, nodejs-set-immediate-shim and nodejs-each-async
depend on each other (circular).

-- 
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 1194798] Review Request: GeoIP-GeoLite-data - Free GeoLite IP geolocation country database

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194798



--- Comment #2 from Philip Prindeville phil...@redfish-solutions.com ---
(In reply to Philip Prindeville from comment #1)
 So we just rebuild it and bump the Version field every month?

Or are we still suggesting everyone use the geoipupdate-cron* subpackages?

-- 
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 1194897] Review Request: nodejs-grunt-util-propery - Grunt util for getting and setting properties

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194897

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Blocks||1194902




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194902
[Bug 1194902] Review Request: nodejs-grunt-util-options - Grunt util for
getting options
-- 
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 1194902] New: Review Request: nodejs-grunt-util-options - Grunt util for getting options

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194902

Bug ID: 1194902
   Summary: Review Request: nodejs-grunt-util-options - Grunt util
for getting options
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-grunt-util-options.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-grunt-util-options-0.0.2-1.fc21.src.rpm
Description: Grunt util for getting options
Fedora Account System Username: piotrp

-- 
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 1194902] Review Request: nodejs-grunt-util-options - Grunt util for getting options

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194902

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Depends On||1194897




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194897
[Bug 1194897] Review Request: nodejs-grunt-util-propery - Grunt util for
getting and setting properties
-- 
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 1194931] New: Review Request: nodejs-each-async - Async concurrent iterator (async forEach)

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194931

Bug ID: 1194931
   Summary: Review Request: nodejs-each-async - Async concurrent
iterator (async forEach)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-each-async.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-each-async-1.1.1-1.fc21.src.rpm
Description: Async concurrent iterator (async forEach)
Fedora Account System Username: piotrp

-- 
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 1194897] New: Review Request: nodejs-grunt-util-propery - Grunt util for getting and setting properties

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194897

Bug ID: 1194897
   Summary: Review Request: nodejs-grunt-util-propery - Grunt util
for getting and setting properties
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-grunt-util-property.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-grunt-util-property-0.0.1-1.fc21.src.rpm
Description: Grunt util for getting and setting properties
Fedora Account System Username: piotrp

-- 
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 1194926] Review Request: nodejs-set-immediate-shim - Simple setImmediate shim

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194926

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Blocks||1194931




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194931
[Bug 1194931] Review Request: nodejs-each-async - Async concurrent iterator
(async forEach)
-- 
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 1194931] Review Request: nodejs-each-async - Async concurrent iterator (async forEach)

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194931

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Depends On||1194927, 1194926




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194926
[Bug 1194926] Review Request: nodejs-set-immediate-shim - Simple
setImmediate shim
https://bugzilla.redhat.com/show_bug.cgi?id=1194927
[Bug 1194927] Review Request: nodejs-onetime - Only call a function once
-- 
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 1194927] Review Request: nodejs-onetime - Only call a function once

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194927

Piotr Popieluch piotr1...@gmail.com changed:

   What|Removed |Added

 Blocks||1194931




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1194931
[Bug 1194931] Review Request: nodejs-each-async - Async concurrent iterator
(async forEach)
-- 
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 1183193] Review Request: ceres-solver - A non-linear least squares minimizer

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1183193

Taylor Braun-Jones tay...@braun-jones.org changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 1194927] New: Review Request: nodejs-onetime - Only call a function once

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194927

Bug ID: 1194927
   Summary: Review Request: nodejs-onetime - Only call a function
once
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-onetime.spec
SRPM URL: https://piotrp.fedorapeople.org/nodejs-onetime-1.0.0-1.fc21.src.rpm
Description: Only call a function once
Fedora Account System Username: piotrp

-- 
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 1188093] Review Request: qtile - A pure-Python tiling window manager

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188093

John Dulaney jdula...@fedoraproject.org changed:

   What|Removed |Added

 Blocks||1191544




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1191544
[Bug 1191544] qtile
-- 
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 1147743] Review Request: python-nose-cov - nose plugin for coverage reporting

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1147743

John Dulaney jdula...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2015-02-20 20:37:31



--- Comment #18 from John Dulaney jdula...@fedoraproject.org ---
Stable

-- 
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 1194950] New: Review Request: nodejs-window-size - Get the height and width of the terminal in a node.js environment

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194950

Bug ID: 1194950
   Summary: Review Request: nodejs-window-size - Get the height
and width of the terminal in a node.js environment
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://ralph.fedorapeople.org//nodejs-window-size.spec
SRPM URL:
http://ralph.fedorapeople.org//nodejs-window-size-0.1.0-1.fc21.src.rpm

Description:
Reliable way to to get the height and width of the terminal/console in a
node.js environment.

-- 
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 1194950] Review Request: nodejs-window-size - Get the height and width of the terminal in a node.js environment

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194950



--- Comment #1 from Ralph Bean rb...@redhat.com ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=9009301

-- 
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 1194576] Review Request:python-docx - Lib for creating/updating docx files [FE-NEEDSPONSOR]

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194576

Kushal kushal...@gmail.com changed:

   What|Removed |Added

 Depends On||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 1194545] Review Request: python-cached-property - A cached-property for decorating methods in Python classes.

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194545

Adam Williamson (Red Hat) awill...@redhat.com changed:

   What|Removed |Added

 CC||awill...@redhat.com



--- Comment #1 from Adam Williamson (Red Hat) awill...@redhat.com ---
you don't need to define sitearch, and it's 'good practice' to make
sitelib/sitearch definition conditional on old EL and version specific:

%if 0%{?rhel}  0%{?rhel} = 6
%{!?python2_sitelib: %global python2_sitelib %(%{__python2} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib()))}
%endif

then use python2_sitelib not python_sitelib (this is all from
https://fedoraproject.org/wiki/Packaging:Python ).

The packaging guidelines require using commit-based tarballs when using
github-generated sources, not versions:

https://fedoraproject.org/wiki/Packaging:SourceURL#Github

that can be a bit tricky to read, but yes, it really does mean you should use a
tarball derived from a commit hash even for stable releases, unless upstream
provides tarballs in some way *other* than github's generation system. The
justification is that commit hashes are less mutable and thus more predictable,
so we should take advantage of that when we can. You can still version the
package as 1.0.0-1 , but the Source0 URL should use a commit hash. See
http://pkgs.fedoraproject.org/cgit/php-google-apiclient.git/tree/php-google-apiclient.spec
for an example.

-- 
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 1194576] Review Request:python-docx - Lib for creating/updating docx files [FE-NEEDSPONSOR]

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194576



--- Comment #1 from Kushal kushal...@gmail.com ---
Updated the spec file to fix tests.
New koji build : http://koji.fedoraproject.org/koji/taskinfo?taskID=9009221

-- 
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 1189460] Review Request: dock - Improved builder for Docker images

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189460

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
dock-1.1.0-1.fc21 has been pushed to the Fedora 21 testing repository.

-- 
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 1122176] Review Request: kubernetes - Kubernetes container management

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1122176

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|kubernetes-0.9.1-0.2.git362 |kubernetes-0.9.1-0.2.git362
   |3a01.fc21   |3a01.fc20



--- Comment #36 from Fedora Update System upda...@fedoraproject.org ---
kubernetes-0.9.1-0.2.git3623a01.fc20 has been pushed to the Fedora 20 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1122176] Review Request: kubernetes - Kubernetes container management

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1122176

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|kubernetes-0.6-4.0.git993ef |kubernetes-0.9.1-0.2.git362
   |88.fc20 |3a01.fc21



--- Comment #35 from Fedora Update System upda...@fedoraproject.org ---
kubernetes-0.9.1-0.2.git3623a01.fc21 has been pushed to the Fedora 21 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
perl-Perl4-CoreLibs-0.003-1.fc21 has been pushed to the Fedora 21 testing
repository.

-- 
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 1177406] Review Request: csvcat - Efficiently concatenate CSVs or other tabular text files

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177406

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||csvcat-0.1-20141205git858ed
   ||fe.fc21
 Resolution|--- |ERRATA
Last Closed||2015-02-20 23:23:16



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
csvcat-0.1-20141205git858edfe.fc21 has been pushed to the Fedora 21 stable
repository.

-- 
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 1188093] Review Request: qtile - A pure-Python tiling window manager

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188093



--- Comment #15 from John Dulaney jdula...@fedoraproject.org ---
Beuler?

-- 
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 1194545] Review Request: python-cached-property - A cached-property for decorating methods in Python classes.

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194545



--- Comment #2 from Pete Travis m...@petetravis.com ---
Thanks for the thorough explanation, Adam. I've made the suggested changes, and
a few more minor ones after reviewing rpmlint and the python guidelines again.

Were you envisioning this package in EPEL, or is the conditional only for best
practices?

Also, I'm wavering on the package name - python-cached-property to match the
upstream project, or python-cached_property to match the module. Leaning
towards the latter - any thoughts?

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Fixed In Version||perl-Perl4-CoreLibs-0.003-1
   ||.fc23



--- Comment #4 from Petr Pisar ppi...@redhat.com ---
Thank you for the review and the repository.

-- 
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 1194058] Review Request: atomic - Tool for managing ProjectAtomic systems and containers

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194058

Colin Walters walt...@redhat.com changed:

   What|Removed |Added

 CC||walt...@redhat.com
  Flags|fedora-review?  |fedora-review+



--- Comment #7 from Colin Walters walt...@redhat.com ---
Ok, the latest fedora-review is giving me errors on RHEL7, so not pasting that.

However, we changed license upstream to LGPLv2+, can you update the spec file
too?  See https://github.com/projectatomic/atomic/issues/21

With that change, LGTM.

-- 
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 1194028] Review Request: python-jwt - JSON Web Token implementation in Python

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194028



--- Comment #3 from Ralph Bean rb...@redhat.com ---
As for the tests... I just got them included in the tarball today. 
https://github.com/jpadilla/pyjwt/pull/95  :)

-- 
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 1191212] Review Request: nodejs-cli-table - Pretty unicode tables for the CLI

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191212



--- Comment #6 from Piotr Popieluch piotr1...@gmail.com ---
just upgrading nodejs-colors will break all packages which depend on it... see
below, all nodejs-colors dependents depend on 1


for line in $(repoquery --whatrequires 'npm(colors)'); do 
   echo ${line}: 
   repoquery -R ${line} | grep colors; 
   echo
done

nodejs-dtree-0:0.0.7-1.fc21.noarch:
npm(colors)  0.7
npm(colors) = 0.6.2

nodejs-grunt-0:0.4.4-4.fc21.noarch:
npm(colors)  1
npm(colors) = 0.6

nodejs-grunt-html-validation-0:0.1.18-2.fc21.noarch:
npm(colors)
npm(colors)  0.7
npm(colors) = 0.6.0

nodejs-grunt-init-0:0.3.1-2.fc21.noarch:
npm(colors)  1
npm(colors) = 0.6

nodejs-grunt-saucelabs-0:8.3.3-1.fc21.noarch:
npm(colors)  0.7
npm(colors) = 0.6.2

nodejs-node-static-0:0.7.3-2.fc21.noarch:
npm(colors) = 0.6.0

nodejs-replace-0:0.3.0-2.fc21.noarch:
npm(colors)

nodejs-winston-0:0.7.3-2.fc21.noarch:
npm(colors)  0.7
npm(colors) = 0.6

-- 
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 1191212] Review Request: nodejs-cli-table - Pretty unicode tables for the CLI

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191212



--- Comment #5 from Ralph Bean rb...@redhat.com ---
Let's wait to upgrade nodejs-colors.  I just requested commit access to help
with updating 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 1191212] Review Request: nodejs-cli-table - Pretty unicode tables for the CLI

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191212



--- Comment #7 from Ralph Bean rb...@redhat.com ---
What a mess.  We'll need to file bugs on each of those asking the maintainers
to update in rawhide, I suppose.  (or, multi-version like you were suggesting).

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
perl-Perl4-CoreLibs-0.003-1.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/perl-Perl4-CoreLibs-0.003-1.fc21

-- 
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 1194184] Review Request: perl-Perl4-CoreLibs - Libraries historically supplied with Perl 4

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194184



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
perl-Perl4-CoreLibs-0.003-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Perl4-CoreLibs-0.003-1.fc20

-- 
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 1194058] Review Request: atomic - Tool for managing ProjectAtomic systems and containers

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194058



--- Comment #6 from Lokesh Mandvekar l...@redhat.com ---
Trouble with github urls sorry. Reverting to fedorapeople urls:

Spec URL: https://lsm5.fedorapeople.org/atomic/atomic.spec
SRPM URL:
https://lsm5.fedorapeople.org/atomic/SRPMS/atomic-0-0.3.gite4081c9.fc23.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 1194028] Review Request: python-jwt - JSON Web Token implementation in Python

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194028



--- Comment #4 from Ralph Bean rb...@redhat.com ---
New release here, addressing your concerns, except:

- The tests should show up in the next release.
- I like shipping python2 and python3 versions of the executables for the
purists out there that want to run python3-only everything.  :)  Other packages
do this (in some cases its necessary like with nosetests-3.4 and coverage,
other cases not, like python3-ansi2html).

Koji scratch build:  http://koji.fedoraproject.org/koji/taskinfo?taskID=9006048

Spec URL: http://threebean.org/rpm/SPECS/python-jwt.spec
SRPM URL: http://threebean.org/rpm/SRPMS/python-jwt-0.4.3-1.fc21.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 1191212] Review Request: nodejs-cli-table - Pretty unicode tables for the CLI

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191212



--- Comment #8 from Piotr Popieluch piotr1...@gmail.com ---
just sent a mail to nodejs sig mailing list for help on this

https://lists.fedoraproject.org/pipermail/nodejs/2015-February/000171.html

-- 
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 1193730] Review Request: apache-jena - Java framework for building Semantic Web and Linked Data applications

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1193730



--- Comment #1 from Donald Pellegrino dapellegr...@dow.com ---
Note that the .spec file should be named apache-jena.spec to prevent a
warning from rpmlint on the resulting SRPM.

-- 
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 1183193] Review Request: ceres-solver - A non-linear least squares minimizer

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1183193



--- Comment #2 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Christopher Meng from comment #1)
 I think I'd submitted this long time ago, although I didn't update it, see
 bug 1128662.

Only one review please. Which one?

-- 
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 837840] Bug for testing SCM requests

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=837840

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
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 837840] Bug for testing SCM requests

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=837840



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

-- 
You are receiving this mail because:
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 1194058] Review Request: atomic - Tool for managing ProjectAtomic systems and containers

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194058

Lokesh Mandvekar l...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Lokesh Mandvekar l...@redhat.com ---
New Package SCM Request
===
Package Name: atomic
Short Description: Tool for managing ProjectAtomic systems and containers
Upstream URL: https://github.com/projectatomic/atomic
Owners: lsm5 walters
Branches: f22 f21 f20
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 837840] Bug for testing SCM requests

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=837840

Patrick Uiterwijk puiterw...@redhat.com changed:

   What|Removed |Added

 CC||puiterw...@redhat.com
  Flags|fedora-cvs+ |fedora-cvs?



-- 
You are receiving this mail because:
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 1192969] Review Request: python-hardware - Hardware detection and classification utilities

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1192969

Antonio Trande anto.tra...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Antonio Trande anto.tra...@gmail.com ---
Package approved.

---

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- If (and only if) the source package includes the text of the license(s) in
  its own file, then that file, containing the text of the license(s) for the
  package is included in %doc.
  Note: Cannot find LICENSE in rpm(s)
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text


= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Apache (v2.0), Unknown or generated, *No copyright* Apache (v2.0).
 2 files have unknown license. Detailed output of licensecheck in
 /home/sagitter/1192969-python-hardware/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
 Note: Test run failed
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Test run failed
[x]: Packages must not store files under /srv, /opt or /usr/local
 Note: Test run failed
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.

Python:
[x]: Binary eggs must be removed in %prep
[x]: Python eggs must not download any dependencies during the build process.
[-]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-hardware , python-hardware-doc
[ ]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 

[Bug 837840] Bug for testing SCM requests

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=837840

Patrick Uiterwijk puiterw...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Patrick Uiterwijk puiterw...@redhat.com ---
New Package SCM Request
===
Package Name: stupid-package
Short Description: Some testing package
Upstream URL: https://fedoraproject.org/
Owners: puiterwijk
Branches: f22 f21 el6
InitialCC:

-- 
You are receiving this mail because:
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 1194058] Review Request: atomic - Tool for managing ProjectAtomic systems and containers

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194058



--- Comment #8 from Lokesh Mandvekar l...@redhat.com ---
fedora-review seems to work fine on fedora rawhide though. but nvm, thanks for
the review, here's the updates:

%changelog
* Fri Feb 20 2015 Lokesh Mandvekar l...@fedoraproject.org - 0-0.4.gitf7f6fa8
- Resolves: rhbz#1194058 - approved version, ready for upload
- change license to LGPLv2+

Spec URL: https://lsm5.fedorapeople.org/atomic/atomic.spec
SRPM URL:
https://lsm5.fedorapeople.org/atomic/SRPMS/atomic-0-0.4.gitf7f6fa8.fc23.src.rpm


scm request follows

-- 
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 837840] Bug for testing SCM requests

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=837840

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

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
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 1192969] Review Request: python-hardware - Hardware detection and classification utilities

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1192969

Dmitry Tantsur dtant...@redhat.com changed:

   What|Removed |Added

 CC||dtant...@redhat.com



--- Comment #2 from Dmitry Tantsur dtant...@redhat.com ---
Thanks for review, update files. I've dropped tests from packaging, as it will
introduce unnecessary dependencies, especially critical as this package will be
used on ramdisks. Please let me know if it works for you.

-- 
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 1107800] Review Request: dl_poly - General purpose classical molecular dynamics (MD) simulation

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1107800



--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
dl_poly-1.9.20140324-9.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/dl_poly-1.9.20140324-9.el6

-- 
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 665733] Review Request: Coin3 - High-level 3D visualization library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=665733



--- Comment #26 from Ralf Corsepius rc040...@freenet.de ---
(In reply to Ralf Corsepius from comment #25)
 Likely I'll revisit my packaging and publish updated versions later this
 week.
Sorry, took a little longer than a week to get back to it:

New versions:
Spec URL: http://corsepiu.fedorapeople.org/packages/Coin3.spec
SRPM URL: http://corsepiu.fedorapeople.org/packages/Coin3-3.1.5-1.fc23.src.rpm

-- 
You are receiving this mail because:
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 665733] Review Request: Coin3 - High-level 3D visualization library

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=665733



--- Comment #27 from Ralf Corsepius rc040...@freenet.de ---
(In reply to Ralf Corsepius from comment #26)
 (In reply to Ralf Corsepius from comment #25)
  Likely I'll revisit my packaging and publish updated versions later this
  week.
 Sorry, took a little longer than a week to get back to it:
 
 New versions:
 Spec URL: http://corsepiu.fedorapeople.org/packages/Coin3.spec
 SRPM URL:
 http://corsepiu.fedorapeople.org/packages/Coin3-3.1.5-1.fc23.src.rpm
Oops, typo:

Spec URL: http://corsepiu.fedorapeople.org/packages/Coin3.spec
SRPM URL: http://corsepiu.fedorapeople.org/packages/Coin3-3.1.3-5.fc23.src.rpm

-- 
You are receiving this mail because:
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 1194798] Review Request: GeoIP-GeoLite-data - Free GeoLite IP geolocation country database

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194798

Philip Prindeville phil...@redfish-solutions.com changed:

   What|Removed |Added

 CC||philipp@redfish-solutions.c
   ||om



--- Comment #1 from Philip Prindeville phil...@redfish-solutions.com ---
So we just rebuild it and bump the Version field every month?

-- 
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 1194925] New: Review Request: nodejs-log-symbols - Colored symbols for various log levels

2015-02-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194925

Bug ID: 1194925
   Summary: Review Request: nodejs-log-symbols - Colored symbols
for various log levels
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: piotr1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://piotrp.fedorapeople.org/nodejs-log-symbols.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-log-symbols-1.0.1-1.fc21.src.rpm
Description: Colored symbols for various log levels
Fedora Account System Username: piotrp


nodejs-ava, nodejs-log-symbols, nodejs-set-immediate-shim and nodejs-each-async
depend on each other (circular).

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

  1   2   >