[Bug 1287756] Review Request: copy-jdk-configs - JDK config files copier

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
copy-jdk-configs-1.1-2.el6 has been submitted as an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-c7f9f6130e

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

[Bug 1287756] Review Request: copy-jdk-configs - JDK config files copier

[bugzilla]

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



--- Comment #24 from Fedora Update System  ---
copy-jdk-configs-1.1-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-02d81c02d9

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

[Bug 1273874] Review Request: nodejs-balanced-match - Match balanced character pairs

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 CC||piotr1...@gmail.com
 Resolution|--- |RAWHIDE
Last Closed||2015-12-17 03:18:53



--- Comment #9 from Piotr Popieluch  ---
Package in Rawhide, closing to unblock bugs.

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

[Bug 1273882] Review Request: nodejs-brace-expansion - Brace expansion as known from sh/bash

[bugzilla]

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

Bug 1273874 Summary: Review Request: nodejs-balanced-match - Match balanced 
character pairs
https://bugzilla.redhat.com/show_bug.cgi?id=1273874

   What|Removed |Added

 Status|ON_QA   |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 1273882] Review Request: nodejs-brace-expansion - Brace expansion as known from sh/bash

[bugzilla]

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

Bug 1273880 Summary: Review Request: nodejs-concat-map - concatenative 
mapdashery
https://bugzilla.redhat.com/show_bug.cgi?id=1273880

   What|Removed |Added

 Status|ON_QA   |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 1273880] Review Request: nodejs-concat-map - concatenative mapdashery

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 CC||piotr1...@gmail.com
 Resolution|--- |RAWHIDE
Last Closed||2015-12-17 03:19:04



--- Comment #12 from Piotr Popieluch  ---
Package in Rawhide, closing to unblock bugs.

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

[Bug 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
perl-Crypt-Salsa20-0.03-2.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 1285428] Review Request: perl-Crypt-ScryptKDF - Scrypt password based key derivation function

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-12-17 03:52: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 1282911] Review Request: perl-Crypt-Salsa20 - Encrypt data with the Salsa20 cipher

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-12-17 03:52: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 1285428] Review Request: perl-Crypt-ScryptKDF - Scrypt password based key derivation function

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
perl-Crypt-ScryptKDF-0.009-2.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 1292237] Review Request: nodejs-array-index - Invoke getter/setter functions on array-like objects

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

 CC||piotr1...@gmail.com



--- Comment #4 from Piotr Popieluch  ---
I just approved a package with the license in %doc and %license

The purpose of using %license was to be able to ship Fedora without
documentation (to save disk space on atomic hosts) but with license. 

If the license is in the readme and readme is only in %doc, deleting the docs
would violate the license.

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

[Bug 1292237] Review Request: nodejs-array-index - Invoke getter/setter functions on array-like objects

[bugzilla]

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

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Tom Hughes  ---
Like I say I had been reading "in its own file" to mean a file just containing
the license but I guess it could be interpreted more widely.

I don't think it's a major problem whichever way things go, so 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 1292181] Review Request: nodejs-difflib - Text diff library ported from Python's difflib module

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||piotr1...@gmail.com
   Assignee|nob...@fedoraproject.org|piotr1...@gmail.com
  Flags||fedora-review+



--- Comment #2 from Piotr Popieluch  ---
APPROVED

Readme is marked as doc and license, which should be correct. We can ignore the
issue.



Package Review
==

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


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

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 11 files have unknown license. Detailed
 output of licensecheck in /home/piotr/rpmbuild/1292181-nodejs-
 difflib/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.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 30720 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]: 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

= 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.
[x]: Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Packager, Vendor, PreReq, 

[Bug 1288100] Review Request: console-image-viewer - Terminal image viewer

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
console-image-viewer-1.2-1.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 1288843] Review Request: nodejs-is-typedarray - Detect whether or not an object is a Typed Array

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
nodejs-is-typedarray-1.0.0-1.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 1282640] Review Request: lastpass-cli - Command line interface to LastPass.com

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
lastpass-cli-0.7.0-2.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 1288930] Review Request: abi-tracker - Tool to visualize ABI changes timeline of a C/C++ library

[bugzilla]

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



--- Comment #19 from Fedora Update System  ---
abi-tracker-1.4-2.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 1286395] Review Request: libqtxdg-qt4 - a Qt4 implementation of XDG standards

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
libqtxdg-qt4-1.2.0-10.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 1287756] Review Request: copy-jdk-configs - JDK config files copier

[bugzilla]

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



--- Comment #26 from Fedora Update System  ---
copy-jdk-configs-1.1-1.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 1292376] New: Review Request: perl-Object-Remote - Call methods on objects in other processes or on other hosts

[bugzilla]

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

Bug ID: 1292376
   Summary: Review Request: perl-Object-Remote - Call methods on
objects in other processes or on other hosts
   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-Object-Remote/perl-Object-Remote.spec
SRPM URL:
https://jplesnik.fedorapeople.org/perl-Object-Remote/perl-Object-Remote-0.003005-1.fc24.src.rpm

Description:
Object::Remote allows you to create an object in another process - usually
one running on another machine you can connect to via ssh, although there
are other connection mechanisms available.

Fedora Account System Username: jples...@redhat.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 1208101] Review Request: libbson - Building, parsing, and iterating BSON documents

[bugzilla]

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

Daniel Kopeček  changed:

   What|Removed |Added

  Flags|needinfo?(dkopecek@redhat.c |
   |om) |



--- Comment #21 from Daniel Kopeček  ---
(In reply to Petr Pisar from comment #19)
> Daniel, would accept the libbson with bundled modified yajl? Guidelines do
> not forbid bundling anymore, but FPC has not yet come with guidelines how to
> bundle.

Yes, I'll accept it.

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

[Bug 1273882] Review Request: nodejs-brace-expansion - Brace expansion as known from sh/bash

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

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



--- Comment #2 from Piotr Popieluch  ---
This needs to be updated to 1.1.2

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

[Bug 1273882] Review Request: nodejs-brace-expansion - Brace expansion as known from sh/bash

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



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

[Bug 1286844] Review Request: nodejs-propagate - Propagate events

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
nodejs-propagate-0.3.1-1.fc22 has been pushed to the Fedora 22 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 nodejs-propagate'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-15467790d6

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

[Bug 1291459] Review Request: sysreporter - Basic system reporter with emailing

[bugzilla]

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



--- Comment #16 from Fedora Update System  ---
sysreporter-3.0.0-0.4.alpha4.fc22 has been pushed to the Fedora 22 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 sysreporter'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-af43634f50

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

[Bug 1278960] Review Request: nodejs-heap - Binary heap (priority queue) algorithms

[bugzilla]

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



--- Comment #13 from Fedora Update System  ---
nodejs-heap-0.2.6-4.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 nodejs-heap'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-e5b84a222f

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

[Bug 1292181] Review Request: nodejs-difflib - Text diff library ported from Python's difflib module

[bugzilla]

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

Bug 1278960 Summary: Review Request: nodejs-heap - Binary heap (priority queue) 
algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=1278960

   What|Removed |Added

 Status|CLOSED  |ON_QA
 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 1290172] Review Request: nodejs-verror - Richer JavaScript errors

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
nodejs-verror-1.6.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 nodejs-verror'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-0dc9eee3f6

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

[Bug 1291539] Review Request: nodejs-has-unicode - Try to guess if your terminal supports unicode

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System  ---
nodejs-has-unicode-2.0.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 nodejs-has-unicode'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-6eb8e15cfd

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

[Bug 1291392] Review Request: nodejs-is-builtin-module - Check if a string matches the name of a Node.js builtin module

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
nodejs-is-builtin-module-1.0.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 nodejs-is-builtin-module'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-8dadc3b0c7

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

[Bug 1291392] Review Request: nodejs-is-builtin-module - Check if a string matches the name of a Node.js builtin module

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
nodejs-is-builtin-module-1.0.0-1.fc22 has been pushed to the Fedora 22 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 nodejs-is-builtin-module'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-a5747a956b

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

[Bug 1284942] Review Request: nodejs-lex-parser - A parser for lexical grammars used by jison

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
nodejs-lex-parser-0.1.4-1.fc22 has been pushed to the Fedora 22 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 nodejs-lex-parser'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-d100b82ad6

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

[Bug 1284945] Review Request: nodejs-jison-lex - Lexical analyzer generator used by jison

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
nodejs-jison-lex-0.3.4-1.fc22 has been pushed to the Fedora 22 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 nodejs-jison-lex'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-c90201c019

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

[Bug 1278364] Review Request: python-whitenoise - Static file serving for Python web apps

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #16 from Fedora Update System  ---
python-whitenoise-2.0.6-2.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-whitenoise'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-705afcd6ef

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

[Bug 1274738] Review Request: nodejs-nth-check - A performant nth-check parser & compiler

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
nodejs-nth-check-1.0.1-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 nodejs-nth-check'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-98f63a2e28

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

[Bug 1278960] Review Request: nodejs-heap - Binary heap (priority queue) algorithms

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---
   Keywords||Reopened



--- Comment #12 from Fedora Update System  ---
nodejs-heap-0.2.6-4.fc22 has been pushed to the Fedora 22 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 nodejs-heap'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-471e1ca190

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

[Bug 1292061] Review Request: perl-POE-Loop-Event - Bridge that allows POE to be driven by Event.pm

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jples...@redhat.com
   Assignee|nob...@fedoraproject.org|jples...@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 1292121] Review Request: perl-Eval-WithLexicals - Pure Perl eval with persistent lexical variables

[bugzilla]

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

Petr Šabata  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  Flags||fedora-review?



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

[Bug 1284945] Review Request: nodejs-jison-lex - Lexical analyzer generator used by jison

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
nodejs-jison-lex-0.3.4-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 nodejs-jison-lex'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-2c1ba7bc38

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

[Bug 1286844] Review Request: nodejs-propagate - Propagate events

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
nodejs-propagate-0.3.1-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 nodejs-propagate'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-258028b781

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

[Bug 1284942] Review Request: nodejs-lex-parser - A parser for lexical grammars used by jison

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
nodejs-lex-parser-0.1.4-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 nodejs-lex-parser'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-daf60f2001

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

[Bug 1292376] Review Request: perl-Object-Remote - Call methods on objects in other processes or on other hosts

[bugzilla]

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

Petr Šabata  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  Flags||fedora-review?



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

[Bug 1292400] Review Request: nodejs-next-tick - Environment agnostic nextTick polyfill

[bugzilla]

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

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 1292376] Review Request: perl-Object-Remote - Call methods on objects in other processes or on other hosts

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 Depends On||1292121




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292121
[Bug 1292121] Review Request: perl-Eval-WithLexicals - Pure Perl eval with
persistent lexical variables
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292392] Review Request: libpwiz - ProteoWizard software library

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

 Whiteboard||NotReady



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

[Bug 1292121] Review Request: perl-Eval-WithLexicals - Pure Perl eval with persistent lexical variables

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 Blocks||1292376




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292376
[Bug 1292376] Review Request: perl-Object-Remote - Call methods on objects
in other processes or on other hosts
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292237] Review Request: nodejs-array-index - Invoke getter/setter functions on array-like objects

[bugzilla]

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



--- Comment #6 from Piotr Popieluch  ---
Yes, you are correct, that is what it literally says. It just doesn't make
sense to me when I take the whole purpose of the %license tag in account. 

I've sent a mail to packaging list, maybe this will clear things up.

https://lists.fedoraproject.org/archives/list/packaging%40lists.fedoraproject.org/thread/7AZ6YYIGRC53E4S3KHJLDOUCNTAGSXEV/

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

[Bug 1292392] New: Review Request: libpwiz - ProteoWizard software library

[bugzilla]

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

Bug ID: 1292392
   Summary: Review Request: libpwiz - ProteoWizard software
library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: anto.tra...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://sagitter.fedorapeople.org/libpwiz/libpwiz.spec
SRPM URL:
https://sagitter.fedorapeople.org/libpwiz/libpwiz-3.0.9205-1.fc23.src.rpm

Description: 
The ProteoWizard Library and Tools are a set of modular and extensible
open-source, cross-platform tools and software libraries that facilitate
proteomics data analysis.

Fedora Account System Username: sagitter

Note: this package is only for 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 1292061] Review Request: perl-POE-Loop-Event - Bridge that allows POE to be driven by Event.pm

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jitka Plesnikova  ---
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

$ rpm -qp --requires perl-POE-Loop-Event-1.305-1.fc24.noarch.rpm | sort | uniq
-c
  1 perl(:MODULE_COMPAT_5.22.1)
  1 perl(Event) >= 1.21
  1 perl(POE) >= 1.356
  1 perl(POE::Loop::PerlSignals)
  1 perl(strict)
  1 perl(vars)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -qp --provides perl-POE-Loop-Event-1.305-1.fc24.noarch.rpm | sort | uniq
-c
  1 perl(POE::Loop::Event) = 1.305
  1 perl-POE-Loop-Event = 1.305-1.fc24
Binary provides are Ok.

$ rpmlint ./perl-POE-Loop-Event*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Rpmlint is ok

BuildRequires are ok.

TODO: perl(POE::Test::Loops) should be part of the first group BRs, because
  it is used in Makefile.PL

Please consider fixing 'TODO' item.

The package looks good
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 1292400] New: Review Request: nodejs-next-tick - Environment agnostic nextTick polyfill

[bugzilla]

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

Bug ID: 1292400
   Summary: Review Request: nodejs-next-tick - Environment
agnostic nextTick polyfill
   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-next-tick/nodejs-next-tick-0.2.2-2.fc24.src.rpm
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-next-tick/nodejs-next-tick-0.2.2-2.fc24.src.rpm
Description: Environment agnostic nextTick polyfill
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 1292057] Review Request: perl-DBIx-QueryLog - Logging queries for DBI

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jples...@redhat.com
   Assignee|nob...@fedoraproject.org|jples...@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 1284400] Review Request: koji-containerbuild - Koji support for building layered container images

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
koji-containerbuild-0.5.5-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-18264539b3

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

[Bug 1290924] Review Request: grive2 - Google Drive client

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
grive2-0.4.2-0.1.20151208gitcc13b8b.el7 has been submitted as an update to
Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-99b18948f5

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

[Bug 1290924] Review Request: grive2 - Google Drive client

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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 1292425] Review Request: nodejs-es6-weak-map - ECMAScript6 WeakMap polyfill

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292429




Referenced Bugs:

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

[Bug 1284400] Review Request: koji-containerbuild - Koji support for building layered container images

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
koji-containerbuild-0.5.5-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-fae06c3c21

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

[Bug 1290924] Review Request: grive2 - Google Drive client

[bugzilla]

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



--- Comment #7 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/grive2

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

[Bug 1292237] Review Request: nodejs-array-index - Invoke getter/setter functions on array-like objects

[bugzilla]

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



--- Comment #7 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-array-index

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

[Bug 1292181] Review Request: nodejs-difflib - Text diff library ported from Python's difflib module

[bugzilla]

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



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-difflib

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

[Bug 1282893] Review Request: quantum-espresso - A suite for electronic-structure calculations and materials modeling

[bugzilla]

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



--- Comment #9 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to marcindulak from comment #8)
> (In reply to Zbigniew Jędrzejewski-Szmek from comment #2)
> > Hm, what does this do?:
> > 
> > %if 0%{?el6}
> > %global mpich mpich
> > %global mpich_load %_mpich_load
> > %global mpich_unload %_mpich_unload
> > %else
> > %global mpich mpich
> > %global mpich_load %_mpich_load
> > %global mpich_unload %_mpich_unload
> > %endif
> > 
> 
> this is a legacy part when there was mpich2 on el6 and mpich on Fedora.
> I removed that global variables.
> 
> > 
> > Requires are normally written with one =. I think == might be 
> > misinterpreted.
> 
> OK
> 
> > 
> > You can replace the calls to `basename ${f}` with ${f%/*}, makes things
> > slightly shorter and more readable.
> 
> let's keep the simple basename. I'm not a fan of shortcuts.
> 
> > 
> > Please use pushd/popd instead of cd/cd -. This is recommended because the
> > path is printed, which makes logs easier to understand.
> > 
> 
> OK
> 
> > %defattr(-,root,root,-) is not needed.
> 
> i believe fedora-review complains if %defattr(-,root,root,-) is missing
Quite the opposite ;) Anyway the authoritative source is
https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions,
which says that they are not required.

> > Use %{!?_licensedir:%global license %doc} and then you don't need
> > conditionals in %files.
> > 
> 
> this works for me when defining this global *after* the License field.
> Apparently License is confused with license if global defined on the top of
> the spec:
> http://koji.fedoraproject.org/koji/getfile?taskID=12125886=build.
> log=-4000
Right.

> > Can you comment on the reason for multiple source tarballs? Are they always
> > released together?
> 
> I would like we clarify two things:
> 
> 1. the source includes L. Peter Deutsch md5 implementation. I think gromacs
> RPM
> also includes this md5 (src/external/tng_io/src/lib/md5.c) so we are fine?
> 
> 2. quantum-espresso bundles a small part (a couple of files) belonging to
> http://www.tddft.org/programs/octopus/wiki/index.php/Libxc
> This is described in this commit message:
> http://qe-forge.org/gf/project/q-e/scmsvn/
> ?action=ScmCommitDetail_commit_id=16950
> 
> I have made an additional change: set ExclusiveArch due to openblas-devel.
> 
> Spec URL:
> https://marcindulak.fedorapeople.org/packages/quantum-espresso/r02/quantum-
> espresso.spec
> SRPM URL:
> https://marcindulak.fedorapeople.org/packages/quantum-espresso/r02/quantum-
> espresso-5.2.1-1.el6.src.rpm

I'll try to review this later today.

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

[Bug 1292416] Review Request: nodejs-es5-ext - ECMAScript 5 extensions and ES6 shims

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292428




Referenced Bugs:

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

[Bug 1292428] Review Request: nodejs-lru-queue - LRU queue

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)
 Depends On||1292416




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=1292416
[Bug 1292416] Review Request: nodejs-es5-ext - ECMAScript 5 extensions and
ES6 shims
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 955996] Review Request: wkhtmltopdf-qt - Customized version of the QT toolkit for generating PDFs using Web-kit

[bugzilla]

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



--- Comment #15 from Frank Ch. Eigler  ---
If QtWebKit is abandoned, then this application's copy of a modified QtWebKit
is no longer a "fork".  It is the upstream's decision how maintainable they
deem it, not ours.

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

[Bug 505154] Tracker: Review Requests for Science and Technology related packages

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

 Depends On||1292392




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292392
[Bug 1292392] Review Request: libpwiz - ProteoWizard software library
-- 
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 1292057] Review Request: perl-DBIx-QueryLog - Logging queries for DBI

[bugzilla]

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



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/perl-DBIx-QueryLog

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

[Bug 1292424] Review Request: nodejs-d - Property descriptor factory

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292429




Referenced Bugs:

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

[Bug 1292428] Review Request: nodejs-lru-queue - LRU queue

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292429




Referenced Bugs:

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

[Bug 1292415] Review Request: nodejs-timers-ext - Timers extension

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292429




Referenced Bugs:

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

[Bug 1292426] Review Request: nodejs-event-emitter - Environment agnostic event emitter

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292429




Referenced Bugs:

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

[Bug 1292429] Review Request: nodejs-memoizee -- Memoize/cache function results

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)
 Depends On||1292424, 1292425, 1292426,
   ||1292428, 1292415




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=1292415
[Bug 1292415] Review Request: nodejs-timers-ext - Timers extension
https://bugzilla.redhat.com/show_bug.cgi?id=1292424
[Bug 1292424] Review Request: nodejs-d - Property descriptor factory
https://bugzilla.redhat.com/show_bug.cgi?id=1292425
[Bug 1292425] Review Request: nodejs-es6-weak-map - ECMAScript6 WeakMap
polyfill
https://bugzilla.redhat.com/show_bug.cgi?id=1292426
[Bug 1292426] Review Request: nodejs-event-emitter - Environment agnostic
event emitter
https://bugzilla.redhat.com/show_bug.cgi?id=1292428
[Bug 1292428] Review Request: nodejs-lru-queue - LRU queue
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292429] New: Review Request: nodejs-memoizee -- Memoize/cache function results

[bugzilla]

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

Bug ID: 1292429
   Summary: Review Request: nodejs-memoizee -- Memoize/cache
function results
   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-memoizee/nodejs-memoizee.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-memoizee/nodejs-memoizee-0.3.9-2.fc24.src.rpm
Description: Memoize/cache function results
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 1284400] Review Request: koji-containerbuild - Koji support for building layered container images

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
koji-containerbuild-0.5.5-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-5b67cbb3ff

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

[Bug 1284400] Review Request: koji-containerbuild - Koji support for building layered container images

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |MODIFIED



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

[Bug 1292061] Review Request: perl-POE-Loop-Event - Bridge that allows POE to be driven by Event.pm

[bugzilla]

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



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/perl-POE-Loop-Event

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

[Bug 1288541] Review Request: perl-Code-TidyAll - Engine for tidyall, your all-in-one code tidier and validator

[bugzilla]

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



--- Comment #2 from Petr Šabata  ---
* You list the Test::Class::Most build time dependency twice.

* Some of the listed build time dependencies aren't actually necessary, namely:
  - autodie
  - IPC::System::Simple
  - List::Compare
  - Log::Any
  - Test::Builder

* One build time dependency missing: lib

* The package description is fairly poorly written and doesn't say much.

* The summary implies this package is just the engine, even though it also
  installs the utility it refers to.
  You could keep that and split the utility into a subpackage, if you think
  it makes sense.  Or change the summary.  Your call.

* Many of the plugins require external utilities you neither require nor
  recommend/suggest.  The package won't work as advertised without them.
  See the Code::TidyAll::Plugin namespace.

* Possibly similar situation with the VCSs -- will this work without git or
  subversion?  Maybe -- I don't know how these hooks are meant to be 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 1290924] Review Request: grive2 - Google Drive client

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
grive2-0.4.2-0.1.20151208gitcc13b8b.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-aa77fce813

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

[Bug 955996] Review Request: wkhtmltopdf-qt - Customized version of the QT toolkit for generating PDFs using Web-kit

[bugzilla]

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



--- Comment #13 from Kevin Kofler  ---
Ewww, please no! A fork of the whole Qt is entirely unmaintainable!

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

[Bug 1292149] Review Request: nodejs-cli-color - Colors, formatting and other tools for the console

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Depends On||1292429, 1292424, 1292416




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292416
[Bug 1292416] Review Request: nodejs-es5-ext - ECMAScript 5 extensions and
ES6 shims
https://bugzilla.redhat.com/show_bug.cgi?id=1292424
[Bug 1292424] Review Request: nodejs-d - Property descriptor factory
https://bugzilla.redhat.com/show_bug.cgi?id=1292429
[Bug 1292429] Review Request: nodejs-memoizee -- Memoize/cache function
results
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292424] Review Request: nodejs-d - Property descriptor factory

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292149




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292149
[Bug 1292149] Review Request: nodejs-cli-color - Colors, formatting and
other tools for the console
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292429] Review Request: nodejs-memoizee -- Memoize/cache function results

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292149




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292149
[Bug 1292149] Review Request: nodejs-cli-color - Colors, formatting and
other tools for the console
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292416] Review Request: nodejs-es5-ext - ECMAScript 5 extensions and ES6 shims

[bugzilla]

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

Jared Smith  changed:

   What|Removed |Added

 Blocks||1292149




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1292149
[Bug 1292149] Review Request: nodejs-cli-color - Colors, formatting and
other tools for the console
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292291] Review Request: nodejs-unc-path-regex - Regular expression to test if a file path is a windows UNC path

[bugzilla]

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



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-unc-path-regex

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

[Bug 1291007] Review Request: python-sphinxcontrib-fulltoc - include a full table of contents in your Sphinx HTML sidebar

[bugzilla]

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



--- Comment #14 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-sphinxcontrib-fulltoc

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



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-css-select

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

[Bug 1292392] Review Request: libpwiz - ProteoWizard software library

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

 Blocks||505154 (FE-SCITECH)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=505154
[Bug 505154] Tracker: Review Requests for Science and Technology related
packages
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1292057] Review Request: perl-DBIx-QueryLog - Logging queries for DBI

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jitka Plesnikova  ---
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

$ rpm -qp --requires perl-DBIx-QueryLog-0.41-1.fc24.noarch.rpm | sort | uniq -c
  1 perl(:MODULE_COMPAT_5.22.1)
  1 perl(:VERSION) >= 5.8.0
  1 perl(DBI)
  1 perl(Data::Dumper)
  1 perl(Term::ANSIColor)
  1 perl(Text::ASCIITable)
  1 perl(Time::HiRes)
  1 perl(constant)
  1 perl(strict)
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -qp --provides perl-DBIx-QueryLog-0.41-1.fc24.noarch.rpm | sort | uniq -c
  1 perl(DBIx::QueryLog) = 0.41
  1 perl-DBIx-QueryLog = 0.41-1.fc24
Binary provides are Ok.

$ rpmlint ./perl-DBIx-QueryLog*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Rpmlint is ok

BuildRequires:
FIX: Please add BRs
 - perl(parent) - builder/MyBuilder.pm: line 5
 - perl(base) - t/Util.pm: line 7
TODO: Please remove perl(Test::Pod), perl(Test::Pod::Coverage). They are
  used by xt tests, which are not executed.

Please correct all 'FIX' issues and consider fixing 'TODO' items.

Otherwise package looks good.
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 1292057] Review Request: perl-DBIx-QueryLog - Logging queries for DBI

[bugzilla]

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



--- Comment #2 from Denis Fateyev  ---
> - perl(parent) - builder/MyBuilder.pm: line 5
> - perl(base) - t/Util.pm: line 7
Ok, seems now we need these basic pragmas, too ;-)
Thanks for the reviews, I'll fix BRs before import.

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

[Bug 1292428] New: Review Request: nodejs-lru-queue - LRU queue

[bugzilla]

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

Bug ID: 1292428
   Summary: Review Request: nodejs-lru-queue - LRU queue
   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-lru-queue/nodejs-lru-queue.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-lru-queue/nodejs-lru-queue-0.1.0-1.fc24.src.rpm
Description: LRU queue
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 1187030] Review Request: giza - scientific plotting library for C/Fortran

[bugzilla]

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



--- Comment #17 from Joachim Frieben  ---
The new release should resolve the issues raised in comment 11. 
Spec URL: http://frieben.fedorapeople.org/giza.spec
SRPM URL: http://frieben.fedorapeople.org/giza-0.9.4-2.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 1292280] Review Request: nodejs-imurmurhash - An incremental implementation of MurmurHash3

[bugzilla]

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



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-imurmurhash

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

[Bug 1292232] Review Request: nodejs-delegates - Delegate methods and accessors to another property

[bugzilla]

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



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-delegates

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

[Bug 955996] Review Request: wkhtmltopdf-qt - Customized version of the QT toolkit for generating PDFs using Web-kit

[bugzilla]

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



--- Comment #14 from Kevin Kofler  ---
I'd also like to point out that QtWebKit, on which this software relies, is no
longer supported by upstream Qt, and that the replacement (QtWebEngine) is
known to NOT be usable for this use case (printing the HTML to PDF), so this
program is effectively a dead end.

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

[Bug 1290172] Review Request: nodejs-verror - Richer JavaScript errors

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
nodejs-verror-1.6.0-1.el7 has been pushed to the Fedora EPEL 7 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-verror'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-970fb2586d

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

[Bug 1278960] Review Request: nodejs-heap - Binary heap (priority queue) algorithms

[bugzilla]

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



--- Comment #15 from Fedora Update System  ---
nodejs-heap-0.2.6-4.el7 has been pushed to the Fedora EPEL 7 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-heap'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-312fd5be8c

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

[Bug 1290171] Review Request: nodejs-extsprintf - Extended POSIX-style sprintf

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
nodejs-extsprintf-1.3.0-1.el7 has been pushed to the Fedora EPEL 7 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-extsprintf'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-e5f435ae5e

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

[Bug 1292392] Review Request: libpwiz - ProteoWizard software library

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

 Whiteboard|NotReady|



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

[Bug 1290924] Review Request: grive2 - Google Drive client

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
grive2-0.4.2-0.1.20151208gitcc13b8b.fc22 has been submitted as an update to
Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-6a6d26d832

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

[Bug 1273882] Review Request: nodejs-brace-expansion - Brace expansion as known from sh/bash

[bugzilla]

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

Piotr Popieluch  changed:

   What|Removed |Added

  Flags|fedora-review-  |fedora-review+



--- Comment #5 from Piotr Popieluch  ---
ive updated nodejs-balanced-match in rawhide. Approving as other issues are
minor ones but please fix before uploading.

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

[Bug 1291459] Review Request: sysreporter - Basic system reporter with emailing

[bugzilla]

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



--- Comment #17 from Fedora Update System  ---
sysreporter-3.0.0-0.4.alpha4.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 sysreporter'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-a31396de4f

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

[Bug 1274738] Review Request: nodejs-nth-check - A performant nth-check parser & compiler

[bugzilla]

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



--- Comment #14 from Fedora Update System  ---
nodejs-nth-check-1.0.1-3.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-nth-check'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-32416af0af

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