[Bug 1166897] Review Request: mono-nat - Mono library for automatic port forwarding (new github project name: Mono.NAT)

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

  Alias||mono-nat



-- 
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 1166916] Review Request: fuzzynet - Fuzzy Logic Library for Mono

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

  Alias||fuzzynet



-- 
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 1159091] Review Request: openra - Libre/Free Real Time Strategy project that recreates the famous Command & Conquer titles

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

  Alias||openra



-- 
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 1269672] New: Review Request: nodejs-media-typer - Simple RFC 6838 media type parser

[bugzilla]

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

Bug ID: 1269672
   Summary: Review Request: nodejs-media-typer - Simple RFC 6838
media type parser
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-media-typer/nodejs-media-typer.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-media-typer/nodejs-media-typer-0.3.0-1.fc23.src.rpm
Description: Simple RFC 6838 media type parser
Fedora Account System Username: jsmith

$ rpmlint nodejs-media-typer.spec nodejs-media-typer-0.3.0-1.fc23.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
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 1229349] Review Request: radcli - Radius client library

[bugzilla]

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



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

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

[Bug 1264683] Review Request: zxing - Java multi-format 1D/2D bar-code image processing library

[bugzilla]

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



--- Comment #5 from gil cattaneo  ---
Please, add a comment in spec file about FPC ticket resolution

-- 
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 1252530] Review Request: smaclient - Provides access to z/VM System Management functions

[bugzilla]

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



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

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

[Bug 1263739] Review Request: perl-B-Hooks-Parser -Interface to perl's parser variables

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
perl-B-Hooks-Parser-0.16-2.el7 has been pushed to the Fedora EPEL 7 stable
repository. If problems still persist, please make note of it in this bug
report.

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

[Bug 1269649] Review Request: aeskulap - Full open source replacement for commercially available DICOM viewer

[bugzilla]

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



--- Comment #1 from Jens Lody  ---
This review is an unretirement review.
aeskulap was retired due to dependency-problems before F22.
I fixed these problems and also a possible crash, that happened to me also in
F21.

-- 
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 1129505] Review Request: bash-completion-extras - EPEL7 only supplemental bash-completion package

[bugzilla]

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

Adam Miller  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #14 from Adam Miller  ---
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 1241447] Review Request: jsonld-java - A JSON-based format to serialize Linked Data for Java

[bugzilla]

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

Michael Cronenworth  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #9 from Michael Cronenworth  ---
With the changes to the guidelines today bundling is largely allowed. Since the
jar does not contain real binaries inside I don't feel it should be a blocker.
you may wish to add a comment in the spec that the jar does not contain binary
class files since it is shipped in a binary rpm.

APPROVED

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

[Bug 1222334] Review Request: bup - Efficient backup system based on git

[bugzilla]

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



--- Comment #4 from Upstream Release Monitoring 
 ---
tadej's scratch build of bup-0.27-0.1.fc23.src.rpm for rawhide failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11361477

-- 
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 1252530] Review Request: smaclient - Provides access to z/VM System Management functions

[bugzilla]

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



--- Comment #21 from Fedora Update System  ---
smaclient-1.1-4.el6 has been pushed to the Fedora EPEL 6 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 1243506] Review Request: perl-Net-HL7 - Simple perl API for HL7 messages

[bugzilla]

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



--- Comment #24 from Fedora Update System  ---
perl-Net-HL7-0.81-2.el6 has been pushed to the Fedora EPEL 6 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 1269672] Review Request: nodejs-media-typer - Simple RFC 6838 media type parser

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1269676




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269676
[Bug 1269676] Review Request: nodejs-type-is - Infer the content-type of a
request
-- 
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 1269676] Review Request: nodejs-type-is - Infer the content-type of a request

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)
 Depends On||1269670, 1269672




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1269670
[Bug 1269670] Review Request: nodejs-mime-types - The ultimate javascript
content-type utility
https://bugzilla.redhat.com/show_bug.cgi?id=1269672
[Bug 1269672] Review Request: nodejs-media-typer - Simple RFC 6838 media
type parser
-- 
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 1269676] New: Review Request: nodejs-type-is - Infer the content-type of a request

[bugzilla]

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

Bug ID: 1269676
   Summary: Review Request: nodejs-type-is - Infer the
content-type of a request
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-type-is/nodejs-type-is.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-type-is/nodejs-type-is-1.6.9-1.fc23.src.rpm
Description: Infer the content-type of a request
Fedora Account System Username: jsmith

$ rpmlint nodejs-type-is.spec nodejs-type-is-1.6.9-1.fc23.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.

Depends on nodejs-media-typer from bug #1269672 and nodejs-mime-types from bug
#1269670.

-- 
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 1269670] Review Request: nodejs-mime-types - The ultimate javascript content-type utility

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1269676




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269676
[Bug 1269676] Review Request: nodejs-type-is - Infer the content-type of a
request
-- 
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 1269648] New: Review Request: nodejs-ee-first - Get the first event in a set of event emitters and event pairs

[bugzilla]

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

Bug ID: 1269648
   Summary: Review Request: nodejs-ee-first - Get the first event
in a set of event emitters and event pairs
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-ee-first/nodejs-ee-first.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-ee-first/nodejs-ee-first-1.1.1-1.fc23.src.rpm
Description: Get the first event in a set of event emitters and event pairs,
then clean up after itself
Fedora Account System Username: jsmith

$ rpmlint nodejs-ee-first.spec nodejs-ee-first-1.1.1-1.fc23.src.rpm 
./nodejs-ee-first.spec: W: invalid-url Source1: tests-1.1.1.tar.bz2
nodejs-ee-first.src: W: invalid-url Source1: tests-1.1.1.tar.bz2
1 packages and 1 specfiles checked; 0 errors, 2 warnings.

-- 
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 1269648] Review Request: nodejs-ee-first - Get the first event in a set of event emitters and event pairs

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
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 1241447] Review Request: jsonld-java - A JSON-based format to serialize Linked Data for Java

[bugzilla]

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



--- Comment #10 from gil cattaneo  ---
(In reply to Michael Cronenworth from comment #9)
> With the changes to the guidelines today bundling is largely allowed. Since
> the jar does not contain real binaries inside I don't feel it should be a
> blocker. you may wish to add a comment in the spec that the jar does not
> contain binary class files since it is shipped in a binary rpm.
> 
> APPROVED

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 1243758] Review Request: apacheds-jdbm - ApacheDS specific JDBM Implementation

[bugzilla]

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



--- Comment #27 from gil cattaneo  ---
Spec URL: https://gil.fedorapeople.org/apacheds-jdbm.spec
SRPM URL: https://gil.fedorapeople.org/apacheds-jdbm-2.0.0-0.4.M3.fc22.src.rpm

- add Provides: bundled(jdbm1)
  see https://fedorahosted.org/fpc/ticket/564 Bundled Library Exception

-- 
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 1241632] Review Request: python-sphinxcontrib-seqdiag - Sphinx "seqdiag" extension

[bugzilla]

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



--- Comment #16 from Fedora Update System  ---
python-sphinxcontrib-seqdiag-0.8.4-5.fc22 has been pushed to the Fedora 22
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 1212909] New package request: clufter - Tool for transforming/analyzing cluster configuration formats

[bugzilla]

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


--- Doc Text *updated* by Jan Pokorný  ---
Release notes: New clufter release with eponymous command directly available

Clufter received plentiful bug fixes and enhancements compared
to previous RHEL minor release, notably:

- configuration of bind-mount, nfsclient,
  SAPDatabase and SAPInstance
  rgmanager-compatible resource agents is now
  migrated in ccs2pcs command family
- ccs-/cib-/pcs-2pcscmd command
  families were added and allow one to produce a sequence to of
  pcs commands to (re)instate the cluster configuration akin
  to the existing one as truly as possible
[possibly but not necessarily also following item:]
- ccs-disable-rg command was added to allow invocation separately
  from commands that call the underlying operation implicitly

Beside using clufter in a very limited way through "pcs config import-cman"
as enabled in the previous minor release of RHEL, users are now welcome to
access full scope of available operations through "clufter" command
(note that clufter is now recognized as a separate component).
Consulting respective clufter(1) or clufter -h is
a good starting point.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1269651] Review Request: nodejs-on-finished - Execute a callback when a HTTP request closes, finishes, or errors

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)
 Depends On||1269648




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1269648
[Bug 1269648] Review Request: nodejs-ee-first - Get the first event in a
set of event emitters and event pairs
-- 
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 1269651] New: Review Request: nodejs-on-finished - Execute a callback when a HTTP request closes, finishes, or errors

[bugzilla]

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

Bug ID: 1269651
   Summary: Review Request: nodejs-on-finished - Execute a
callback when a HTTP request closes, finishes, or
errors
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-on-finished/nodejs-on-finished.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-on-finished/nodejs-on-finished-2.3.0-1.fc23.src.rpm
Description: Execute a callback when a HTTP request closes, finishes, or errors
Fedora Account System Username: jsmith

$ rpmlint nodejs-on-finished.spec nodejs-on-finished-2.3.0-1.fc23.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.

Please note that this depends on the nodejs-ee-first package being reviewed in
bug #1269648

-- 
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 1269648] Review Request: nodejs-ee-first - Get the first event in a set of event emitters and event pairs

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1269651




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269651
[Bug 1269651] Review Request: nodejs-on-finished - Execute a callback when
a HTTP request closes, finishes, or errors
-- 
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 1241447] Review Request: jsonld-java - A JSON-based format to serialize Linked Data for Java

[bugzilla]

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

gil cattaneo  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 1212909] New package request: clufter - Tool for transforming/analyzing cluster configuration formats

[bugzilla]

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

Steven J. Levine  changed:

   What|Removed |Added

 CC||slev...@redhat.com


--- Doc Text *updated* ---
Support for clufter command for transforming and analyzing cluster 
configuration formats

The clufter command provides a tool for transforming and analyzing cluster 
configuration formats. The clufter command can be used to assist with migration 
from an older stack configuration to a newer configuration that leverages 
Pacemaker. For information on the capabilities of the clufter command, see the 
clufter(1) man page or the output of the `clufter -h` command.   


-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1241447] Review Request: jsonld-java - A JSON-based format to serialize Linked Data for Java

[bugzilla]

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

gil cattaneo  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 1269649] Review Request: aeskulap - Full open source replacement for commercially available DICOM viewer

[bugzilla]

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



--- Comment #2 from Jens Lody  ---
It currently does not build in F23 and Rawhide, I will dig into this as soon as
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 1269642] Review Request: nodejs-http-errors - Create HTTP errors for Express, Koa, Connect, etc. with ease

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
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 1269649] New: Review Request: aeskulap - Full open source replacement for commercially available DICOM viewer

[bugzilla]

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

Bug ID: 1269649
   Summary: Review Request: aeskulap - Full open source
replacement for commercially available DICOM viewer
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@jenslody.de
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rpm.jenslody.de/review/aeskulap.spec
SRPM URL:
https://rpm.jenslody.de/review/aeskulap-0.2.2-0.20beta1.el7.centos.src.rpm
Description: Aeskulap is able to load a series of special images stored in the
DICOM format for review. Additionally Aeskulap is able to query
and fetch DICOM images from archive nodes (also called PACS) over
the network.

The goal of this project is to create a full open source replacement
for commercially available DICOM viewers.

Aeskulap is based on gtkmm, glademm and gconfmm and designed to run
under Linux. Ports of these packages are available for different
platforms. It should be quite easy to port Aeskulap to any platform
were these packages are available.
Fedora Account System Username: jenslody

-- 
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 1269658] New: Review Request: nodejs-mime-db - Database of all mime types

[bugzilla]

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

Bug ID: 1269658
   Summary: Review Request: nodejs-mime-db - Database of all mime
types
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-mime-db/nodejs-mime-db.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-mime-db/nodejs-mime-db-1.19.0-1.fc23.src.rpm
Description: This is a database of all mime types in a single JSON file
Fedora Account System Username: jsmith

$ rpmlint ./nodejs-mime-db.spec
~/Build/SRPMS/nodejs-mime-db-1.19.0-1.fc23.src.rpm 
nodejs-mime-db.src: W: spelling-error %description -l en_US un -> UN, nu, in
1 packages and 1 specfiles checked; 0 errors, 1 warnings.

-- 
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 1269658] Review Request: nodejs-mime-db - Database of all mime types

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
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 1212909] New package request: clufter - Tool for transforming/analyzing cluster configuration formats

[bugzilla]

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

Jan Pokorný  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1262271



-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1269658] Review Request: nodejs-mime-db - Database of all mime types

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1269670




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269670
[Bug 1269670] Review Request: nodejs-mime-types - The ultimate javascript
content-type utility
-- 
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 1269670] Review Request: nodejs-mime-types - The ultimate javascript content-type utility

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)
 Depends On||1269658




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1269658
[Bug 1269658] Review Request: nodejs-mime-db - Database of all mime types
-- 
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 1269670] New: Review Request: nodejs-mime-types - The ultimate javascript content-type utility

[bugzilla]

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

Bug ID: 1269670
   Summary: Review Request: nodejs-mime-types - The ultimate
javascript content-type utility
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-mime-types/nodejs-mime-types.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-mime-types/nodejs-mime-types-2.1.7-1.fc23.src.rpm
Description: The ultimate javascript content-type utility
Fedora Account System Username: jsmith

$ rpmlint nodejs-mime-types.spec nodejs-mime-types-2.1.7-1.fc23.src.rpm 
nodejs-mime-types.src: W: spelling-error Summary(en_US) javascript -> java
script, java-script, JavaScript
nodejs-mime-types.src: W: spelling-error %description -l en_US javascript ->
java script, java-script, JavaScript
1 packages and 1 specfiles checked; 0 errors, 2 warnings.


Depends on nodejs-mime-db being reviewed in bug #1269658.

-- 
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 1222334] Review Request: bup - Efficient backup system based on git

[bugzilla]

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



--- Comment #5 from Upstream Release Monitoring 
 ---
tadej's scratch build of bup-0.27-0.2.fc21.src.rpm for rawhide failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11361744

-- 
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 1243506] Review Request: perl-Net-HL7 - Simple perl API for HL7 messages

[bugzilla]

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



--- Comment #23 from Fedora Update System  ---
perl-Net-HL7-0.81-2.el7 has been pushed to the Fedora EPEL 7 stable repository.
If problems still persist, please make note of it in this bug report.

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

[Bug 1269672] Review Request: nodejs-media-typer - Simple RFC 6838 media type parser

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #10 from Tom Hughes  ---
That looks fine now. Package 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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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



--- Comment #8 from Tom Hughes  ---
Looks like you haven't actually uploaded the new version?

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

[Bug 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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



--- Comment #9 from Parag Nemade  ---
ah right, done now.

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

[Bug 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json

[bugzilla]

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

Parag Nemade  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |nodejs-gettext-parser - |nodejs-gettext-parser -
   |Parse and compile gettext   |Parse and compile gettext
   |po and mo files to/from |po and mo files to/from
   |json, nothing more, nothing |json
   |less|
  Flags||fedora-cvs?



--- Comment #11 from Parag Nemade  ---
Thank you Tom for this package review.

New Package SCM Request
===
Package Name: nodejs-gettext-parser
Short Description: Parse and compile gettext po and mo files to/from json
Upstream URL: https://github.com/andris9/gettext-parser
Owners: pnemade
Branches: f23
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 1245791] Review Request: python-influxdb - Python client for interacting with InfluxDB

[bugzilla]

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

Haïkel Guémar  changed:

   What|Removed |Added

 CC||pkila...@redhat.com
  Flags||needinfo?(pkilambi@redhat.c
   ||om)



--- Comment #4 from Haïkel Guémar  ---
Few comments:
* Do not pin a requirements to a specific version, it's guaranteed to break at
some point. six >= 1.9.0 is ok.
Requires:   python-six == 1.9.0
* python-requests >= 2.5.2 (or it will never pull newer requests on EL)
* Please add python3 subpackage
* As this is a new package, please follow latest python 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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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



--- Comment #4 from Upstream Release Monitoring 
 ---
pnemade's scratch build of nodejs-gettext-parser-1.1.2-1.fc23.src.rpm for f24
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11355189

-- 
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 1252530] Review Request: smaclient - Provides access to z/VM System Management functions

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-10-07 02:21:12



-- 
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 1252530] Review Request: smaclient - Provides access to z/VM System Management functions

[bugzilla]

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



--- Comment #18 from Fedora Update System  ---
smaclient-1.1-4.fc22 has been pushed to the Fedora 22 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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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



--- Comment #7 from Parag Nemade  ---
Fixed in new upstream release
Spec URL:
http://pnemade.fedorapeople.org/fedora-work/nodejs-gettext-parser.spec
SRPM URL:
http://pnemade.fedorapeople.org/fedora-work/nodejs-gettext-parser-1.1.2-1.fc23.src.rpm

and sorry for above koji comments, I was not aware that every attempt of
scratch build is now recorded in its review bugzilla.

-- 
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 1257178] Review Request: python-yaql - Yet Another Query Language

[bugzilla]

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



--- Comment #6 from Haïkel Guémar  ---
Missing definition of with_python3, so it fails to build.
Please drop max version in requirements,

%python_siteXXX definitions are not needed and should be dropped too.
Needinfo me when it's done.

-- 
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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json, nothing more, nothing less

[bugzilla]

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



--- Comment #5 from Upstream Release Monitoring 
 ---
pnemade's scratch build of nodejs-gettext-parser-1.1.2-1.fc23.src.rpm for f24
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11355232

-- 
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 1212148] Review Request: python-pyeclib - Python interface to erasure codes

[bugzilla]

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

Haïkel Guémar  changed:

   What|Removed |Added

  Flags||needinfo?(zait...@redhat.co
   ||m)



--- Comment #15 from Haïkel Guémar  ---
since liberasurecode was approved, could we refresh the spec and finish this
review?
EL7 conditionals would be appreciated so that we keep the same spec in CentOS
Cloud 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 1221781] Review Request: zipios - C++ library for reading and writing Zip files

[bugzilla]

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



--- Comment #31 from Richard Shaw  ---
SPEC: https://hobbes1069.fedorapeople.org/zipios.spec
SRPM: https://hobbes1069.fedorapeople.org/zipios-2.1.0-4.fc21.src.rpm

Ok, hope you don't mind I didn't bump the release for that little fix, now
%license is used.

-- 
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 1269609] Review Request: ari-backup - A wrapper around rdiff-backup

[bugzilla]

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



--- Comment #2 from Richard Shaw  ---
Spec URL:
https://raw.githubusercontent.com/rbarlow/ari-backup/fedora-package/distribution_packages/rpm/ari-backup.spec
SRPM URL: http://media.electronsweatshop.com/ari-backup-1.0.10-1.fc22.src.rpm

Fixing the SRPM url for 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 1269609] Review Request: ari-backup - A wrapper around rdiff-backup

[bugzilla]

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



--- Comment #3 from Richard Shaw  ---
Ok, I'm not going to post the full fedora-review output since there's some
things that need fixing, but here's some excerpts.

Issues:
===
- Permissions on files are set properly.
  Note: See rpmlint output
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions

I noticed these were intentional but it may help to describe in comments why
the special permissions are necessary (more detail in rpmlint output)


- Package contains BR: python2-devel or python3-devel

I'm questioning this one, assuming this package is both python 2 and 3
compatible, and you're using %{_python}, then you would want the build
requirement to be whichever python is the default for that system. If it's not
python3 compatible we'll need to specify python2 everywhere.


- Package does not contain duplicates in %files.
  Note: warning: File listed twice: /etc/ari-backup/ari-backup.conf.yaml
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#DuplicateFiles


- 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 %license.
  Note: License file LICENSE.txt is marked as %doc instead of %license
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

If you want to support EPEL 6 the simplest solution is something like this:
%if 0%{?rhel} < 7 || 0%{?fedora} < 21
%doc LICENSE.txt
%else
%license LICENSE.txt
%endif

Rpmlint
---
Checking: ari-backup-1.0.10-1.fc24.noarch.rpm
  ari-backup-1.0.10-1.fc24.src.rpm
ari-backup.noarch: E: description-line-too-long C A helpful wrapper around
rdiff-backup that allows backup jobs to be simple Python modules.
ari-backup.noarch: E: no-changelogname-tag
ari-backup.noarch: W: conffile-without-noreplace-flag
/etc/cron.daily/ari-backup
ari-backup.noarch: E: non-readable
/etc/ari-backup/jobs.d/ari-backup-remote-lvm-demo 600
ari-backup.noarch: E: non-executable-script
/etc/ari-backup/jobs.d/ari-backup-remote-lvm-demo 600 /usr/bin/env
ari-backup.noarch: E: non-readable /usr/share/doc/ari-backup/LICENSE.txt 700
ari-backup.noarch: E: non-standard-executable-perm
/usr/share/doc/ari-backup/LICENSE.txt 700
ari-backup.noarch: W: spurious-executable-perm
/usr/share/doc/ari-backup/LICENSE.txt
ari-backup.noarch: E: wrong-script-end-of-line-encoding
/usr/share/doc/ari-backup/LICENSE.txt
ari-backup.noarch: E: non-readable
/etc/ari-backup/jobs.d/ari-backup-local-lvm-demo 600
ari-backup.noarch: E: non-executable-script
/etc/ari-backup/jobs.d/ari-backup-local-lvm-demo 600 /usr/bin/env
ari-backup.noarch: E: non-readable
/etc/ari-backup/jobs.d/ari-backup-remote-demo 600
ari-backup.noarch: E: non-executable-script
/etc/ari-backup/jobs.d/ari-backup-remote-demo 600 /usr/bin/env
ari-backup.noarch: E: non-readable /etc/cron.daily/ari-backup 700
ari-backup.noarch: E: non-standard-executable-perm /etc/cron.daily/ari-backup
700
ari-backup.noarch: E: executable-marked-as-config-file
/etc/cron.daily/ari-backup
ari-backup.noarch: E: non-standard-dir-perm /etc/ari-backup 700
ari-backup.noarch: E: non-standard-dir-perm /var/lib/ari-backup 700
ari-backup.noarch: E: non-standard-dir-perm /etc/ari-backup/jobs.d 700
ari-backup.noarch: E: non-readable /etc/ari-backup/jobs.d/ari-backup-local-demo
600
ari-backup.noarch: E: non-executable-script
/etc/ari-backup/jobs.d/ari-backup-local-demo 600 /usr/bin/env
ari-backup.noarch: E: non-standard-dir-perm /usr/share/doc/ari-backup 700
ari-backup.noarch: E: non-readable /usr/share/doc/ari-backup/README.md 700
ari-backup.noarch: E: non-standard-executable-perm
/usr/share/doc/ari-backup/README.md 700
ari-backup.noarch: W: spurious-executable-perm
/usr/share/doc/ari-backup/README.md
ari-backup.noarch: E: non-readable /etc/ari-backup/ari-backup.conf.yaml 600
ari-backup.src: E: description-line-too-long C A helpful wrapper around
rdiff-backup that allows backup jobs to be simple Python modules.
ari-backup.src: E: no-changelogname-tag

Hopefully we can clear these up or justify them (in the case of the
permissions).

If it's important for the executables to only be readable by root, I wonder if
they should be in /usr/sbin instead?

-- 
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 1102942] Review Request: php-phpoffice-phpexcel - A pure PHP library for reading and writing spreadsheet files

[bugzilla]

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



--- Comment #8 from Remi Collet  ---
According to https://fedorahosted.org/fesco/ticket/1483 the bundling is no more
an issue.

So check if the package need to be refreshed and add the required

bundled(pear/ole)

-- 
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 1252530] Review Request: smaclient - Provides access to z/VM System Management functions

[bugzilla]

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



--- Comment #22 from Fedora Update System  ---
smaclient-1.1-4.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 1243758] Review Request: apacheds-jdbm - ApacheDS specific JDBM Implementation

[bugzilla]

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

Till Hofmann  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #28 from Till Hofmann  ---
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 1241812] Review Request: python-keystoneauth1 - OpenStack Identity Authentication Library

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 06:48:15



-- 
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 1252803] Review Request: python-zake - kazoo library testing utility

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||python-zake-0.2.2-1.fc24
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 07:06:44



-- 
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 1222334] Review Request: bup - Efficient backup system based on git

[bugzilla]

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

Christopher Meng  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||i...@cicku.me



--- Comment #3 from Christopher Meng  ---
Have been tracking bup for 3 years, I'd like to say it's good to be packaged
into Fedora.

You need to add some missing BRs for testing, and drop all programs which
require /usr/bin/env.

-- 
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 1266429] Review Request: cmark - CommonMark parsing and rendering

[bugzilla]

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

Marco Driusso  changed:

   What|Removed |Added

 CC||marcodriu...@gmail.com



--- Comment #4 from Marco Driusso  ---
This is an unofficial review, for my sponsorship process.

Comments on the spec file:
- you are including a patch for fixing cmake lib64 problem, but actually not
using it (no %patch macro in %prep section); moreover, I don't think that patch
will fix the issue;
- instead of the mv workaround for fixing cmake lib64 issue, you may patch
src/CMakeLists.txt by including GNUInstallDirs, that defines
CMAKE_INSTALL_LIBDIR according to the system architecture (lib if i686, lib64
if x86_64); then, you can use CMAKE_INSTALL_LIBDIR in install(.. DESTINATION
..) directives;
- you should give a full valid URL in the Source0 tag;
- I would put both the command line binary and the versioned shared lib in the
main package, and leave source headers and unversioned lib in the devel
sub-package, without any lib sub-package;
- devel package must require the base package (the one containing the versioned
shared lib); moreover, the dependency must be a fully versioned dependency
(since there is also an architecture dependency);
- you should include additional documentation in the main package using %doc
(e.g. the README.md file) in the %files section;
- as Miroslav pointed out, ldconfig must be called in %post and %postun
sections, which should be placed after %check and before %files

You should then fix all errors and warnings highlighted by rpmlint (some
related to the comments above):

$ rpmlint cmark.spec
cmark.spec:59: E: hardcoded-library-path in %{_prefix}/lib
cmark.spec: W: patch-not-applied Patch1: cmark-0.22.0-lib64.patch
cmark.spec: W: invalid-url Source0: cmark-0.22.0.tar.gz
0 packages and 1 specfiles checked; 1 errors, 2 warnings.

$ rpmlint cmark*.rpm
cmark.src: W: spelling-error %description -l en_US groff -> gruff, gr off,
gr-off
cmark.src:59: E: hardcoded-library-path in %{_prefix}/lib
cmark.src: W: patch-not-applied Patch1: cmark-0.22.0-lib64.patch
cmark.src: W: invalid-url Source0: cmark-0.22.0.tar.gz
cmark.x86_64: W: spelling-error %description -l en_US groff -> gruff, gr off,
gr-off
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/utf8.h
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/utf8.c
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/references.c
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/render.c
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/houdini_html_u.c
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/inlines.c
cmark-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/cmark-0.22.0/src/blocks.c
cmark-devel.x86_64: W: no-dependency-on cmark/cmark-libs/libcmark
cmark-devel.x86_64: W: spelling-error Summary(en_US) libcmark -> millibar
cmark-devel.x86_64: W: summary-not-capitalized C libcmark development files
cmark-devel.x86_64: W: spelling-error %description -l en_US libcmark ->
millibar
cmark-devel.x86_64: W: only-non-binary-in-usr-lib
cmark-lib.x86_64: W: spelling-error %description -l en_US libcmark -> millibar
cmark-lib.x86_64: W: shared-lib-calls-exit /usr/lib64/libcmark.so.0.22.0
exit@GLIBC_2.2.5
cmark-lib.x86_64: W: no-documentation
cmark-lib.x86_64: E: library-without-ldconfig-postin
/usr/lib64/libcmark.so.0.22.0
cmark-lib.x86_64: E: library-without-ldconfig-postun
/usr/lib64/libcmark.so.0.22.0
5 packages and 0 specfiles checked; 3 errors, 19 warnings.

-- 
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 995974] Review Request: libbson - BSON library for C

[bugzilla]

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

Christopher Meng  changed:

   What|Removed |Added

 Resolution|NOTABUG |DUPLICATE



--- Comment #34 from Christopher Meng  ---


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

-- 
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 1208101] Review Request: libbson - Building, parsing, and iterating BSON documents

[bugzilla]

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

Christopher Meng  changed:

   What|Removed |Added

 CC||crtm...@gmx.us



--- Comment #15 from Christopher Meng  ---
*** Bug 995974 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 1242896] Review Request: python-ironic-inspector-client - Python client and CLI tool for Ironic Inspector

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||python-ironic-inspector-cli
   ||ent-1.2.0-2.fc24
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 06:50: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 1254851] Review Request: python-os-testr : A testr wrapper to provide functionality for OpenStack projects

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||python-os-testr-0.3.0-1.fc2
   ||4
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 07:05:06



-- 
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 1259919] Review Request: python-castellan - Generic Key Manager interface for OpenStack

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||python-castellan-0.2.1-1.fc
   ||24
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 07:04:20



-- 
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 1264686] Review Request: itpp - C++ library for math, signal/speech processing, and communications

[bugzilla]

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



--- Comment #11 from Marco Driusso  ---
I performed an unofficial review for my sponsorship process:
https://bugzilla.redhat.com/show_bug.cgi?id=1266429

-- 
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 1241812] Review Request: python-keystoneauth1 - OpenStack Identity Authentication Library

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Fixed In Version||python-keystoneauth1-1.0.0-
   ||1.fc24



-- 
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 1266273] Review Request: python-unicodecsv - Drop-in replacement for csv module which supports unicode strings

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-10-07 06:52:24



--- Comment #9 from Alan Pevec  ---
Conclusion from upstream discussion will be clarify package Summary:
https://github.com/jdunck/python-unicodecsv/issues/66#issuecomment-145508768

-Summary: Drop-in replacement for csv module which supports unicode strings
+Summary: Drop-in replacement for Python 2.7's csv module which supports
unicode strings

-- 
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 1267003] Review Request: rows - A common, beautiful interface to tabular data, no matter the format

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1267003
Bug 1267003 depends on bug 1266273, which changed state.

Bug 1266273 Summary: Review Request: python-unicodecsv - Drop-in replacement 
for csv module which supports unicode strings
https://bugzilla.redhat.com/show_bug.cgi?id=1266273

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE



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

[Bug 1060920] Review Request: openni2 - OpenNI libraries for 3D-sensing

[bugzilla]

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



--- Comment #22 from Jiri Kastner  ---
for fedora 22 'dnf copr enable -y jkastner/various'. trying to install
openni2-examples, i get this:

"Error: nothing provides libMWClosestPoint.so()(64bit) needed by
openni2-examples-2.2.0.33-2.fc22.x86_64"

-- 
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 1268920] Review Request: perl-Dist-Zilla-App-Command-cover - Code coverage metrics for your distribution

[bugzilla]

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



--- Comment #4 from Fedora Update System  ---
perl-Dist-Zilla-App-Command-cover-1.101001-1.fc23 has been submitted as an
update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b80bd52a95

-- 
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 1268920] Review Request: perl-Dist-Zilla-App-Command-cover - Code coverage metrics for your distribution

[bugzilla]

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



--- Comment #4 from Fedora Update System  ---
perl-Dist-Zilla-App-Command-cover-1.101001-1.fc23 has been submitted as an
update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b80bd52a95

--- Comment #5 from Fedora Update System  ---
perl-Dist-Zilla-App-Command-cover-1.101001-1.fc22 has been submitted as an
update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-652fec44c1

-- 
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 1268920] Review Request: perl-Dist-Zilla-App-Command-cover - Code coverage metrics for your distribution

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



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

[Bug 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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

Chandan Kumar  changed:

   What|Removed |Added

 CC||chkumar...@gmail.com
  Flags||fedora-cvs?



--- Comment #2 from Chandan Kumar  ---
New Package SCM Request
===
Package Name: python-weakrefmethod
Short Description: This project is a backport of the WeakMethod class, and
tests, for Python 2.6.
The tests require the `unittest2 package
`_.

Upstream URL: https://github.com/twang817/weakrefmethod
Owners: chandankumar social
Branches: f23 f22
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 759818] Review Request: buffer - General purpose buffer program

[bugzilla]

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



--- Comment #16 from Miroslav Suchý  ---
Package Review
==

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


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

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.
[x]: %build honors applicable compiler flags or justifies otherwise.
[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]: Useful -debuginfo package or justification otherwise.
[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 10240 bytes in 1 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: 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 %license.
[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]: Dist tag is present.
[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 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:
[-]: 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.
[!]: Patches link to upstream bugs/comments/lists or are otherwise
 justified.
 -- it is clear to my why those patches are there, so I leave it as is.
[x]: 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]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[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]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Uses parallel make %{?_smp_mflags} macro.
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless justified.

= EXTRA items =

Generic:
[x]: Rpmlint 

[Bug 1060920] Review Request: openni2 - OpenNI libraries for 3D-sensing

[bugzilla]

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



--- Comment #24 from Jiri Kastner  ---
well MWClosestPointApp needs libMWClosestPoint.so in reachable path:
[indy@localhost ~]$ MWClosestPointApp
MWClosestPointApp: error while loading shared libraries: libMWClosestPoint.so:
cannot open shared object file: No such file or directory

there are two solutions
1.) add _libdir/openni2 to /etc/ld-config.d, but i think openni2 is folder for
openni2 itself
2.) put libMWClosestPoint.so to _libdir itself

-- 
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 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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

Lukas Bezdicka  changed:

   What|Removed |Added

 CC||soc...@v3.sk
   Assignee|nob...@fedoraproject.org|soc...@v3.sk
  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 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json

[bugzilla]

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



--- Comment #13 from Tom Hughes  ---
Shouldn't that have been announced somewhere?

-- 
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 1249743] Review Request: kaadbg - Remote debugger runner for kaa

[bugzilla]

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

René Ribaud  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from René Ribaud  ---
Hello William,

Everything is ok from my point of view.
Just inform upstream to create a license file to be able to remove the License
patch for the next release.


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. Licenses
 found: "Unknown or generated". 3 files have unknown license. Detailed
 output of licensecheck in
 /home/uggla/rpmbuild/SPECS/1249743-kaadbg/licensecheck.txt
[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.
[-]: 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 10240 bytes in 1 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: 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 %license.
[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]: Dist tag is present.
[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 does not use a name that already exists.
[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.
[-]: 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:
[?]: 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.
[-]: 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.
[x]: %check is present 

[Bug 1060920] Review Request: openni2 - OpenNI libraries for 3D-sensing

[bugzilla]

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



--- Comment #23 from Jiri Kastner  ---
fixed with
http://copr-dist-git.fedorainfracloud.org/cgit/jkastner/various/openni2.git/patch/?id=8a086e993394436bdcf75d11969ab31a32987055

-- 
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 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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



--- Comment #1 from Lukas Bezdicka  ---
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.
[-]: 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 %license.
[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". 3 files have unknown license. Detailed
 output of licensecheck in /home/social/1269039-python-
 weakrefmethod/licensecheck.txt
[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.
[x]: 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.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 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: 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]: Dist tag is present.
[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 does not use a name that already exists.
[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).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: 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.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[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]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a 

[Bug 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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

Jon Ciesla  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 1264199] Review Request: php-wikimedia-avro - A library for using Avro with PHP

[bugzilla]

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



--- Comment #6 from Fedora Update System  ---
php-wikimedia-avro-1.7.7-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-71c0e977f3

-- 
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 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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



--- Comment #3 from Jon Ciesla  ---
This SCM request method has been deprecated. Please see
https://fedoraproject.org/wiki/PackageDB_admin_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 759818] Review Request: buffer - General purpose buffer program

[bugzilla]

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



--- Comment #15 from Miroslav Suchý  ---
(In reply to Miroslav Suchý from comment #12)
> buffer.x86_64: E: incorrect-fsf-address /usr/share/licenses/buffer/COPYING
> You should containt upstream so they update they license file. However, you
> should not alter the file yourself!

Did you contact upstream about this 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 1269287] Review Request: nodejs-when - A lightweight Promises/A+ and when() implementation

[bugzilla]

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



--- Comment #3 from Jared Smith  ---
I've updated the spec file and the associated SRPM at the same locations listed
above.

I've fixed the inconsistency between the .spec file and the SRPM, as well as
added (provisional) support for running the test suite.  Unfortunately, the
tests won't run right now because the test suite requires the "buster" package,
which has not yet been packaged in Fedora, and itself has a long list of
dependencies.

As such, I've left the %global enable_tests set to 0, but this could easily be
flipped in the future should someone package up buster.

-- 
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 1264199] Review Request: php-wikimedia-avro - A library for using Avro with PHP

[bugzilla]

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



--- Comment #7 from Fedora Update System  ---
php-wikimedia-avro-1.7.7-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-b886aeed0d

-- 
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 1264199] Review Request: php-wikimedia-avro - A library for using Avro with PHP

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



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

[Bug 1265321] Review Request: nodejs-gettext-parser - Parse and compile gettext po and mo files to/from json

[bugzilla]

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

Jon Ciesla  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 1209166] Review Request: palo - cell-oriented, multi-dimensional in-memory OLAP server

[bugzilla]

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



--- Comment #9 from Upstream Release Monitoring 
 ---
myk52348's scratch build of palo-5.1-4.fc22.x86_64.rpm for f22 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11357361

-- 
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 1269039] Review Request: python-weakrefmethod - A WeakMethod class for storing bound methods using weak references

[bugzilla]

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

Lukas Bezdicka  changed:

   What|Removed |Added

   Assignee|soc...@v3.sk|nob...@fedoraproject.org
  Flags|fedora-review?  |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 1269539] New: Review Request: mozjs38 - JavaScript interpreter and libraries

[bugzilla]

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

Bug ID: 1269539
   Summary: Review Request: mozjs38 - JavaScript interpreter and
libraries
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mskal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://mskalick.fedorapeople.org/mozjs38/mozjs38.spec
SRPM URL:
https://mskalick.fedorapeople.org/mozjs38/mozjs38-38.2.1-1.fc22.src.rpm
Description: JavaScript is the Netscape-developed object scripting language
used in millions of web pages and server applications worldwide. Netscape's
JavaScript is a super set of the ECMA-262 Edition 3 (ECMAScript) standard
scripting language, with only mild differences from the published standard.
Fedora Account System Username: mskalick

-- 
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 1269559] Review Request: nodejs-basic-auth - Basic auth Authorization header field parser

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js 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 1057770] Review Request: Squeryl - ORM and DSL for SQL databases in Scala

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
Available newer release 0.9.5-7
Url: https://github.com/squeryl/squeryl/tags

-- 
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 1269545] Review Request: python-gnocchiclient - Gnocchi CLI

[bugzilla]

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

Haïkel Guémar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 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 1264209] Review Request: php-wikimedia-ip-set - Library to match IP addresses against CIDR specifications

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
php-wikimedia-ip-set-0-0.1.20150917git.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-151353fbfb

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