Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1120366 - Review Request: nodejs-set-immediate - A shim for the setImmediate API
Summary: Review Request: nodejs-set-immediate - A shim for the setImmediate API
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: nodejs-reviews 1120117
TreeView+ depends on / blocked
 
Reported: 2014-07-16 19:13 UTC by Tom Hughes
Modified: 2014-07-17 17:31 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-07-17 17:31:17 UTC
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Tom Hughes 2014-07-16 19:13:20 UTC
Spec URL: http://download.compton.nu/nodejs/nodejs-set-immediate.spec
SRPM URL: http://download.compton.nu/nodejs/nodejs-set-immediate-0.1.1-1.fc20.src.rpm
Fedora Account System Username: tomh

Description:
A simple and lightweight shim for the setImmediate W3C Draft API, for
use in any browsers and NodeJS.

Comment 1 Parag AN(पराग) 2014-07-17 03:13:08 UTC
[!]: 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)

Diff spec file in url and in SRPM
---------------------------------
--- /home/parag/1120366-nodejs-set-immediate/srpm/nodejs-set-immediate.spec     2014-07-17 08:37:20.243230431 +0530
+++ /home/parag/1120366-nodejs-set-immediate/srpm-unpacked/nodejs-set-immediate.spec    2014-07-16 23:46:42.000000000 +0530
@@ -7,7 +7,7 @@
 URL:            https://github.com/jussi-kalliokoski/setImmediate.js
 Source0:        http://registry.npmjs.org/set-immediate/-/set-immediate-%{version}.tgz
-# https://github.com/jussi-kalliokoski/setImmediate.js/pull/5
+#
 Source1:        nodejs-set-immediate-license.txt
-# https://github.com/jussi-kalliokoski/setImmediate.js/pull/4
+#
 Patch0:         nodejs-set-immediate-test.patch
 BuildArch:      noarch


Please fix above issue for further review.

Comment 2 Tom Hughes 2014-07-17 07:50:06 UTC
I've rebuilt the srpm with the updated spec:

Spec URL: http://download.compton.nu/nodejs/nodejs-set-immediate.spec
SRPM URL: http://download.compton.nu/nodejs/nodejs-set-immediate-0.1.1-1.fc20.src.rpm

Comment 3 Parag AN(पराग) 2014-07-17 09:21:08 UTC
Review:

+ mock build is successful for f22

+ rpmlint output on rpms gave
nodejs-set-immediate.noarch: W: spelling-error Summary(en_US) setImmediate -> set Immediate, set-immediate, immediateness
nodejs-set-immediate.noarch: W: spelling-error %description -l en_US setImmediate -> set Immediate, set-immediate, immediateness
nodejs-set-immediate.noarch: W: only-non-binary-in-usr-lib
nodejs-set-immediate.src: W: spelling-error Summary(en_US) setImmediate -> set Immediate, set-immediate, immediateness
nodejs-set-immediate.src: W: spelling-error %description -l en_US setImmediate -> set Immediate, set-immediate, immediateness
2 packages and 0 specfiles checked; 0 errors, 5 warnings.

==> Not a issue

+ Source verified with upstream as (sha256sum)
srpm tarball     :f30ca71c7b81a9d32163e09cd1aa40d83f7772c6de9f416d3a9d374419dda4d2
upstream tarball :f30ca71c7b81a9d32163e09cd1aa40d83f7772c6de9f416d3a9d374419dda4d2

+ License "MIT" is valid and included in separate text file as patch

+ rest looks Ok

Suggestion:
1) cp command in %prep should preserve timestamp though its not a upstream file.

APPROVED.

Comment 4 Tom Hughes 2014-07-17 12:45:38 UTC
New Package SCM Request
=======================
Package Name: nodejs-set-immediate
Short Description: A shim for the setImmediate API
Owners: tomh jamielinux
Branches: f19 f20 f21
InitialCC:

Comment 5 Gwyn Ciesla 2014-07-17 15:48:23 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.