[Bug 1082332] Review Request: nodejs-tiny-lr-fork - A tiny LiveReload server implementation you can spawn in the background
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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()
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
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
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
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()
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
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
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
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
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
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
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
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()
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
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()
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
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
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
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
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
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
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
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
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
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()
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()
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()
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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()
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()
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
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()
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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