[Bug 1212148] Review Request: python-pyeclib - Python interface to erasure codes

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #27 from Fedora Update System  ---
liberasurecode-1.0.9-3.fc23, python-pyeclib-1.0.9-1.fc23 has been pushed to the
Fedora 23 testing repository. If problems still persist, please make note of it
in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-pyeclib liberasurecode'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-9fa214a26f

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

[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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



--- Comment #2 from Jared Smith  ---
Ooops, sorry.

SRPM:
https://jsmith.fedorapeople.org/Packaging/nodejs-is-regexp/nodejs-is-regexp-1.0.0-1.fc23.src.rpm

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

[Bug 1272235] Review Request: distribution-gpg-keys - Keys of various Linux distributions

[bugzilla]

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl
   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1246701



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
https://bugzilla.redhat.com/show_bug.cgi?id=1246701 is about including more
Fedora keys in fedora-repos.

There's also https://apps.fedoraproject.org/packages/archlinux-keyring for
similar purpose.

I think it is very useful and increases security of various cross-distro
installation. I wonder though whether not to remove Fedora and EPEL keys from
this, since they will be included in fedora-repos, or maybe to add a check to
make sure that they are identical in both packages.

Regarding packaging:
- why not use a github tarball directly? It's much nicer than to force a git
clone and additional steps.

- GPL, seriously? I'm all for GPL, but in this case CC-0 seems a much better
choice. After all, this should be freely copied.

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

[Bug 1241383] Review Request: mkchroot - Fedora Chroot Directory Maker

[bugzilla]

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

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



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

[Bug 1268372] Review Request: openstack-app-catalog-ui - openstack horizon plugin for the openstack app-catalog

[bugzilla]

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

Alan Pevec  changed:

   What|Removed |Added

 CC||ape...@gmail.com,
   ||kevin@pnnl.gov
  Flags||needinfo?(kevin@pnnl.go
   ||v)



--- Comment #5 from Alan Pevec  ---
To proceed, please update your draft spec according to comment 3

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

[Bug 1255815] Review Request: apacheds-ldap-api - Apache Directory LDAP API

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-10-15 18:30:39



--- Comment #11 from gil cattaneo  ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=11466108

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

[Bug 1255815] Review Request: apacheds-ldap-api - Apache Directory LDAP API

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

   Fixed In Version||apacheds-ldap-api-1.0.0-0.2
   ||.M31



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

[Bug 1272255] Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

[bugzilla]

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

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 1272255] New: Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

[bugzilla]

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

Bug ID: 1272255
   Summary: Review Request: nodejs-lcov-parse - Parse lcov results
files and return JSON
   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-lcov-parse/nodejs-lcov-parse.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-lcov-parse/nodejs-lcov-parse-0.0.10-1.fc23.src.rpm
Description: Parse lcov results files and return JSON
Fedora Account System Username: jsmith

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

[Bug 1090499] Review Request: netresolve - Generic name resolution library

[bugzilla]

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



--- Comment #19 from Pavel Šimerda (pavlix)  ---

Spec URL: http://pavlix.fedorapeople.org//netresolve.spec
SRPM URL:
http://pavlix.fedorapeople.org//netresolve-0.0.1-0.6.20151015git.fc24.src.rpm

Description:
Netresolve is a package for non-blocking network name resolution via backends
intended as a replacement for name service switch based name resolution in
glibc as well as a testbed for future glibc improvements.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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

awill...@redhat.com  changed:

   What|Removed |Added

 CC||awill...@redhat.com
 Whiteboard||AcceptedFreezeException



--- Comment #26 from awill...@redhat.com  ---
Discussed at 2015-10-15 freeze exception review meeting:
https://meetbot-raw.fedoraproject.org/fedora-blocker-review/2015-10-15/f23-blocker-review.2015-10-15-20.11.log.txt
. In principle we generally approved this kind of FE, as avoiding dependency
problems in the frozen trees is a useful goal, and there's no apparent
complications with this one, so it's 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 1272248] Review Request: heketi - RESTful based volume management framework for GlusterFS

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Depends On||1271684




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1271684
[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic
HTTP Middleware for Golang
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic HTTP Middleware for Golang

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Blocks||1272248




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272248
[Bug 1272248] Review Request: heketi - RESTful based volume management
framework for GlusterFS
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272248] Review Request: heketi - RESTful based volume management framework for GlusterFS

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Depends On||1271805




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1271805
[Bug 1271805] Review Request: golang-github-lpabon-godbc - Design by
contract for Go
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272248] Review Request: heketi - RESTful based volume management framework for GlusterFS

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Depends On||1269813




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269813
[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang
implementation of JSON Web Tokens (JWT)
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Blocks||1272248




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272248
[Bug 1272248] Review Request: heketi - RESTful based volume management
framework for GlusterFS
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1271805] Review Request: golang-github-lpabon-godbc - Design by contract for Go

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Blocks||1272248




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272248
[Bug 1272248] Review Request: heketi - RESTful based volume management
framework for GlusterFS
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272248] Review Request: heketi - RESTful based volume management framework for GlusterFS

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 CC||jchal...@redhat.com
 Depends On||1272116




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272116
[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A
Middleware for Go Programming Language to check for JWTs on HTTP 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 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Luis Pabón  changed:

   What|Removed |Added

 Blocks||1272248




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272248
[Bug 1272248] Review Request: heketi - RESTful based volume management
framework for GlusterFS
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272248] New: Review Request: heketi - RESTful based volume management framework for GlusterFS

[bugzilla]

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

Bug ID: 1272248
   Summary: Review Request: heketi - RESTful based volume
management framework for GlusterFS
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: lpa...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://lpabon.fedorapeople.org/reviews/heketi/heketi.spec

SRPM URL:
https://lpabon.fedorapeople.org/reviews/heketi/heketi-1.0.0-1.fc22.src.rpm

Description: Heketi provides a RESTful management interface which can be used
to manage
the life cycle of GlusterFS volumes.  With Heketi, cloud services like
OpenStack Manila, Kubernetes, and OpenShift can dynamically provision
GlusterFS volumes with any of the supported durability types.  Heketi
will automatically determine the location for bricks across the cluster,
making sure to place bricks and its replicas across different failure
domains.  Heketi also supports any number of GlusterFS clusters, allowing
cloud services to provide network file storage without being limited to a
single GlusterFS cluster.

Fedora Account System Username: lpabon

$ rpmlint heketi-1.0.0-1.fc22.src.rpm heketi-1.0.0-1.fc22.x86_64.rpm
heketi-devel-1.0.0-1.fc22.noarch.rpm
heketi-unit-test-devel-1.0.0-1.fc22.x86_64.rpm
heketi-debuginfo-1.0.0-1.fc22.x86_64.rpm
heketi.src:153: W: rpm-buildroot-usage %build export
GOPATH=%{buildroot}/%{gopath}:%{gopath}
heketi.x86_64: W: non-standard-uid /var/lib/heketi heketi
heketi.x86_64: W: non-standard-gid /var/lib/heketi heketi
heketi.x86_64: W: no-manual-page-for-binary heketi
heketi.x86_64: W: no-manual-page-for-binary heketi-cli
5 packages and 0 specfiles checked; 0 errors, 5 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 1107127] Review Request: procenv - Utility to show process environment

[bugzilla]

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



--- Comment #9 from Dave Love  ---
SPEC URL: https://loveshack.fedorapeople.org/review/procenv.spec
SRPM URL: https://loveshack.fedorapeople.org/review/procenv-0.40-1.el5.src.rpm

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

[Bug 1266258] Review Request: python-urlobject - A utility class for manipulating URLs

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #14 from Fedora Update System  ---
python-urlobject-2.4.0-3.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-urlobject'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-fbab609b5d

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.fc23 has been pushed to the Fedora 23
testing repository. If problems still persist, please make note of it in this
bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update
golang-github-dgrijalva-jwt-go'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-cc6d4648de

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

[Bug 1272235] New: Review Request: distribution-gpg-keys - Keys of various Linux distributions

[bugzilla]

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

Bug ID: 1272235
   Summary: Review Request: distribution-gpg-keys - Keys of
various Linux distributions
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: msu...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://miroslav.suchy.cz/fedora/distribution-gpg-keys.spec
SRPM URL:
http://miroslav.suchy.cz/fedora/distribution-gpg-keys-1.1-1.fc22.src.rpm
Description: 
GPG keys used by various Linux distributions to sign packages.

Fedora Account System Username: msuchy

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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

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

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
python-pyeclib-1.0.9-1.fc23 liberasurecode-1.0.9-3.fc23 has been submitted as
an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-9fa214a26f

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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

Kevin Fenzi  changed:

   What|Removed |Added

 CC||ke...@scrye.com



--- Comment #24 from Kevin Fenzi  ---
+1 FE

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

[Bug 1272194] Review Request: target-isns - An iSNS client for the Linux LIO iSCSI target

[bugzilla]

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

Chris Leech  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||agro...@redhat.com,
   ||cle...@redhat.com
   Assignee|nob...@fedoraproject.org|cle...@redhat.com
  Flags||fedora-review?
   ||needinfo?(agro...@redhat.co
   ||m)



--- Comment #1 from Chris Leech  ---
/etc/target-isns.conf isn't marked as a config file

 %files
 %{_bindir}/target-isns
-%{_sysconfdir}/target-isns.conf
+%config(noreplace) %{_sysconfdir}/target-isns.conf
 %{_mandir}/man8/target-isns.8.gz
 %{_unitdir}/target-isns.service
 %doc README.md

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

[Bug 1271768] Review Request: python-shade -- client library for operating OpenStack clouds

[bugzilla]

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

Lars Kellogg-Stedman  changed:

   What|Removed |Added

  Flags||needinfo?(karlthered@gmail.
   ||com)



--- Comment #5 from Lars Kellogg-Stedman  ---
Haïkel: I've updated the spec in a way that hopefully addresses your comments. 
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 1228389] Review Request: gcalcli - Google Calendar Command Line Interface

[bugzilla]

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



--- Comment #2 from Michele Baldessari  ---
Spec URL: http://acksyn.org/files/rpms/gcalcli/gcalcli.spec
SRPM URL: http://acksyn.org/files/rpms/gcalcli/gcalcli-3.3.2-1.fc22.src.rpm

Fedora Account System Username: mbaldessari

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

[Bug 1228389] Review Request: gcalcli - Google Calendar Command Line Interface

[bugzilla]

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

Michele Baldessari  changed:

   What|Removed |Added

 Depends On||1272187




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272187
[Bug 1272187] Review Request: google-api-python-client - Google APIs Client
Library for Python
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272187] Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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

Michele Baldessari  changed:

   What|Removed |Added

 Blocks||1228389




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1228389
[Bug 1228389] Review Request: gcalcli - Google Calendar Command Line
Interface
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272194] New: Review Request: target-isns - An iSNS client for the Linux LIO iSCSI target

[bugzilla]

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

Bug ID: 1272194
   Summary: Review Request: target-isns - An iSNS client for the
Linux LIO iSCSI target
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: agro...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fedorapeople.org/~grover/new/target-isns.spec
SRPM URL: https://fedorapeople.org/~grover/new/target-isns-0.5-1.fc22.src.rpm
Description: 

Target-isns is an Internet Storage Name Service (iSNS) client for the Linux
LIO iSCSI target. It allows registering LIO iSCSI targets with an iSNS server.

Fedora Account System Username: grover

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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 #12 from Marco Driusso  ---
Info for sponsorship process: find below the link to a bug for which I
submitted a patch and an updated spec, which have been used by the maintainer
for releasing an update.
https://bugzilla.redhat.com/show_bug.cgi?id=1268289

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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



--- Comment #7 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-cc6d4648de

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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



--- Comment #6 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.fc22 has been submitted as an update to
Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-0161e398ec

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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



--- Comment #5 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.fc21 has been submitted as an update to
Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-a76e0f23fb

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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

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 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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



--- Comment #4 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.el6 has been submitted as an update to
Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-f0cadbf1f2

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

[Bug 1265885] Review Request: zeal - Offline documentation browser inspired by Dash

[bugzilla]

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



--- Comment #6 from Michel Alexandre Salim  ---
Excellent, 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 989068] Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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



--- Comment #28 from Michele Baldessari  ---
Thanks Paul, I've fixed that up. I submitted 1272187 for a new review. If you
feel like reviewing that'd be grand. If not, I'll check if Matthias has some
spare cycles.

cheers,
Michele

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

[Bug 1272187] New: Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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

Bug ID: 1272187
   Summary: Review Request: google-api-python-client - Google APIs
Client Library for Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: mich...@acksyn.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://acksyn.org/files/rpms/google-api-python-client/google-api-python-client.spec
SRPM URL:
http://acksyn.org/files/rpms/google-api-python-client/google-api-python-client-1.4.2-1.fc22.src.rpm

Description: Written by Google, this library provides a small, flexible, and
powerful Python client library for accessing Google APIs.


Fedora Account System Username: mbaldessari

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

[Bug 1265885] Review Request: zeal - Offline documentation browser inspired by Dash

[bugzilla]

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

Jackson Isaac  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+
   |needinfo?(jacksonisaac2008@ |
   |gmail.com)  |



--- Comment #5 from Jackson Isaac  ---
Hi,

Looks good to me. Review + from me. Thank you for adding relevant links of mail
discussions and upstream issue link in the spec file.

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

[Bug 1203749] Review Request: dssp - Protein secondary structure assignment

[bugzilla]

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



--- Comment #21 from Dave Love  ---
(In reply to Antonio Trande from comment #15)

> Packaging fails on EPEL5:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=10543567

That seems to have been a koji problem, like the first of the builds noted.
The second worked, and it's running here on EPEL5.

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

[Bug 1272174] New: Review Request: golang-github-pmezard-go-difflib - Partial port of Python difflib package to Go

[bugzilla]

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

Bug ID: 1272174
   Summary: Review Request: golang-github-pmezard-go-difflib -
Partial port of Python difflib package to Go
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jchal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-pmezard-go-difflib/golang-github-pmezard-go-difflib.spec

SRPM URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-pmezard-go-difflib/golang-github-pmezard-go-difflib-0-0.1.gitf78a839.fc20.src.rpm

Description: Partial port of Python difflib package to Go

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

Fedora Account System Username: jchaloup

$ rpmlint golang-github-pmezard-go-difflib-0-0.1.gitf78a839.fc20.src.rpm
golang-github-pmezard-go-difflib-devel-0-0.1.gitf78a839.fc20.noarch.rpm
golang-github-pmezard-go-difflib-unit-test-devel-0-0.1.gitf78a839.fc20.x86_64.rpm
3 packages and 0 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 1270376] Review Request: nodejs-destroy - Destroy a stream if possible

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
nodejs-destroy-1.0.3-1.el6 has been pushed to the Fedora EPEL 6 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'yum --enablerepo=epel-testing update nodejs-destroy'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-6ba1044b63

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

[Bug 1270377] Review Request: nodejs-escape-html - Escape string for HTML

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
nodejs-escape-html-1.0.3-1.el6 has been pushed to the Fedora EPEL 6 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'yum --enablerepo=epel-testing update nodejs-escape-html'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8c3859f0b0

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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



--- Comment #10 from Fedora Update System  ---
nodejs-media-typer-0.3.0-1.el6 has been pushed to the Fedora EPEL 6 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'yum --enablerepo=epel-testing update nodejs-media-typer'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-29459d23ec

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

[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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

Parag AN(पराग)  changed:

   What|Removed |Added

 CC||panem...@gmail.com



--- Comment #1 from Parag AN(पराग)  ---
ah this missed the srpm link

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

[Bug 1272134] Review Request: python-journal-brief - Find new systemd journal entries since last run

[bugzilla]

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



--- Comment #1 from Matthew Miller  ---
I'm getting build errors in mock (both f22 and rawhide). Will have to diagnose
whether that's my system or a real problem. :)

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

[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Parag AN(पराग)  changed:

   What|Removed |Added

 CC||panem...@gmail.com



--- Comment #4 from Parag AN(पराग)  ---
ah fedora-cvs+ that should have been fedora-review+ actually.

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

[Bug 1272134] New: Review Request: python-journal-brief - Find new systemd journal entries since last run

[bugzilla]

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

Bug ID: 1272134
   Summary: Review Request: python-journal-brief - Find new
systemd journal entries since last run
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: twa...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://twaugh.fedorapeople.org/journal-brief/python-journal-brief.spec
SRPM URL:
https://twaugh.fedorapeople.org/journal-brief/python-journal-brief-1.1.1-1.fc22.src.rpm
Description:
Python module for examining, bookmarking, and filtering systemd
journal entries.

Fedora Account System Username: twaugh

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

[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



--- Comment #3 from Jan Chaloupka  ---
The spec file conforms to the current packaging guidelines draft. Before this
package is built in Koji please make sure all its dependencies are built as
well and overridden.

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

[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic HTTP Middleware for Golang

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Blocks||1272116




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272116
[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A
Middleware for Go Programming Language to check for JWTs on HTTP 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 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Depends On||1271684




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1271684
[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic
HTTP Middleware for Golang
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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



--- Comment #2 from Jan Chaloupka  ---
Sorry, all deps are already reviewed.

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

[Bug 1239267] Review Request: innoextract - Inno Setup installers extractor

[bugzilla]

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



--- Comment #5 from Juan Orti  ---
Hi, I was packaging this program when I realized it was already in the review
queue :D

Could you update it to version 1.5?

I don't think you need all those flags for %cmake, I have built it only with
%cmake and everything seems ok.

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

[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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



--- Comment #1 from Jan Chaloupka  ---
The current package depends on other packages that are not yet on review. Can
you first generate the missing spec files? I will then continue with this
review.

Summary:
- all [Build]Requires correct
- devel subpackage provided
- unit-test provided
- license OK

Notes:
- ass check is disabled, the spec file has no build-time dependency on any
golang project. So it is very easy to miss some. In this case it is better to
first install built devel rpm and see if it works.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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 #13 from Richard Shaw  ---
(In reply to Randy Barlow from comment #12)
> 
> Hi again Richard! Sure thing, I've now done one informal review here on
> another Python package:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1268380#c2

That's a good start, it wouldn't hurt to find a few more.

Here's a good way to find some, I would skip over the older ones unless you
find one you are really interested in:

https://fedoraproject.org/PackageReviewStatus/NEW.html


> > Also, what are you interests? Mostly python packaging? Are there any
> > packages you'd like to co-maintain?
> 
> I do work as a full time Python developer so Python is my strength these
> days. In the past I have also done work in C/C++, Java, and PHP. I would not
> rate my Java or PHP knowlege to be current enough to be very useful, but I
> am still pretty good in C/C++ land.

There is a Fedora Python SIG you may want to join then:

https://fedoraproject.org/wiki/SIGs/Python

We can certainly work towards me becoming your sponsor but if someone in the
Python SIG is intereated in becoming your sponsor then that's probably a better
fit.


> One of my long term goals for wanting to join the Fedora project is getting
> my full time project included in Fedora:
> 
> http://www.pulpproject.org/

That looks pretty neat. I wouldn't mind trying it out but it may be a bit
overkill for my 2 desktop and 3 laptop home network :)


> It is also a Python project, and I suspect that you might cringe if you saw
> it's current spec file ☺ It's a pretty complicated project with quite a few
> packages (it's a distributed system) so I didn't want to use it as my first
> package review. Once I gain some more experience I'd love to get it included
> in Fedora. We currently host our own repository for Fedora on
> repos.fedorapeople.org/pulp. I also happen to know that Fedora has some
> interest in possibly using Pulp to manage some of their systems so it may be
> beneficial to the project itself.

Yeah, we should use Fedora supplied packages wherever possible. If you want to
start working towards getting it into Fedora the best option might be to start
with COPR[1] as the only real requirement is that all the content is legal
(acceptable license). The full packaging guidelines don't apply.

[1] https://copr.fedoraproject.org/coprs/

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

[Bug 1272116] Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jchal...@redhat.com
  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 1239267] Review Request: innoextract - Inno Setup installers extractor

[bugzilla]

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

Juan Orti  changed:

   What|Removed |Added

 CC||j.orti.alca...@gmail.com
   Assignee|nob...@fedoraproject.org|j.orti.alca...@gmail.com
  Flags||fedora-review?



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

[Bug 1265628] Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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



--- Comment #11 from Upstream Release Monitoring 
 ---
pavlix's scratch build of dhcpcd-6.9.3-2.fc24.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11460085

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

[Bug 1272116] New: Review Request: golang-github-auth0-go-jwt-middleware - A Middleware for Go Programming Language to check for JWTs on HTTP requests

[bugzilla]

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

Bug ID: 1272116
   Summary: Review Request: golang-github-auth0-go-jwt-middleware
- A Middleware for Go Programming Language to check
for JWTs on HTTP requests
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: lpa...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://lpabon.fedorapeople.org/reviews/golang-github-auth0-go-jwt-middleware/golang-github-auth0-go-jwt-middleware.spec

SRPM URL:
https://lpabon.fedorapeople.org/reviews/golang-github-auth0-go-jwt-middleware/golang-github-auth0-go-jwt-middleware-0.1-1.git8c897f7.fc22.src.rpm

Description: A Middleware for Go Programming Language to check for JWTs on HTTP
requests

Fedora Account System Username: lpabon

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

$ rpmlint golang-github-auth0-go-jwt-middleware-0.1-1.git8c897f7.fc22.src.rpm
golang-github-auth0-go-jwt-middleware-devel-0.1-1.git8c897f7.fc22.noarch.rpm
golang-github-auth0-go-jwt-middleware-unit-test-devel-0.1-1.git8c897f7.fc22.x86_64.rpm
3 packages and 0 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 1265628] Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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



--- Comment #9 from Upstream Release Monitoring 
 ---
pavlix's scratch build of dhcpcd-6.9.3-2.fc24.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=11460080

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

[Bug 1265628] Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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



--- Comment #10 from Pavel Šimerda (pavlix)  ---

Spec URL: http://pavlix.fedorapeople.org//dhcpcd.spec
SRPM URL: http://pavlix.fedorapeople.org//dhcpcd-6.9.3-2.fc24.src.rpm

Description:
The dhcpcd package provides a minimalistic network configuration daemon
that supports IPv4 and IPv6 configuration including configuration discovery
through NDP, DHCPv4 and DHCPv6 protocols.

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

[Bug 1272115] Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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

Pavel Šimerda (pavlix)  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2015-10-15 10:06:24



--- Comment #1 from Pavel Šimerda (pavlix)  ---


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

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

[Bug 1265628] Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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



--- Comment #8 from Pavel Šimerda (pavlix)  ---
*** Bug 1272115 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 1272115] New: Review Request: dhcpcd - A minimalistic network configuration daemon with DHCP and rdisc support

[bugzilla]

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

Bug ID: 1272115
   Summary: Review Request: dhcpcd - A minimalistic network
configuration daemon with DHCP and rdisc support
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: psime...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://pavlix.fedorapeople.org//dhcpcd.spec
SRPM URL: http://pavlix.fedorapeople.org//dhcpcd-6.9.3-2.fc24.src.rpm

Description:
The dhcpcd package provides a minimalistic network configuration daemon
that supports IPv4 and IPv6 configuration including configuration discovery
through NDP, DHCPv4 and DHCPv6 protocols.

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

[Bug 1255973] Review Request: libndn-cxx - C++ library, implementing Named Data Networking (NDN) primitives

[bugzilla]

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



--- Comment #24 from Antonio Trande  ---
(In reply to Susmit from comment #23)
> I have fixed the directory error but not sure how to fix the 
> 'unused-direct-shlib-dependency' warnings with waf. Any suggestions?

%build
CXXFLAGS="%{optflags} -std=c++11" \
LINKFLAGS="-Wl,--as-needed" \
%{__python2} ./waf --enable-shared --disable-static --with-tests \
 --prefix=%{_prefix} --bindir=%{_bindir} --libdir=%{_libdir} \
 --datadir=%{_datadir} --sysconfdir=%{_sysconfdir} configure

%{__python2} ./waf -v %{?_smp_mflags}


Please, use python2-devel as BuildRequires.

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

[Bug 709328] Review Request: psi-plus - Jabber client based on Qt

[bugzilla]

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



--- Comment #134 from Raphael Groner  ---
Package Review
==

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


Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://fedoraproject.org/wiki/Packaging:Guidelines
=> Ignore, something general seems to be b0rken in my build system.
- gtk-update-icon-cache is invoked in %postun and %posttrans if package
  contains icons.
  Note: icons in psi-plus
  See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
=> Incomplete implementation, please use snippets as given in the link.
- update-desktop-database is invoked in %post and %postun if package
  contains desktop file(s) with a MimeType: entry.
  Note: desktop file(s) with MimeType entry in psi-plus
  See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#desktop-
  database
- Are you sure about licenses? You do not need to mention licenses of stuff
  that doesn't make it into built packages (binary RPM).
 
https://fedoraproject.org/wiki/Licensing:FAQ?rd=Licensing/FAQ#Does_the_License:_tag_cover_the_SRPM_or_the_binary_RPM.3F
- You could use the new %make_build and %make_install macros.
- psi-plus-snapshot has revision 475, maybe you want to update. My experience
  is though 454 was the last one with support for Qt4 only.
- Add desktop-file-validate to %check, Requires: desktop-file-utils
 
https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage
- Please use cp -p to preserve timestamps.
- Please replace %define with %global.
- Please create a subpackage for huge content in /usr/share/psi
  with BuildArch: noarch.
- Output of rpmlint needs to be fixed,
  at least the errors (e.g. E: fsf-incorrect-address).


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[?]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
[x]: Header files in -devel subpackage, if present.
[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.
[?]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "*No copyright* GPL (v2 or later) (with incorrect FSF
 address)", "GPL (v2 or later)", "Unknown or generated", "MIT/X11 (BSD
 like)", "zlib/libpng", "BSD (3 clause)", "GPL (v2.0)", "Beerware",
 "LGPL (v2.1)", "GPL (v2 or later) (with incorrect FSF address)", "LGPL
 (v2.1 or later)", "GPL (v2)". 1602 files have unknown license.
 Detailed output of licensecheck in /home/builder/fedora-review/709328
 -psi-plus/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[!]: Package must own all directories that it creates.
 Note: Directories without known owners:
 /usr/share/icons/hicolor/64x64/apps,
 /usr/share/icons/hicolor/48x48/apps,
 /usr/share/icons/hicolor/32x32/apps,
 /usr/share/icons/hicolor/16x16/apps, /usr/share/icons/hicolor/16x16,
 /usr/share/icons/hicolor/128x128/apps,
 /usr/share/icons/hicolor/128x128, /usr/share/icons/hicolor/64x64,
 /usr/share/icons/hicolor/48x48, /usr/share/icons/hicolor/32x32,
 /usr/share/icons/hicolor
=> Those folders are managed in hicolor-icon-theme, please add as Requires.
https://fedoraproject.org/wiki/Packaging:Guidelines#The_directory_is_also_owned_by_a_package_implementing_required_functionality_of_your_package

[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
=> Provides: bundled(iris), issue is reported to upstream.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: 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 k

[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1272090




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272090
[Bug 1272090] Review Request: nodejs-expect - Write better assertions
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272090] Review Request: nodejs-expect - Write better assertions

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

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




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=1272088
[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable
is a regular expression
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272090] New: Review Request: nodejs-expect - Write better assertions

[bugzilla]

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

Bug ID: 1272090
   Summary: Review Request: nodejs-expect - Write better
assertions
   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-serve/nodejs-serve.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-serve/nodejs-serve-1.4.0-1.fc23.src.rpm
Description: Write better assertions
Fedora Account System Username: jsmith

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

[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic HTTP Middleware for Golang

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Jan Chaloupka  ---
Upstream URL: https://github.com/codegangsta/negroni

Summary:
- license OK
- list of provides ok, no bundled deps
- only one test, passing
- package name and summary ok
- devel subpackage provide, can be [B]R by other golang projects
- unit-test provided as well (for future CI testing)
- no need for debug info as there are no binaries provided

Spec files conforms to the current packaging 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 1270322] Review Request: chromium - A WebKit (Blink) powered web browser

[bugzilla]

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



--- Comment #1 from Tom "spot" Callaway  ---
SPEC URL: https://spot.fedorapeople.org/chromium.spec
SRPM URL: https://spot.fedorapeople.org/chromium-45.0.2454.101-1.fc23.src.rpm

That spec is now cleaned up. In the process of making 45.0.2454.101, upstream
jumped to 46, so I'm going to go start working on that now. Sigh. If the
dependent reviews are somehow finished before that happens, we can review on
45.

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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 CC||lpa...@redhat.com



--- Comment #3 from Jan Chaloupka  ---
*** Bug 1271708 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 1271805] Review Request: golang-github-lpabon-godbc - Design by contract for Go

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jan Chaloupka  ---
Upstream URL: https://github.com/lpabon/godbc

Summary:
- license OK
- list of provides ok, no bundled deps
- only one test, passing
- list of [Build]Requires ok
- package name and summary ok
- devel subpackage provide, can be [B]R by other golang projects
- unit-test provided as well (for future CI testing)

Spec files conforms to the current packaging 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 1271805] Review Request: golang-github-lpabon-godbc - Design by contract for Go

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jchal...@redhat.com
  Flags||fedora-review?



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

[Bug 1271684] Review Request: golang-github-codegangsta-negroni - Idiomatic HTTP Middleware for Golang

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jchal...@redhat.com
  Flags||fedora-review?



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

[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks|1271795 |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1271795
[Bug 1271795] Review Request: nodejs-css-select - A CSS selector
compiler/engine
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1271795] Review Request: nodejs-css-select - A CSS selector compiler/engine

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Depends On|1272088 |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272088
[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable
is a regular expression
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1271708] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||jchal...@redhat.com
 Resolution|--- |DUPLICATE
Last Closed||2015-10-15 09:10:32



--- Comment #3 from Jan Chaloupka  ---
ALready send a SCM request for new package bz#1269813

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

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

[Bug 1271795] Review Request: nodejs-css-select - A CSS selector compiler/engine

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Depends On||1272088




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1272088
[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable
is a regular expression
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272088] Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

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




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=1271795
[Bug 1271795] Review Request: nodejs-css-select - A CSS selector
compiler/engine
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1272088] New: Review Request: nodejs-is-regexp - Check whether a variable is a regular expression

[bugzilla]

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

Bug ID: 1272088
   Summary: Review Request: nodejs-is-regexp - Check whether a
variable is a regular expression
   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-is-regexp/nodejs-is-regexp.spec
SRPM URL: Check whether a variable is a regular expressionDescription: Check
whether a variable is a regular expression
Fedora Account System Username: jsmith

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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



--- Comment #17 from Petr Pisar  ---
I know. I will update the package once FPC updates packaging guidelines. There
is none currently. FeSCO has not yet finished the policy.

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

[Bug 234861] Review Request: perl-Mail-Transport-Dbx - Parse Outlook Express mailboxe

[bugzilla]

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

Petr Pisar  changed:

   What|Removed |Added

 CC||ppi...@redhat.com



--- Comment #9 from Petr Pisar  ---
This package bundles libdbx
.

-- 
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 1272054] New: Review Request: perl-Roman - Functions for converting between Roman and Arabic numerals

[bugzilla]

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

Bug ID: 1272054
   Summary: Review Request: perl-Roman - Functions for converting
between Roman and Arabic numerals
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jples...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://jplesnik.fedorapeople.org/perl-Roman/perl-Roman.spec
SRPM URL:
https://jplesnik.fedorapeople.org/perl-Roman/perl-Roman-1.24-1.fc24.src.rpm

Description:
This package provides some functions which help conversion of numeric
notation between Roman and Arabic.

Fedora Account System Username: jples...@redhat.com

Package is required by perl-HTML-Toc.

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

[Bug 989068] Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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



--- Comment #27 from Paul Howarth  ---
By the way, your python3 package for uri-templates has its %description
cut-and-pasted from oauth2client.

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

[Bug 989068] Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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



--- Comment #26 from Michele Baldessari  ---
Hi Paul,

no not yet, thanks for reminding me ;) I'll get to it today or tomorrow

cheers,
Michele

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about 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

Piotr Popieluch  changed:

   What|Removed |Added

 CC||piotr1...@gmail.com



--- Comment #6 from Piotr Popieluch  ---
FYI: created pull request to include license in pip distribution

https://github.com/influxdb/influxdb-python/pull/257

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

[Bug 1223673] Review Request: llvm35 - The Low Level Virtual Machine

[bugzilla]

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



--- Comment #12 from Jens Petersen  ---
I reported bug 1272001 about the unversioned .so files.
I will try to continue the reviews tomorrow, 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 1271582] Review Request: libbytesize - A library for working with sizes in bytes

[bugzilla]

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



--- Comment #3 from Vratislav Podzimek  ---
(In reply to Šimon Lukašík from comment #2)
> Issues:
> ===
> - Please raise release number each time you update new specfile and srpm
I don't see a point in this. I'm not doing releases. I'm just getting to the
first release.

> - Your upstream URL points to https://github.com/vpodzime/libbytesize/
>   I think it should point out to https://github.com/rhinstaller/libbytesize
>   The latter location has more up-todate sources and the former is just an
>   outdated fork
Oh, I forgot that by doing my own fork, GH stopped redirecting the original URL
to the new one. Fixing.

> - Could not found:
> https://github.com/vpodzime/libbytesize/archive/libbytesize-0.1.tar.gz
>   Please release upstream before I grant the fedora-review+
Sure will do when other things are addressed. I want the first release to
really be the first released one. I don't want to force-push the tag.

> - No documentation included. Please put upstream's README.rst into the base
>   package.
Sounds good, fixing.

> - Python sub-packages should require the base package, or does it work
> without 
>   the lib?
Good point, fixing.

> - Please consider naming python sub-packages python-bytesize instead of
>   python-libbytesize. I don't know the right answer here. Just think about
>   this and let me know.
I don't know the right answer either, but python-bytesize sounds better to me.
Changing.

> - Fix rpmlint issue:
>   libbytesize.x86_64: W: summary-ended-with-dot C A library for working with
> sizes in bytes.
Alright, alright.

> - Aren't here missing dependencies in python sub-packages?
>   I would thing that python3-gobject and pygobject3-base should be required.
>   But I may be wrong, you certainly have more gobject experience. ;-)
You are right, those are needed. :)

> - Similarly, there are directories without owners, in the packages.
>   Please review them and see if you can solve this somehow (my suggestion
>   is to add a dependency.
> * /usr/share/gir-1.0 (-> libbytesize-devel should require GConf2-devel)
Why GConf2-devel? Here's what I have no my system:
# rpm -qf /usr/share/gir-1.0
atk-devel-2.14.0-1.fc21.x86_64
libxklavier-devel-5.4-4.fc21.x86_64
gobject-introspection-devel-1.42.0-1.fc21.x86_64
libgee-devel-0.16.1-1.fc21.x86_64
gtk2-devel-2.24.28-1.fc21.x86_64
vte3-devel-0.36.3-3.fc21.x86_64
gtk3-devel-3.14.15-1.fc21.x86_64
gdk-pixbuf2-devel-2.31.6-1.fc21.x86_64

From those, I think gobject-introspection-devel makes most sense. Adding that.

> * /usr/lib64/girepository-1.0 (-> libbytesize should require
> gobject-introspection)
Yes, adding it.

> 
> This is complete list of issues, I have been able to identify today.
> Overall good job!
Thanks! All of the above should be addressed in the current version of the spec
and SRPM (same URLs).

I also changed the licence of the package (and upstream) to LGPLv2+.

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

[Bug 989068] Review Request: google-api-python-client - Google APIs Client Library for Python

[bugzilla]

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

Paul Howarth  changed:

   What|Removed |Added

 CC||p...@city-fan.org



--- Comment #25 from Paul Howarth  ---
Michele, did you submit a review request for your package?

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

[Bug 1212148] Review Request: python-pyeclib - Python interface to erasure codes

[bugzilla]

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

Kalev Lember  changed:

   What|Removed |Added

 Blocks||1170822
   ||(F23FinalFreezeException,Fi
   ||nalFreezeException)



--- Comment #23 from Kalev Lember  ---
Proposing as a Freeze Exception.

For past three releases we have had a goal to have no broken dependencies in
Fedora base repos. openstack-swift is one of the two remaining packages with
broken deps; the new python-pyeclib package satisfies that broken dependency.
Would be great to pull this in through the freeze as a Freeze Exception to
avoid getting openstack-swift dropped from Fedora.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1170822
[Bug 1170822] Fedora 23 Final freeze exception bug 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 1212148] Review Request: python-pyeclib - Python interface to erasure codes

[bugzilla]

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



--- Comment #22 from Kalev Lember  ---
Looks like the repos got created,
https://admin.fedoraproject.org/pkgdb/package/python-pyeclib/

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

[Bug 1271954] New: Review Request: php-hamcrest - PHP port of Hamcrest Matchers

[bugzilla]

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

Bug ID: 1271954
   Summary: Review Request: php-hamcrest - PHP port of Hamcrest
Matchers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/remicollet/remirepo/20298e564a11ce9fbe5f5ed5e308c2d672f57a62/php/php-hamcrest/php-hamcrest.spec
SRPM URL: http://rpms.famillecollet.com/SRPMS/php-hamcrest-1.2.2-1.remi.src.rpm
Description: 
Hamcrest is a matching library originally written for Java,
but subsequently ported to many other languages.

php-hamcrest is the official PHP port of Hamcrest and essentially follows
a literal translation of the original Java API for Hamcrest,
with a few Exceptions, mostly down to PHP language barriers.

To use this library, you just have to add, in your project:
  require_once '/usr/share/php/Hamcrest/autoload.php';

Fedora Account System Username: remi

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

[Bug 1271954] Review Request: php-hamcrest - PHP port of Hamcrest Matchers

[bugzilla]

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

Remi Collet  changed:

   What|Removed |Added

  Alias||hamcrest/hamcrest-php



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