Bug 1364186

Summary: Review Request: nodejs-log-driver - A simple logging framework for logging to stdout
Product: [Fedora] Fedora Reporter: Jared Smith <jsmith.fedora>
Component: Package ReviewAssignee: Jeroen van Meeuwen <vanmeeuwen+fedora>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: jsmith.fedora, package-review
Target Milestone: ---Flags: vanmeeuwen+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: 2016-08-09 17:16:40 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: 1356348    
Bug Blocks: 956806, 1364187, 1387531    

Description Jared Smith 2016-08-04 16:14:56 UTC 
Spec URL: https://jsmith.fedorapeople.org/Packaging/nodejs-log-driver/nodejs-log-driver.spec
SRPM URL: https://jsmith.fedorapeople.org/Packaging/nodejs-log-driver/nodejs-log-driver-1.2.5-1.fc26.src.rpm
Description: A simple logging framework for logging to stdout
Fedora Account System Username: jsmith
Comment 1 Jeroen van Meeuwen 2016-08-05 09:51:14 UTC 
Jared owes me yet another beer, or Bug #1362491
Comment 2 Jeroen van Meeuwen 2016-08-05 12:18:13 UTC 
I'm mixing up bug numbers ... :/

This package does not seem to include any license file, mention or headers. Can you clarify where the ISC license tag comes from?
Comment 3 Jared Smith 2016-08-06 14:19:04 UTC 
Sorry, the license tag comes both from the LICENSE file in the master branch, and the update to the package.json file in the master branch.  I've fixed up the spec file to pull in the LICENSE file from master, to avoid confusion.

Spec URL: https://jsmith.fedorapeople.org/Packaging/nodejs-log-driver/nodejs-log-driver.spec
SRPM URL: https://jsmith.fedorapeople.org/Packaging/nodejs-log-driver/nodejs-log-driver-1.2.5-2.fc26.src.rpm
Comment 4 Gwyn Ciesla 2016-08-07 01:42:30 UTC 
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/nodejs-log-driver
Comment 5 Jared Smith 2016-08-09 17:16:40 UTC 
In rawhide, closing bug