Bug 1478718

Summary: http_parser not deleted during the srpm build
Product: Red Hat Software Collections Reporter: Cedric Buissart <cbuissar>
Component: nodejsAssignee: Zuzana Svetlikova <zsvetlik>
Status: CLOSED ERRATA QA Contact: BaseOS QE - Apps <qe-baseos-apps>
Severity: low Docs Contact:
Priority: unspecified    
Version: rh-nodejs6   
Target Milestone: ---   
Target Release: 3.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1478899 (view as bug list) Environment:
Last Closed: 2017-11-20 11:40:44 UTC Type: Bug
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: 1478899    

Description Cedric Buissart 2017-08-06 14:39:35 UTC 
Description of problem:
As shipped in RHSCL, the nodejs building process is supposed to pre-emptively delete unused dependencies from the deps/ directory.

However, there is a tiny typo for the http-parser dependency in the srpm spec file :

---8<----
142 rm -rf deps/npm \
143        deps/uv \
144        deps/http-parser \
145        deps/zlib
--->8----

The directory name is http_parser (underscore, not hyphen) :
$ ls node-v4.6.2/deps/
cares  gtest  http_parser  v8

Resulting in the deps/http_parser/ directory not being deleted as expected.

.... It does not change anything to the end result, but might be worth reporting ☺