[Bug 1103554] Review Request: CppCMS - a Free High Performance Web Development Framework

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

Ralf Corsepius rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #4 from Ralf Corsepius rc040...@freenet.de ---
This package has lot of issues. I do not want to sound too negative, but I fear
it'll be a long and tiresome process to get this package into Fedora.

Let start with pointing out a few selected issues.

1. Packaging:
Please properly split the package into main and *-devel subpackages:
- *.so.* belong into *-devel unless they are dlopened by name.
- man3/ are devel doc and therefore belong into *-devel.
- *.a are not supposed to be shipped in Fedora at all.
- Please be more selective in %files sections. The way you have written them is
very sensitive to accidentially picking up files a package is not supposed to
contain (cf. *devel vs. main).

2. Contents:
- Most of the man3/* manpages are junk and are likely to conflict with other
packages.
- The *-doc package contains a lot of files which are not supposed to be
shipped.

3. Bundled libraries:
This package bundles quite a number of sources from other origins (boost,
json2*.js, sha1, etc.). Please modify the package to use external versions
whenever possible.

-- 
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 1103545] Review Request: granite - GTK extensions for the elementary desktop

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

Christopher Meng i...@cicku.me changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||i...@cicku.me
   Assignee|nob...@fedoraproject.org|i...@cicku.me
  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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
  Flags||fedora-cvs?



--- Comment #6 from Jens Petersen peter...@redhat.com ---
Thanks for reviewing!


New Package SCM Request
===
Package Name: ghc-io-streams
Short Description: Simple composable stream I/O
Upstream URL: http://hackage.haskell.org/package/io-streams
Owners: petersen codeblock
Branches: f20 f19 epel7 el6
InitialCC: haskell-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 1064352] Review Request: rubygem-deep_merge - Merges deep hashes in ruby.

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



--- Comment #6 from Steve Traylen steve.tray...@cern.ch ---
Thanks for the pointer:

http://cern.ch/straylen/rpms/rubygem-deep_merge/rubygem-deep_merge-1.0.1-3.fc20.src.rpm

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

[Bug 1093095] Review Request: rubygem-sys-proctable - An interface for providing process table information

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



--- Comment #1 from Steve Traylen steve.tray...@cern.ch ---
http://cern.ch/straylen/rpms/rubygem-sys-proctable/rubygem-sys-proctable-0.9.4_universal_linux-2.fc20.src.rpm

update for latest ruby guidelines.

-- 
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 1100926] Review Request: php-guzzlehttp-streams - Provides a simple abstraction over streams of data

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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||fed...@famillecollet.com
   Assignee|nob...@fedoraproject.org|fed...@famillecollet.com



--- Comment #1 from Remi Collet fed...@famillecollet.com ---
Created attachment 901382
  -- https://bugzilla.redhat.com/attachment.cgi?id=901382action=edit
phpci.log

-- 
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 1100926] Review Request: php-guzzlehttp-streams - Provides a simple abstraction over streams of data

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



--- Comment #2 from Remi Collet fed...@famillecollet.com ---
Created attachment 901383
  -- https://bugzilla.redhat.com/attachment.cgi?id=901383action=edit
review.txt

-- 
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 1092431] Review Request: nanodesigner - 3D molecular editor and simulator

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



--- Comment #4 from Florian Tani floriant...@gmail.com ---
New URLs 

Spec URL: http://openlabs.cc/nanodesigner/Nanodesigner.spec
SRPM URL: http://openlabs.cc/nanodesigner/Nanodesigner-O.1.2-1.fc18.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 1100926] Review Request: php-guzzlehttp-streams - Provides a simple abstraction over streams of data

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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #3 from Remi Collet fed...@famillecollet.com ---
[x]: Package is named according to the Package Naming Guidelines.
  Match php-vendor-lib

[x]: URL: https://github.com/guzzle/http
  Could use http://guzzlephp.org/ (from composer.json homepage)
  Both are fine for me.

[x]: Rpmlint is run on all installed packages.
  Could drop the anchor from the link.



No blockers.

== APPROVED ==

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

[Bug 1102723] Review Request: kf5-kdnssd - KDE Frameworks 5 Tier 1 integration module for DNS-SD services (Zeroconf)

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

Dan Vrátil dvra...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-06-02 05:05:36



-- 
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 1087956] Review Request: ghc-wl-pprint - The Wadler/Leijen Pretty Printer

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



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
ghc-wl-pprint-1.1-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/ghc-wl-pprint-1.1-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 1087956] Review Request: ghc-wl-pprint - The Wadler/Leijen Pretty Printer

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



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
ghc-wl-pprint-1.1-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/ghc-wl-pprint-1.1-1.fc19

-- 
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 1087956] Review Request: ghc-wl-pprint - The Wadler/Leijen Pretty Printer

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

Fedora Update System upda...@fedoraproject.org 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1103034] Review Request: python-django-stopforumspam - Django middleware for blocking IPs listed in stopforumspam.com

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

Fedora Update System upda...@fedoraproject.org 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1103034] Review Request: python-django-stopforumspam - Django middleware for blocking IPs listed in stopforumspam.com

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



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
python-django-stopforumspam-1.4.1-1.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/python-django-stopforumspam-1.4.1-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 1075605] Review Request: ghc-distributive - Haskell 98 Distributive functors -- Dual to Traversable

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



--- Comment #2 from Jens Petersen peter...@redhat.com ---
(In reply to Jens Petersen from comment #1)
 I can update this I am happy to review. :)

Sorry: If you can update this, I am happy to 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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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

Robert Kuska rku...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Robert Kuska rku...@redhat.com ---
New Package SCM Request
===
Package Name: python-bintrees
Short Description: Binary-, RedBlack- and AVL-Trees in Python and Cython
Upstream URL: http://bitbucket.org/mozman/bintrees
Owners: rkuska
Branches: 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 542416] Review Request: yagtd - Yet Another Getting Things Done

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



--- Comment #20 from Matthias Runge mru...@redhat.com ---
any progress here?

-- 
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 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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



--- Comment #3 from Mat Booth mat.bo...@redhat.com ---
Thanks for the review.

(In reply to Pete MacKinnon from comment #2)
 MUST:
 [!]: Package must own all directories that it creates.
  Note: Directories without known owners: /usr/share/java/antlr32


Hmm, I'm not sure why this is reported, the java subpackage should own this. In
the spec file, you should see this:

%files java -f .mfiles-java
%doc tool/LICENSE.txt
%dir %{_datadir}/java/%{name}

The %dir line should expand to /usr/share/java/antlr32

Do you have any thoughts about what is wrong here?


 SHOULD:
 [!]: Patches link to upstream bugs/comments/lists or are otherwise justified.
 It would be nice to have a link to the JIRA where Java 8 patch originated. 


Good point, I will add a link to the upstream commit.

-- 
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 923959] Review Request: JOrtho - A spell checker for Java

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

Christian von Behren vonbehre...@gmail.com changed:

   What|Removed |Added

 CC||vonbehre...@gmail.com



--- Comment #5 from Christian von Behren vonbehre...@gmail.com ---
I hope someone will track this down for me...

The updated package version is missing some dictionaries!

/usr/share/java/jortho.jar references SIX languages in dictionaries.cnf:

extension=.ortho
languages=de,en,it,fr,es,ru


The dictionaries shipped with this package is just ONE: dictionary_en.ortho!

When I open Freemind I get an error:

java.io.FileNotFoundException: JAR entry dictionary_de.ortho not found in
/usr/share/java/jortho.jar


Any ideas how to fix that?

-- 
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 1103554] Review Request: CppCMS - a Free High Performance Web Development Framework

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

Björn besser82 Esser bjoern.es...@gmail.com changed:

   What|Removed |Added

 CC||bjoern.es...@gmail.com
 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 1103420] Review Request: autowrap - Generates Python Extension modules from [Cython] PXD files

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



--- Comment #5 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Björn besser82 Esser from comment #3)
 Package Review
 ==
 
 Legend:
 [x] = Pass
 [!] = Fail
 [-] = Not applicable
 [?] = Not evaluated
 
 
 [!]: License field in the package spec file matches the actual license.
  Note: Checking patched sources after %prep for licenses. Licenses found:
  Unknown or generated. 47 files have unknown license. Detailed output
 of
  licensecheck in /home/besser82/shared/fedora/review/1103420-python-
  autowrap/licensecheck.txt
 
  --- please ask upstream to prepend the actual license to the
   source-files.
 
   did you even read the LICENSE-file *BEFORE* packaging?!?
 
   License: GPLv3+ (spec)   vs.   License: BSD (LICENSE-file in src)
 
   Please refer to the process mentioned in:
  
 https://fedoraproject.org/wiki/Packaging/
 LicensingGuidelines#License_Clarification


I had not checked License file; it was a my mistake.
https://github.com/uweschmitt/autowrap/issues/6

SPEC: http://sagitter.fedorapeople.org/python-autowrap/autowrap.spec
SRPM:
http://sagitter.fedorapeople.org/python-autowrap/autowrap-0.4.0-3.20140321git26e901.fc20.src.rpm

- Removed conditional macros for EPEL
- Fixed Source0

-- 
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 1103420] Review Request: autowrap - Generates Python Extension modules from [Cython] PXD files

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



--- Comment #6 from Antonio Trande anto.tra...@gmail.com ---
Koji build in rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6917582

-- 
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 173040] Review Request: rlog - Runtime Logging for C++

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

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 173040] Review Request: rlog - Runtime Logging for C++

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



--- Comment #24 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 1054394] Review Request: python-django-admin-bootstrapped - Bootstrap support for Django projects

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

Matthias Runge mru...@redhat.com changed:

   What|Removed |Added

   Assignee|mru...@redhat.com   |nob...@fedoraproject.org



-- 
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 229417] Review Request: python-qpid - qpid's python implementation

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



--- Comment #17 from Jon Ciesla limburg...@gmail.com ---
Added you to commiters but am unable to unorphan, investigating.

-- 
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 229417] Review Request: python-qpid - qpid's python implementation

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

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

   What|Removed |Added

  Flags|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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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



--- Comment #7 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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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

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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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

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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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



--- Comment #6 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 1100899] Review Request: ratools - Framework for IPv6 Router Advertisements

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

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 1103543] Review Request: python-pyramid-chameleon - Bindings for the Chameleon templating system in the Pyramid web framework

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



--- Comment #6 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 1103543] Review Request: python-pyramid-chameleon - Bindings for the Chameleon templating system in the Pyramid web framework

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

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 1100899] Review Request: ratools - Framework for IPv6 Router Advertisements

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



--- Comment #17 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 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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

Pete MacKinnon pmack...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Pete MacKinnon pmack...@redhat.com ---
$ rpm -qf /usr/share/java/antlr32
antlr32-java-3.2-1.fc20.noarch

Hmm, false positive from fedora-review 0.5.1 for some reason. Sorry about
that...

APPROVED

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

[Bug 1102941] Review Request: php-pclzip - Compression and extraction functions for Zip formatted archives

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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||fed...@famillecollet.com
   Assignee|nob...@fedoraproject.org|fed...@famillecollet.com



--- Comment #1 from Remi Collet fed...@famillecollet.com ---
Created attachment 901455
  -- https://bugzilla.redhat.com/attachment.cgi?id=901455action=edit
phpci.log

-- 
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 1102941] Review Request: php-pclzip - Compression and extraction functions for Zip formatted archives

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



--- Comment #2 from Remi Collet fed...@famillecollet.com ---
Created attachment 901456
  -- https://bugzilla.redhat.com/attachment.cgi?id=901456action=edit
review.txt

-- 
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 1102941] Review Request: php-pclzip - Compression and extraction functions for Zip formatted archives

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

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #3 from Remi Collet fed...@famillecollet.com ---
[!]: Dist tag is present (not strictly required in GL).
  Use %{?dist} instead of %{dist}
[!] rpmlint
  php-pclzip.noarch: E: incorrect-fsf-address
/usr/share/doc/php-pclzip/gnu-lgpl.txt
  Should be reported to upstream
[x]: Latest version is packaged.
  Ok, but this version was released 5 years ago
  This project seems dead.
  You should understand you will probably have to maintain it in case of bug



Please fix %{?dist} on import

No blocker

== APPROVED ==

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

[Bug 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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

Mat Booth mat.bo...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Mat Booth mat.bo...@redhat.com ---
New Package SCM Request
===
Package Name: antlr32
Short Description: ANother Tool for Language Recognition
Upstream URL: http://www.antlr3.org/
Owners: mbooth
Branches: 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 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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



--- Comment #6 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 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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

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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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

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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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

Jan Grulich jgrul...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|jgrul...@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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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



--- Comment #2 from Petr Šabata psab...@redhat.com ---
You're missing the usual perl BR.

Also, Time::HiRes isn't needed for anything.  It's only mentioned in POD.

The rest looks okay to me.

-- 
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 692733] Review Request: openvas-cli - Command-line tool to drive OpenVAS Manager

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

Michal Ambroz re...@seznam.cz changed:

   What|Removed |Added

  Flags|needinfo?(re...@seznam.cz)  |



--- Comment #10 from Michal Ambroz re...@seznam.cz ---
Thanks Fabian,
here is update to OpenVAS7 openvas-cli version 1.3.0

SPEC: http://people.fedoraproject.org/~rebus/SPECS/openvas-cli.spec
SRPM:
http://people.fedoraproject.org/~rebus/SRPMS/openvas-cli-1.3.0-1.fc20.src.rpm

Best regards
Michal Ambroz

-- 
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 1098778] Review Request: perl-Redis - Perl binding for Redis database

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

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 1103543] Review Request: python-pyramid-chameleon - Bindings for the Chameleon templating system in the Pyramid web framework

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-06-02 11:03:52



--- Comment #7 from Mathieu Bridon boche...@fedoraproject.org ---
Thanks for the review Kevin, and thank you for the git Jon!

Package built, closing.

-- 
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 1103226] Review Request: antlr32 - ANother Tool for Language Recognition

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

Mat Booth mat.bo...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||antlr32-3.2-3.fc21
 Resolution|--- |NEXTRELEASE
Last Closed||2014-06-02 11:28:22



--- Comment #7 from Mat Booth mat.bo...@redhat.com ---
Successfully built: http://koji.fedoraproject.org/koji/buildinfo?buildID=520948

-- 
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 1092431] Review Request: nanodesigner - 3D molecular editor and simulator

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



--- Comment #5 from Florian Tani floriant...@gmail.com ---
Thank you for your feedback. Actually i have to packaged as Java application so
i will revised in depth .spec file . i will test,build in Mock and Koji before
i update the package.

  (In reply to Michael Schwendt from comment #3)
  http://openlabs.cc/nanodesigner/Nanodesigner.spec
 
 That spec file is very unusual, because it builds a package that only
 includes two %doc files:
 
   | %files
   | %doc README COPYING
 
 Have you done a test-build of this packge yet? With Mock or in koji (by
 following the Join process:
 https://fedoraproject.org/wiki/Join_the_package_collection_maintainers ).
 
 Also consider pointing the fedora-review tool at this ticket. Run
 fedora-review -b 1092431 and let it perform many helpful checks on your
 package(s). Keep the Spec URL: and SRPM URL: lines accurate, since the
 fedora-review tool downloads the files from those URLs.

-- 
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 1103420] Review Request: autowrap - Generates Python Extension modules from [Cython] PXD files

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



--- Comment #7 from Antonio Trande anto.tra...@gmail.com ---
New commit; fixed license headers.

SPEC: http://sagitter.fedorapeople.org/python-autowrap/autowrap.spec
SRPM:
http://sagitter.fedorapeople.org/python-autowrap/autowrap-0.4.0-4.20140602gitebde9f.fc20.src.rpm

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

[Bug 877096] Review Request: perl-Fsdb - A set of commands for manipulating flat-text databases from the shell

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

John Heidemann jo...@isi.edu changed:

   What|Removed |Added

  Flags||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 1055366] Review Request: snapraid - Disk array backup for many large rarely-changed files

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

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

   What|Removed |Added

   Fixed In Version|snapraid-6.1-3.fc20 |snapraid-6.1-3.el6



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
snapraid-6.1-3.el6 has been pushed to the Fedora EPEL 6 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 1078364] Review Request: js-sizzle - A pure-JavaScript CSS selector engine

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

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  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 1078364] Review Request: js-sizzle - A pure-JavaScript CSS selector engine

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



--- Comment #7 from Tom Hughes t...@compton.nu ---
Package Review
==

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



= 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. No licenses
 found. Please check the source files for licenses manually.
[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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 4 files.
[!]: 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]: 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.
[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.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[-]: 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).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[?]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed files.
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in GL).
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless justified.

= EXTRA items =

Generic:
[x]: Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: Spec file according to URL is the same as in SRPM.


Rpmlint
---
Checking: js-sizzle-1.10.19-2.fc21.noarch.rpm
  

[Bug 1103852] New: Review Request: drupal7-site_map - This module provides a site map that gives visitors an overview of your site

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

Bug ID: 1103852
   Summary: Review Request: drupal7-site_map - This module
provides a site map that gives visitors an overview of
your site
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: peter.bo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://asrob.fedorapeople.org/SOURCES/drupal7-site_map.spec
SRPM URL:
http://asrob.fedorapeople.org/SOURCES/drupal7-site_map-1.2-1.fc21.src.rpm
Description: This module provides a site map that gives visitors an overview of
your site.
Fedora Account System Username: asrob

-- 
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 1078364] Review Request: js-sizzle - A pure-JavaScript CSS selector engine

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



--- Comment #8 from Tom Hughes t...@compton.nu ---
I haven't managed to run the tests, but other than that this generally looks
good.

I think the only problem I can see is that the source URL does not comply with
the rules at https://fedoraproject.org/wiki/Packaging:SourceURL#Github for
packaging from github.

-- 
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 928841] Review Request: drupal7-theme-zen - Zen is a powerful, yet simple, HTML5 starting theme

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

Peter Borsa peter.bo...@gmail.com changed:

   What|Removed |Added

 CC||peter.bo...@gmail.com



--- Comment #23 from Peter Borsa peter.bo...@gmail.com ---
*** Bug 1103453 has been marked as a duplicate of this bug. ***

-- 
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 1103453] Review Request: drupal7-zen - Zen is a powerful, yet simple, HTML5 starting theme with a responsive, mobile-first grid design

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

Peter Borsa peter.bo...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2014-06-02 14:40:06



--- Comment #1 from Peter Borsa peter.bo...@gmail.com ---


*** This bug has been marked as a duplicate of bug 928841 ***

-- 
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 908088] Review Request: ascend - ASCEND modelling environment

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

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #24 from Rex Dieter rdie...@math.unl.edu ---
OK, looks like we have a winner, APPROVED.

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

[Bug 1103875] New: Review Request: openstack-tempest - set of integration tests to be run against a live OpenStack cluster

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

Bug ID: 1103875
   Summary: Review Request: openstack-tempest - set of integration
tests to be run against a live OpenStack cluster
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: slina...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://file.rdu.redhat.com/~slinaber/openstack-tempest.spec
SRPM URL:
http://file.rdu.redhat.com/~slinaber/openstack-tempest-0.0.1-ee7ce47.fc20.src.rpm
Description: This is a set of integration tests to be run against a live
OpenStack cluster. Tempest has batteries of tests for OpenStack API validation,
Scenarios, and other specific tests useful in validating an OpenStack
deployment.
Fedora Account System Username: slinabery

-- 
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 1103875] Review Request: openstack-tempest - set of integration tests to be run against a live OpenStack cluster

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



--- Comment #1 from Steve Linabery slina...@redhat.com ---
scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6919027

-- 
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 908088] Review Request: ascend - ASCEND modelling environment

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

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

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #25 from Antonio Trande anto.tra...@gmail.com ---
Thank you very much Rex.

New Package SCM Request
===
Package Name: ascend
Short Description:  ASCEND modelling environment
Upstream URL: http://ascend4.org/
Owners: sagitter
Branches: f20

-- 
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 928609] Review Request: xpra - screen for X

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

Nicolas Chauvet (kwizart) kwiz...@gmail.com changed:

   What|Removed |Added

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



--- Comment #40 from Nicolas Chauvet (kwizart) kwiz...@gmail.com ---
Ok, sorry but I was not the right person for review swap.

-- 
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 1030755] Review Request: python-livereload - LiveReload server in Python

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

Pete Travis m...@petetravis.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW



--- Comment #3 from Pete Travis m...@petetravis.com ---
Releasing, feel free to ping me if you want to continue 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 1103897] New: Review Request: python-rxjson - JSON RX Schema validation tool

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

Bug ID: 1103897
   Summary: Review Request: python-rxjson - JSON RX Schema
validation tool
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: boche...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://bochecha.fedorapeople.org/packages/python-rxjson.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/python-rxjson-0.2-1.fc20.src.rpm

Description:
JSON RX Schema validation tool.

Fedora Account System Username: bochecha

-- 
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 1103543] Review Request: python-pyramid-chameleon - Bindings for the Chameleon templating system in the Pyramid web framework

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Blocks||1103898




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103898
[Bug 1103898] Package Bodhi 2
-- 
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 1103897] Review Request: python-rxjson - JSON RX Schema validation tool

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Blocks||1103898




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103898
[Bug 1103898] Package Bodhi 2
-- 
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 1082656] Review Request: python-diff-cover - Automatically find diff lines that need test coverage

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



--- Comment #2 from David Shea ds...@redhat.com ---
Updated to latest upstream version.

Spec URL:
http://dshea.fedorapeople.org/python-diff-cover/v1/python-diff-cover.spec
SRPM URL:
http://dshea.fedorapeople.org/python-diff-cover/v1/python-diff-cover-0.4.1-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 1102858] Review Request: python-repoze-sphinx-autointerface - Auto-generate Sphinx API docs from Zope interfaces

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



--- Comment #2 from Jerry James loganje...@gmail.com ---
Thank you very much for the review, Björn.

(In reply to Björn besser82 Esser from comment #1)
  --- It might be nice to ask upstream for adding copyright / license
   headers atop the source-files

I will make the attempt.

 = Additional Information =
 
 * Please conditionalize the build of the py3-pkg, so this can be easily
   ported to other releases (EPEL7), which don't offer support for py3, yet.

Done.

 * The %%clean-section is unneccessary and will override the default %%clean
   in rpm-macros.  Please drop it.

This is not true.  With the %clean section as is, rpmbuild --clean removes both
repoze.sphinx.autointerface-0.7.1 and
python3-python-repoze-sphinx-autointerface-0.7.1-1.fc21 from BUILD.  If I
remove the %clean section, then rpmbuild --clean removes the former but leaves
the latter behind.

 * For a quick fix-up, I attached you a patch.

Thank you, I appreciate the patch.  I have modified your suggested patch a
little due to differences in taste.  New URLs:

Spec URL:
http://jjames.fedorapeople.org/python-repoze-sphinx-autointerface/python-repoze-sphinx-autointerface.spec
SRPM URL:
http://jjames.fedorapeople.org/python-repoze-sphinx-autointerface/python-repoze-sphinx-autointerface-0.7.1-2.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 1103917] New: Review Request: python-cornice - Define Web Services in Pyramid

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

Bug ID: 1103917
   Summary: Review Request: python-cornice - Define Web Services
in Pyramid
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: boche...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://bochecha.fedorapeople.org/packages/python-cornice.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/python-cornice-0.16.2-1.fc20.src.rpm

Description:
Define Web Services in Pyramid

Fedora Account System Username: bochecha

-- 
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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Blocks||1103898




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103898
[Bug 1103898] Package Bodhi 2
-- 
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 1103897] Review Request: python-rxjson - JSON RX Schema validation tool

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Blocks|1103898 |1103917




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103898
[Bug 1103898] Package Bodhi 2
https://bugzilla.redhat.com/show_bug.cgi?id=1103917
[Bug 1103917] Review Request: python-cornice - Define Web Services in
Pyramid
-- 
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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Depends On||1103897




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103897
[Bug 1103897] Review Request: python-rxjson - JSON RX Schema validation
tool
-- 
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 1103897] Review Request: python-rxjson - JSON RX Schema validation tool

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

Haïkel Guémar karlthe...@gmail.com changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
   Assignee|nob...@fedoraproject.org|karlthe...@gmail.com
  Flags||fedora-review?



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

[Bug 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Haïkel Guémar karlthe...@gmail.com changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
   Assignee|nob...@fedoraproject.org|karlthe...@gmail.com



-- 
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 1102858] Review Request: python-repoze-sphinx-autointerface - Auto-generate Sphinx API docs from Zope interfaces

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



--- Comment #3 from Michael Schwendt bugs.mich...@gmx.net ---
 the %clean section

Not the full story.

The default %clean section removes the BUILDROOT. It doesn't remove the BUILD
directory.

rpmbuild --clean removes the BUILD dir in addition to %clean.

If you override %clean the way you do so far, you don't remove the %buildroot.
But if you added rm -rf %buildroot, that would result in different behaviour
compared with what %clean does by default.


I've filed  https://fedorahosted.org/fpc/ticket/435  about this.

-- 
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 1103897] Review Request: python-rxjson - JSON RX Schema validation tool

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

Haïkel Guémar karlthe...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Haïkel Guémar karlthe...@gmail.com ---
I hereby approve of this package into Fedora Packages Collection.
Formal review here:

Package Review
==

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



= 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]: 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.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 Unknown or generated. 4 files have unknown license. Detailed output of
 licensecheck in /home/haikel/1103897-python-rxjson/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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[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: No rpmlint messages.
[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.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: 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
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[x]: 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).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-rxjson
[x]: 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
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests 

[Bug 1101043] Review Request: ming - A library for generating Macromedia Flash files

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



--- Comment #2 from Dominik 'Rathann' Mierzejewski domi...@greysector.net ---
Thanks for the thorough review, Ralf. Here's an updated package.

Spec URL: http://rathann.fedorapeople.org/review/ming/ming.spec
SRPM URL: http://rathann.fedorapeople.org/review/ming/ming-0.4.5-2.fc20.src.rpm

* Tue May 27 2014 Dominik Mierzejewski r...@greysector.net - 0.4.5-2
- fix ming-config to be multilib-compatible
- enable testsuite
- disable silent rules in configure call
- drop defattr
- build perl, php, python and tcl bindings
- don't change ChangeLog timestamp

-- 
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 1103927] New: Review Request: imaptest - A generic IMAP server compliancy tester

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

Bug ID: 1103927
   Summary: Review Request: imaptest - A generic IMAP server
compliancy tester
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: redhat-bugzi...@linuxnetz.de
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://labs.linuxnetz.de/bugzilla/imaptest.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/imaptest-20140528-1.src.rpm
Description:
ImapTest is a generic IMAP server compliancy tester that works with all IMAP
servers. It supports stress testing with state tracking, scripted testing and
benchmarking. When stress testing with state tracking ImapTest sends random
commands to the server and verifies that server's output looks correct. Using
the scripted testing ImapTest runs a list of predefined scripted tests and
verifies that server returns expected output.

Examples and details are provided online at: http://www.imapwiki.org/ImapTest

-- 
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 1082825] Review Request: mozilla-lightbeam - An add-on for visualizing HTTP requests between websites in real time

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



--- Comment #1 from Dominik 'Rathann' Mierzejewski domi...@greysector.net ---
Spec URL:
http://rathann.fedorapeople.org/review/mozilla-lightbeam/mozilla-lightbeam.spec
SRPM URL:
http://rathann.fedorapeople.org/review/mozilla-lightbeam/mozilla-lightbeam-1.0.10.1-1.fc20.src.rpm

* Mon Jun 02 2014 Dominik 'Rathann' Mierzejewski r...@greysector.net -
1.0.10.1-1
- updated to 1.0.10.1

-- 
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 1087956] Review Request: ghc-wl-pprint - The Wadler/Leijen Pretty Printer

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
ghc-wl-pprint-1.1-1.fc20 has been pushed to the Fedora 20 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 1070510] Review Request: getdns - Modern asynchronous API to the DNS

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

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||getdns-0.1.1-1.fc20
 Resolution|--- |ERRATA
Last Closed||2014-06-02 18:56:58



--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
getdns-0.1.1-1.fc20 has been pushed to the Fedora 20 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 1103034] Review Request: python-django-stopforumspam - Django middleware for blocking IPs listed in stopforumspam.com

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
python-django-stopforumspam-1.4.1-1.fc20 has been pushed to the Fedora 20
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 1086444] Review Request: ghc-optparse-applicative - Utilities and combinators for parsing command line options

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

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

   What|Removed |Added

   Fixed In Version|ghc-optparse-applicative-0. |ghc-optparse-applicative-0.
   |8.0.1-2.fc20|8.0.1-2.fc19



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
ghc-optparse-applicative-0.8.0.1-2.fc19 has been pushed to the Fedora 19 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 1103897] Review Request: python-rxjson - JSON RX Schema validation tool

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Mathieu Bridon boche...@fedoraproject.org ---
Thanks Haikel!

New Package SCM Request
===
Package Name: python-rxjson
Short Description: JSON RX Schema validation tool
Upstream URL: https://pypi.python.org/pypi/rxjson/
Owners: bochecha lmacken
Branches: devel el6 el7
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 1064352] Review Request: rubygem-deep_merge - Merges deep hashes in ruby.

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



--- Comment #7 from Ken Dreyer ktdre...@ktdreyer.com ---
Package Review
==

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


Issues:
===
- Package must contain a full copy of the MIT license.  The good news is that
  upstream has a LICENSE file in Git. The bad news is that they are not
  shipping the file in the gem from rubygems.org. I've filed an issue for this
  at https://github.com/danielsdeleo/deep_merge/pull/14, and in the mean time,
  you can add a SOURCE1 pointing to
  https://raw.githubusercontent.com/danielsdeleo/deep_merge/master/LICENSE
  and ship the file in the main rubygem-deep_merge package.

- This is a minor thing that doesn't block approval: the URL field can be
  updated to use HTTPS for github.com. (Reported upstream at
  https://github.com/danielsdeleo/deep_merge/pull/15).

= 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.
[-]: 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.
[x]: License field in the package spec file matches the actual license.
[!]: License file installed when any subpackage combination is installed.
 See issue listed above.
[x]: Package must own all directories that it creates.
[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.
[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 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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

Ruby:
[x]: Platform dependent files must all go under %{gem_extdir_mri}, platform
 independent under %{gem_dir}.
[x]: Gem package must not define a non-gem subpackage
[x]: Macro %{gem_extdir} is deprecated.
[x]: Gem package is named rubygem-%{gem_name}
[x]: Package contains BuildRequires: rubygems-devel.
[x]: Gem package must define %{gem_name} macro.
[x]: Pure Ruby package must be built as noarch
[x]: Package does not contain Requires: ruby(abi).
[x]: Package contains Requires: ruby(release).

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Avoid bundling fonts in non-fonts packages.
[!]: 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).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include 

[Bug 1091659] Review Request: iwyu - #include analysis tool

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



--- Comment #5 from Dave Johansen davejohan...@gmail.com ---
I added running of the tests to the check section and fixed the description
because the 3rd line was incorrectly being viewed as a comment (is there a way
to escape the # other than putting it later on the line like I did?).

-- 
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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
ghc-io-streams-1.1.4.5-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/ghc-io-streams-1.1.4.5-1.fc19

-- 
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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
ghc-io-streams-1.1.4.5-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/ghc-io-streams-1.1.4.5-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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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

Fedora Update System upda...@fedoraproject.org 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076463] Review Request: playitagainsam - Record and replay interactive terminal sessions

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

Praveen Kumar prku...@redhat.com changed:

   What|Removed |Added

 CC||prku...@redhat.com



--- Comment #10 from Praveen Kumar prku...@redhat.com ---
(In reply to Clint Savage from comment #9)
 I've updated the spec and srpm with these changes.
Can you provide latest spec and srpm location if you replaced older one please
mention same and paste url's.
 
 Thanks,
 
 herlo

-- 
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 1045963] Review Request: ghc-gtk3 - Binding to the Gtk+ graphical user interface library

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



--- Comment #4 from Jens Petersen peter...@redhat.com ---
Latest release is now 0.12.5.7 but releases  0.12.5.2 need newer
glib, pango, cairo.

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