[Bug 1120366] Review Request: nodejs-set-immediate - A shim for the setImmediate API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Parag AN(पराग) <panemade@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |panemade@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |panemade@xxxxxxxxx
              Flags|                            |fedora-review?



--- Comment #1 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
[!]: 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.

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





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]