Bug 1433793

Summary: Review Request: nodejs-touch - Like touch in node
Product: [Fedora] Fedora Reporter: Honza Horak <hhorak>
Component: Package ReviewAssignee: Jared Smith <jsmith.fedora>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: hhorak, jsmith.fedora, package-review
Target Milestone: ---Flags: jsmith.fedora: fedora-review+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-04-30 13:16:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1433764, 1497464, 1571001    

Description Honza Horak 2017-03-19 23:09:51 UTC
Spec URL: https://hhorak.fedorapeople.org/nodemon/nodejs-touch.spec
SRPM URL: https://hhorak.fedorapeople.org/nodemon/nodejs-touch-1.0.0-5.fc25.src.rpm

Description:
For all your node touching needs.

scratch build of nodejs-touch-1.0.0-5.fc25.src.rpm for f26: http://koji.fedoraproject.org/koji/taskinfo?taskID=18354186

Fedora Account System Username: hhorak

Comment 1 Jared Smith 2017-09-30 17:44:42 UTC
Please add the following line to the %check section, to make sure all the dependencies are in place:

%{__nodejs} -e 'require("./")'

Also, please update to the latest 3.1.0 release.  Otherwise, the package is approved.

Comment 2 Jared Smith 2017-09-30 17:50:26 UTC
Also, please remember to update the script interpreter in bin/touch.js

Comment 3 Jared Smith 2018-04-23 12:05:00 UTC
Are you going to finish this package review?  Otherwise, I'd like to submit my own (updated) review for consideration.

Comment 4 Honza Horak 2018-04-30 13:16:29 UTC
Sorry, please, feel free to submit your own (updated) review for consideration. Closing this one.

Comment 5 Jared Smith 2018-04-30 15:26:36 UTC

*** This bug has been marked as a duplicate of bug 1573248 ***