[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082332

T.C. Hollingsworth  changed:

   What|Removed |Added

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



--- Comment #2 from T.C. Hollingsworth  ---
Looking at the names in %{_bindir} I'd guess this will conflict with the
original tiny-lr.  But that upstream seems really dead, so I'm happy to ship
this.

I'd suggest contacting upstream and let them know they can steal the real
tiny-lr name if that upstream is really dead, following the advice in 'npm help
disputes':
https://www.npmjs.org/doc/misc/npm-disputes.html

The note about the tests kinda freaks me out a little, are we too old or they?

Nothing blocking here though, so 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 1082339] Review Request: nodejs-grunt-git-authors - A Grunt module to generate a list of authors from git history

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082339

T.C. Hollingsworth  changed:

   What|Removed |Added

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



--- Comment #1 from T.C. Hollingsworth  ---
This module shells out to the git command, so it needs:
Requires: git

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

[Bug 1074242] Review Request: perl-X11-GUITest - Provides GUI testing/interaction routines

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1074242

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-X11-GUITest-0.28-1.fc2
   ||0
 Resolution|--- |ERRATA
Last Closed||2014-03-30 22:12:44



--- Comment #12 from Fedora Update System  ---
perl-X11-GUITest-0.28-1.fc20 has been pushed to the Fedora 20 stable
repository.

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

[Bug 1078950] Review Request: perl-Test-Modern - Precision testing for modern perl

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1078950

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
perl-Test-Modern-0.005-1.fc20 has been pushed to the Fedora 20 testing
repository.

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

[Bug 1058174] Review Request: ghc-aeson-pretty - JSON pretty-printing library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1058174

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
ghc-aeson-pretty-0.7.1-1.fc19 has been pushed to the Fedora 19 testing
repository.

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

[Bug 1077956] Review Request: perl-Image-SubImageFind - Perl extension for locating a sub-image within an image

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1077956

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Image-SubImageFind-0.0
   ||3-1.fc20
 Resolution|--- |ERRATA
Last Closed||2014-03-30 22:10:05



--- Comment #11 from Fedora Update System  ---
perl-Image-SubImageFind-0.03-1.fc20 has been pushed to the Fedora 20 stable
repository.

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

[Bug 1073978] Review Request: photocollage - An image assembler with a Gtk GUI

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073978

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|photocollage-1.0.2-1.fc19   |photocollage-1.0.2-1.fc20



--- Comment #19 from Fedora Update System  ---
photocollage-1.0.2-1.fc20 has been pushed to the Fedora 20 stable repository.

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

[Bug 1074482] Review Request: perl-Net-Twitter-Lite - Perl interface to the Twitter API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1074482

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Net-Twitter-Lite-0.120
   ||06-1.fc20
 Resolution|--- |ERRATA
Last Closed||2014-03-30 22:09:58



--- Comment #9 from Fedora Update System  ---
perl-Net-Twitter-Lite-0.12006-1.fc20 has been pushed to the Fedora 20 stable
repository.

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

[Bug 1073978] Review Request: photocollage - An image assembler with a Gtk GUI

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073978

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||photocollage-1.0.2-1.fc19
 Resolution|NEXTRELEASE |ERRATA



--- Comment #18 from Fedora Update System  ---
photocollage-1.0.2-1.fc19 has been pushed to the Fedora 19 stable repository.

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

[Bug 1036755] Review Request: python-httpretty - HTTP client mock for Python

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1036755

Jamie Lennox  changed:

   What|Removed |Added

 Depends On||1082400



--- Comment #25 from Jamie Lennox  ---
Interesting that's not coming from my package. 

I've traced it back to a change in the testing library of python-sure and i've
filed a bug against the package:
https://bugzilla.redhat.com/show_bug.cgi?id=1082400

I'll see what i can do to fix it upstream. The author is the same as the author
of httpretty and he seems to have a habit of incorrectly packaging this for
pypi.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082400
[Bug 1082400] Release has dependency on non-packaged rednose
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082401] New: Review Request: nodejs-lazystream - Create streams lazily when they are read from or written to

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082401

Bug ID: 1082401
   Summary: Review Request: nodejs-lazystream - Create streams
lazily when they are read from or written to
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: tchollingswo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec: http://patches.fedorapeople.org/js-future/SPECS/nodejs-lazystream.spec
SRPM:
http://patches.fedorapeople.org/js-future/SRPMS/nodejs-lazystream-0.1.0-1.fc20.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6691077
FAS:  patches
Description:
Create streams lazily when they are read from or written to.

% rpmlint SPECS/nodejs-lazystream.spec
SRPMS/nodejs-lazystream-0.1.0-1.fc20.src.rpm 
1 packages and 1 specfiles checked; 0 errors, 0 warnings.

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

[Bug 1082401] Review Request: nodejs-lazystream - Create streams lazily when they are read from or written to

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082401

T.C. Hollingsworth  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 1034413] Review Request: SDL2_ttf - TrueType font rendering library for SDL2

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1034413

Paulo Andrade  changed:

   What|Removed |Added

 CC||paulo.cesar.pereira.de.andr
   ||a...@gmail.com



--- Comment #8 from Paulo Andrade  ---
I think you forgot to ask for the repository and build it :-)

I am making some experimental SDL bindings, and did choose
SDL2, so for now using the srpm from the review for SDL2_ttf.

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

[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Tom Hughes  ---
Looks good, and tests pass. 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 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328



--- Comment #2 from Jamie Nguyen  ---
Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-websocket-driver.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-websocket-driver-0.3.2-1.fc21.src.rpm

* Sun Mar 30 2014 Jamie Nguyen  - 0.3.2-2
- include test files as separate source

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

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Jamie Nguyen  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Jamie Nguyen  ---
New Package SCM Request
===
Package Name: nodejs-noptify
Short Description: A wrapper for the nopt module with a commander-like API
Owners: jamielinux patches
Branches: f19 f20 el6 
InitialCC:

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

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Jamie Nguyen  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #6 from Jamie Nguyen  ---
New Package SCM Request
===
Package Name: nodejs-multimatch
Short Description: Adds multiple patterns support to minimatch.match()
Owners: jamielinux patches
Branches: f19 f20 el6
InitialCC:

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

[Bug 1078588] Review Request: ts - Task Spooler

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1078588



--- Comment #3 from Jean-Marie Renouard  ---
Thanks for your review, I will take all remarks into account during this week.

Jean-Marie

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

[Bug 1076825] Review Request: python-fudge - A Python module for using fake objects (mocks and stubs) to test real ones

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076825



--- Comment #7 from Athmane Madjoudj  ---
With more strict python2 macros:

SPEC: http://athmane.fedorapeople.org/pkgs/python-fudge.spec
SRPM: http://athmane.fedorapeople.org/pkgs/python-fudge-1.0.3-3.fc20.src.rpm

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

[Bug 1080669] Review Request: vertica-python - A native Python adapter for the Vertica database

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1080669

Jakub Jedelsky  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #12 from Jakub Jedelsky  ---
New Package SCM Request
===
Package Name: vertica-python
Short Description: A native Python adapter for the Vertica database
Owners: kubo 
Branches: f19 f20 el6 epel7

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

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Tom Hughes  ---
I guess in that case it's not worth pursuing, and it's only a SHOULD so package
is 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 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Tom Hughes  ---
Make sure you use the newer spec from the srpm, but other than that the package
is 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 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330



--- Comment #3 from Tom Hughes  ---
Package Review
==

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



= MUST items =

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

= SHOULD items =

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

= EXTRA items =

Generic:
[!]: Spec file according to URL is the same as in SRPM.
 Note: Spec file as given by url is not the same as in SRPM (see attached
 diff).
 See: (this test has no URL)
[x]: Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).


Rpmlint
---
Checking: nodejs-noptify-0.0.3-2.fc21.noa

[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328



--- Comment #1 from Tom Hughes  ---
Package Review
==

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

Issues
==

[!]: %check is present and all tests pass.

Although it is present the tests are not included in the archive
so I haven't been able to run them.


= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. No licenses
 found. Please check the source files for licenses manually.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 3 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]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

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

= EXTRA items =

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


Rpmlint
---
Chec

[Bug 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #21 from Fedora Update System  ---
libykneomgr-0.1.2-1.el6 has been pushed to the Fedora EPEL 6 testing
repository.

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

[Bug 977124] Review Request: nodejs-grunt-contrib-nodeunit - Run Nodeunit unit tests with grunt

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=977124

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|nodejs-grunt-contrib-nodeun |nodejs-grunt-contrib-nodeun
   |it-0.3.2-1.fc20 |it-0.3.2-1.el6



--- Comment #12 from Fedora Update System  ---
nodejs-grunt-contrib-nodeunit-0.3.2-1.el6 has been pushed to the Fedora EPEL 6
stable repository.

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

[Bug 1082349] Review Request: nodejs-testswarm - A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Jamie Nguyen  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |nodejs-testswarm -A Node.js |nodejs-testswarm - A
   |module for interacting with |Node.js module for
   |TestSwarm   |interacting with TestSwarm
  Flags||fedora-cvs?



--- Comment #3 from Jamie Nguyen  ---
New Package SCM Request
===
Package Name: nodejs-testswarm
Short Description: A Node.js module for interacting with TestSwarm
Owners: jamielinux patches
Branches: f19 f20 el6
InitialCC:

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

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Jamie Nguyen  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Jamie Nguyen  ---
New Package SCM Request
===
Package Name: nodejs-exit
Short Description: A process.exit alternative that ensures STDIO are fully
drained before exiting
Owners: jamielinux patches
Branches: f19 f20 el6 
InitialCC:

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

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342



--- Comment #4 from Jamie Nguyen  ---
(In reply to Tom Hughes from comment #2)
> [!]: If the source package does not include license text(s) as a separate
> file
>  from upstream, the packager SHOULD query upstream to include it.

I opened a whole bunch of pull requests for a lot of this person's other
modules and they were all closed. The reason was "don't want clutter".

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

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330



--- Comment #2 from Jamie Nguyen  ---
Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-noptify.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-noptify-0.0.3-2.fc21.src.rpm

* Sun Mar 30 2014 Jamie Nguyen  - 0.0.3-2
- add BR: npm(nopt)

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

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342



--- Comment #3 from Jamie Nguyen  ---
(In reply to Tom Hughes from comment #1)
> This has a dependency on lodash 2.4.1 but 1.3.1 is the current version in
> rawhide.

I've posted updates for a handful of packages earlier today but must not have
reached mirrors yet:

https://admin.fedoraproject.org/updates/nodejs-findup-sync-0.1.3-1.fc20,nodejs-globule-0.2.0-1.fc20,nodejs-grunt-0.4.1-3.fc20,nodejs-lodash-2.4.1-2.fc20,nodejs-gaze-0.5.1-2.fc20

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

[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330



--- Comment #1 from Tom Hughes  ---
This is missing a BR on nopt.

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

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337



--- Comment #1 from Tom Hughes  ---
Package Review
==

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



= MUST items =

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

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: 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]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in GL).
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: Spec use %global instead of %define unless justified.

= EXTRA items =

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


Rpmlint
---
Checking: nodejs-exit-0.1.2-1.fc21.noarch.rpm
  nodejs-exit-0.1.2-1.fc2

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Tom Hughes  ---
Looks good. 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 1076825] Review Request: python-fudge - A Python module for using fake objects (mocks and stubs) to test real ones

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076825



--- Comment #6 from Athmane Madjoudj  ---
Thank you for your feedback, here's a new one:

SPEC: http://athmane.fedorapeople.org/pkgs/python-fudge.spec
SRPM: http://athmane.fedorapeople.org/pkgs/python-fudge-1.0.3-2.fc20.src.rpm

%changelog
* Sun Mar 30 2014 Athmane Madjoudj   1.0.3-2
- Specify python version in the build requirement
- Remove unused macros
- Enable the test suite

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

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1082341] Review Request: nodejs-shelljs - Portable Unix shell commands for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082341



--- Comment #1 from Tom Hughes  ---
Package Review
==

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

Issues
==

[!]: License field in the package spec file matches the actual license.

- License is BSD not MIT.
- Some fragments (eg src.cp/js) under MIT though.

[!]: Package contains no bundled libraries without FPC exception.

- Both src/cp.js and src/rm.js has code "adapted" from wrench-js

[!]: Rpmlint is run on all installed packages.

nodejs-shelljs.noarch: W: no-manual-page-for-binary shjs

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

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[x]: SourceX tarball generation or download is documented.
 Note: Package contains tarball without URL, check comments
[-]: 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]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in GL).
[x]: No file requires outsid

[Bug 1082341] Review Request: nodejs-shelljs - Portable Unix shell commands for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082341

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342



--- Comment #2 from Tom Hughes  ---
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

[!]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.


= MUST items =

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

= SHOULD items =

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

= EXTRA items =

Generic:
[!]: Rpmlint is run on all installed packages.
 Note: Mock build fail

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342



--- Comment #1 from Tom Hughes  ---
This has a dependency on lodash 2.4.1 but 1.3.1 is the current version in
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 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1082349] Review Request: nodejs-testswarm -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Tom Hughes  ---
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 1082349] Review Request: nodejs-testswarm -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349



--- Comment #1 from Tom Hughes  ---
Package Review
==

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



= MUST items =

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

= SHOULD items =

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

= EXTRA items =

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


Rpmlint
---
Checking: nodejs-testswarm-1.1.0-1.fc21.noarch.rpm
  nodejs-testswarm-1.1.0-1.fc21.src.rpm
nodejs-testswarm.noarch: W: spelling-error Summary(en_US) js -> d

[Bug 1082349] Review Request: nodejs-testswarm -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Tom Hughes  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||t...@compton.nu
   Assignee|nob...@fedoraproject.org|t...@compton.nu
  Flags||fedora-review?



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

[Bug 1076825] Review Request: python-fudge - A Python module for using fake objects (mocks and stubs) to test real ones

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076825



--- Comment #5 from Mukundan Ragavan  ---
Sorry! Copy paste bungled! :)

Yes, that's correct. See
https://fedoraproject.org/wiki/Packaging:Python#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 1076825] Review Request: python-fudge - A Python module for using fake objects (mocks and stubs) to test real ones

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076825



--- Comment #4 from Athmane Madjoudj  ---

Thank you for reviewing this package.

(In reply to Mukundan Ragavan from comment #3)
> Sorry for the delay in review. Please take a look below - 
> 

> [!]: Package meets the Packaging Guidelines::Python
> 
> ---> BuildRequires does not have 
> 


It seems that the message was truncated, I'm guessing that I need to specify
which version of python should be used (python2 vs. python3)

Is this assumption correct ?

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

[Bug 1036755] Review Request: python-httpretty - HTTP client mock for Python

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1036755



--- Comment #24 from Dridi Boukelmoune  ---
Overall, the spec looks fine, but it's missing at least one BuildRequires:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6689180

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

[Bug 1082349] Review Request: nodejs-testswarm -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Jamie Nguyen  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 1082349] New: Review Request: -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Bug ID: 1082349
   Summary: Review Request:  -A Node.js
module for interacting with TestSwarm
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-testswarm.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-testswarm-1.1.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A Node.js module for interacting with TestSwarm.

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

[Bug 1082349] Review Request: nodejs-testswarm -A Node.js module for interacting with TestSwarm

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082349

Jamie Nguyen  changed:

   What|Removed |Added

Summary|Review Request:  -A   |nodejs-testswarm -A Node.js
   |Node.js module for  |module for interacting with
   |interacting with TestSwarm  |TestSwarm



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

[Bug 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260

Nick Bebout  changed:

   What|Removed |Added

  Flags|needinfo?(nb@fedoraproject. |
   |org)|



--- Comment #20 from Nick Bebout  ---
Andy, feel free to apply for commit acl in pkgdb.  I will approve it.  

I have made all the changes mentioned in 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 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260



--- Comment #19 from Fedora Update System  ---
libykneomgr-0.1.2-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/libykneomgr-0.1.2-1.el6

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

[Bug 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260



--- Comment #18 from Fedora Update System  ---
libykneomgr-0.1.2-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/libykneomgr-0.1.2-1.fc19

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

[Bug 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260



--- Comment #17 from Fedora Update System  ---
libykneomgr-0.1.2-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/libykneomgr-0.1.2-1.fc20

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

[Bug 1076260] Review Request: libykneomgr - Yubico YubiKey NEO Manager C Library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076260

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 1082343] Review Request: nodejs-load-grunt-tasks - Load multiple grunt tasks using globbing patterns

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082343

Jamie Nguyen  changed:

   What|Removed |Added

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




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=1082342
[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns
support to minimatch.match()
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082343




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082343
[Bug 1082343] Review Request: nodejs-load-grunt-tasks - Load multiple grunt
tasks using globbing patterns
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082342] Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Jamie Nguyen  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 1082343] New: Review Request: nodejs-load-grunt-tasks - Load multiple grunt tasks using globbing patterns

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082343

Bug ID: 1082343
   Summary: Review Request: nodejs-load-grunt-tasks - Load
multiple grunt tasks using globbing patterns
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-load-grunt-tasks.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-load-grunt-tasks-0.4.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Load multiple grunt tasks using globbing patterns.

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

[Bug 1082342] New: Review Request: nodejs-multimatch - Adds multiple patterns support to minimatch.match()

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082342

Bug ID: 1082342
   Summary: Review Request: nodejs-multimatch - Adds multiple
patterns support to minimatch.match()
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-multimatch.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-multimatch-0.1.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Adds multiple patterns support to minimatch.match().

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

[Bug 1082341] Review Request: nodejs-shelljs - Portable Unix shell commands for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082341

Jamie Nguyen  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 1082341] New: Review Request: nodejs-shelljs - Portable Unix shell commands for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082341

Bug ID: 1082341
   Summary: Review Request: nodejs-shelljs - Portable Unix shell
commands for Node.js
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-shelljs.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-shelljs-0.2.6-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Portable Unix shell commands for Node.js.

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

[Bug 1082340] Review Request: nodejs-grunt-compare-size - Compare file sizes on this branch to master

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082340

Jamie Nguyen  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 1082340] New: Review Request: nodejs-grunt-compare-size - Compare file sizes on this branch to master

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082340

Bug ID: 1082340
   Summary: Review Request: nodejs-grunt-compare-size - Compare
file sizes on this branch to master
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-grunt-compare-size.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-grunt-compare-size-0.4.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Compare file sizes on this branch to master.

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

[Bug 1082339] Review Request: nodejs-grunt-git-authors - A Grunt module to generate a list of authors from git history

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082339

Jamie Nguyen  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 1082339] New: Review Request: nodejs-grunt-git-authors - A Grunt module to generate a list of authors from git history

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082339

Bug ID: 1082339
   Summary: Review Request: nodejs-grunt-git-authors - A Grunt
module to generate a list of authors from git history
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-grunt-git-authors.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-grunt-git-authors-1.2.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A Grunt module to generate a list of authors from git history.

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

[Bug 1082338] Review Request: nodejs-getobject - Get and set deep objects easily

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082338

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1060040




Referenced Bugs:

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

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1060040




Referenced Bugs:

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

[Bug 1082336] Review Request: nodejs-grunt-legacy-util - Deprecated Grunt utils provided for backwards compatibility

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082336

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1060040




Referenced Bugs:

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

[Bug 1082336] Review Request: nodejs-grunt-legacy-util - Deprecated Grunt utils provided for backwards compatibility

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082336

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)
 Depends On||1082338, 1082337




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=1082337
[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that
ensures STDIO are fully drained before exiting
https://bugzilla.redhat.com/show_bug.cgi?id=1082338
[Bug 1082338] Review Request: nodejs-getobject - Get and set deep objects
easily
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082336




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082336
[Bug 1082336] Review Request: nodejs-grunt-legacy-util - Deprecated Grunt
utils provided for backwards compatibility
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082338] Review Request: nodejs-getobject - Get and set deep objects easily

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082338

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082336




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082336
[Bug 1082336] Review Request: nodejs-grunt-legacy-util - Deprecated Grunt
utils provided for backwards compatibility
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082338] Review Request: nodejs-getobject - Get and set deep objects easily

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082338

Jamie Nguyen  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 1082338] New: Review Request: nodejs-getobject - Get and set deep objects easily

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082338

Bug ID: 1082338
   Summary: Review Request: nodejs-getobject - Get and set deep
objects easily
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-getobject.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-getobject-0.1.0-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Get and set deep objects easily.

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

[Bug 1082337] Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Jamie Nguyen  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 1082337] New: Review Request: nodejs-exit - A process.exit alternative that ensures STDIO are fully drained before exiting

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082337

Bug ID: 1082337
   Summary: Review Request: nodejs-exit - A process.exit
alternative that ensures STDIO are fully drained
before exiting
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-exit.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-exit-0.1.2-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A process.exit alternative that ensures STDIO are fully drained before exiting.

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

[Bug 1082336] New: Review Request: nodejs-grunt-legacy-util - Deprecated Grunt utils provided for backwards compatibility

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082336

Bug ID: 1082336
   Summary: Review Request: nodejs-grunt-legacy-util - Deprecated
Grunt utils provided for backwards compatibility
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-grunt-legacy-util.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-grunt-legacy-util-0.1.2-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Deprecated Grunt utils provided for backwards compatibility.

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

[Bug 1082334] Review Request: nodejs-grunt-contrib-watch - Run predefined tasks whenever watched file patterns are added/changed/deleted

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082334

Jamie Nguyen  changed:

   What|Removed |Added

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




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=1082332
[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload
server implementation you can spawn in the background
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082332

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082334




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082334
[Bug 1082334] Review Request: nodejs-grunt-contrib-watch - Run predefined
tasks whenever watched file patterns are added/changed/deleted
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082334] New: Review Request: nodejs-grunt-contrib-watch - Run predefined tasks whenever watched file patterns are added/changed/deleted

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082334

Bug ID: 1082334
   Summary: Review Request: nodejs-grunt-contrib-watch - Run
predefined tasks whenever watched file patterns are
added/changed/deleted
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-grunt-contrib-watch.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-grunt-contrib-watch-0.6.1-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
Run predefined tasks whenever watched file patterns are added/changed/deleted.

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

[Bug 1082329] Review Request: nodejs-faye-websocket - A standards-compliant WebSocket server and client for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082329

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082332




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082332
[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload
server implementation you can spawn in the background
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082332




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082332
[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload
server implementation you can spawn in the background
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082332

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews)
 Depends On||1082329, 1082330




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=1082329
[Bug 1082329] Review Request: nodejs-faye-websocket - A standards-compliant
WebSocket server and client for Node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1082330
[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt
module with a commander-like API
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1079484] Review Request: python-lazy - Lazy attributes for python objects

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079484

David Shea  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from David Shea  ---
New Package SCM Request
===
Package Name: python-lazy
Short Description: Lazy attributes for python objects
Owners: dshea
Branches: f19 f20
InitialCC:

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

[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082332



--- Comment #1 from Jamie Nguyen  ---
Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-tiny-lr-fork.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-tiny-lr-fork-0.0.5-2.fc21.src.rpm

* Sun Mar 30 2014 Jamie Nguyen  - 0.0.5-2
- symlink to _bindir

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

[Bug 1082332] New: Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082332

Bug ID: 1082332
   Summary: Review Request: nodejs-tiny-lr-fork - A tiny
LiveReload server implementation you can spawn in the
background
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-tiny-lr-fork.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-tiny-lr-fork-0.0.5-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A tiny LiveReload server implementation you can spawn in the background.

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

[Bug 1082330] New: Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Bug ID: 1082330
   Summary: Review Request: nodejs-noptify - A wrapper for the
nopt module with a commander-like API
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-noptify.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-noptify-0.0.3-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A wrapper for the nopt module with a commander-like API.

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

[Bug 1082330] Review Request: nodejs-noptify - A wrapper for the nopt module with a commander-like API

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082330

Jamie Nguyen  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 1082329] New: Review Request: nodejs-faye-websocket - A standards-compliant WebSocket server and client for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082329

Bug ID: 1082329
   Summary: Review Request: nodejs-faye-websocket - A
standards-compliant WebSocket server and client for
Node.js
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://jamielinux.fedorapeople.org/gruntjs/nodejs-faye-websocket.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-faye-websocket-0.7.2-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
A standards-compliant WebSocket server and client for Node.js.

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

[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328

Jamie Nguyen  changed:

   What|Removed |Added

 Blocks||1082329




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1082329
[Bug 1082329] Review Request: nodejs-faye-websocket - A standards-compliant
WebSocket server and client for Node.js
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082329] Review Request: nodejs-faye-websocket - A standards-compliant WebSocket server and client for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082329

Jamie Nguyen  changed:

   What|Removed |Added

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




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=1082328
[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol
handler with pluggable I/O for Node.js
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1082328] Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328

Jamie Nguyen  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 1082328] New: Review Request: nodejs-websocket-driver - WebSocket protocol handler with pluggable I/O for Node.js

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082328

Bug ID: 1082328
   Summary: Review Request: nodejs-websocket-driver - WebSocket
protocol handler with pluggable I/O for Node.js
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jamieli...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://jamielinux.fedorapeople.org/gruntjs/nodejs-websocket-driver.spec
SRPM URL:
http://jamielinux.fedorapeople.org/gruntjs/SRPMS/nodejs-websocket-driver-0.3.2-1.fc21.src.rpm
Fedora Account System Username: jamielinux

Description:
WebSocket protocol handler with pluggable I/O for Node.js.

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

[Bug 1048493] Review Request: icecat - GNU version of Firefox browser

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1048493



--- Comment #25 from Antonio Trande  ---
New build:

Spec URL: http://sagitter.fedorapeople.org/Icecat/icecat.spec
SRPM URL: http://sagitter.fedorapeople.org/Icecat/icecat-24.0-8.fc20.src.rpm

Koji build for f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6687444 

Most but not all bundled files are removed; i hope that i can manage more
easily the others in the new release.
Built browser and xulrunner.

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

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=804125



--- Comment #25 from Gianluca Sforna  ---
(In reply to Christopher Meng from comment #24)
> 1. F20+ has solved this bug, you can drop these:
> 
> # we don't want to provide private python extension libs, snippet from:
> # http://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering
> %{?filter_setup:
> %filter_provides_in %{python_sitearch}/.*\.so$ 
> %filter_setup
> }

great. Removed


> 
> 2. %package devel
> Summary:Development files for %{name}
> Group:  Development/Libraries
> Requires:   %{name}%{?_isa} = %{version}-%{release}
> Requires:   cmake
> Requires:   python2-devel
> Requires:   boost-devel
> 
> %{?_isa} should be used for all Requires.

Are you sure? The relevant page does not mention it at all
https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

> 
> 3. A software suite for chemical informatics, computational chemistry, 
> and predictive modeling
> 
> Missing a dot.
> 

Fixed

> 4. %{python_sitearch} --> %{python2_sitearch}

Fixed

> 
> 5. All Group tags can be dropped.

Fixed

> 
> 6. Project has moved to github, with a new version:
> 
> https://github.com/rdkit/rdkit/releases

Changed

Updated spec and srpm:
http://giallu.fedorapeople.org/rdkit-2013.09.2-1.fc20.src.rpm
http://giallu.fedorapeople.org/rdkit.spec

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

[Bug 1078950] Review Request: perl-Test-Modern - Precision testing for modern perl

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1078950

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |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 1047804] Review Request: ghc-regex-base - Haskell regex base library

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047804



--- Comment #7 from Fedora Update System  ---
ghc-async-2.0.1.4-30.fc20, ghc-html-1.0.1.2-28.fc20, ghc-HTTP-4000.2.8-32.fc20,
ghc-HUnit-1.2.5.2-31.fc20, ghc-network-2.4.1.2-31.fc20,
ghc-parsec-3.1.3-30.fc20, ghc-QuickCheck-2.6-32.fc20,
ghc-regex-base-0.93.2-28.fc20, ghc-regex-posix-0.95.2-29.fc20,
ghc-transformers-0.3.0.0-33.fc20, haskell-platform-2013.2.0.0-33.fc20 has been
submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/ghc-async-2.0.1.4-30.fc20,ghc-html-1.0.1.2-28.fc20,ghc-HTTP-4000.2.8-32.fc20,ghc-HUnit-1.2.5.2-31.fc20,ghc-network-2.4.1.2-31.fc20,ghc-parsec-3.1.3-30.fc20,ghc-QuickCheck-2.6-32.fc20,ghc-regex-base-0.93.2-28.fc20,ghc-regex-posix-0.95.2-29.fc20,ghc-transformers-0.3.0.0-33.fc20,haskell-platform-2013.2.0.0-33.fc20

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

[Bug 1069070] Review Request: ghc-QuickCheck - Automatic testing of Haskell programs

2014-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069070



--- Comment #7 from Fedora Update System  ---
ghc-async-2.0.1.4-30.fc20, ghc-html-1.0.1.2-28.fc20, ghc-HTTP-4000.2.8-32.fc20,
ghc-HUnit-1.2.5.2-31.fc20, ghc-network-2.4.1.2-31.fc20,
ghc-parsec-3.1.3-30.fc20, ghc-QuickCheck-2.6-32.fc20,
ghc-regex-base-0.93.2-28.fc20, ghc-regex-posix-0.95.2-29.fc20,
ghc-transformers-0.3.0.0-33.fc20, haskell-platform-2013.2.0.0-33.fc20 has been
submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/ghc-async-2.0.1.4-30.fc20,ghc-html-1.0.1.2-28.fc20,ghc-HTTP-4000.2.8-32.fc20,ghc-HUnit-1.2.5.2-31.fc20,ghc-network-2.4.1.2-31.fc20,ghc-parsec-3.1.3-30.fc20,ghc-QuickCheck-2.6-32.fc20,ghc-regex-base-0.93.2-28.fc20,ghc-regex-posix-0.95.2-29.fc20,ghc-transformers-0.3.0.0-33.fc20,haskell-platform-2013.2.0.0-33.fc20

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

  1   2   >