Bug 1373542 - rh-nodejs4 bundles openssl
Summary: rh-nodejs4 bundles openssl
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Software Collections
Classification: Red Hat
Component: rh-nodejs4
Version: rh-nodejs4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: alpha
: 2.2
Assignee: Zuzana Svetlikova
QA Contact: BaseOS QE - Apps
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-06 14:41 UTC by Matthias Runge
Modified: 2016-09-08 06:26 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-09-08 06:26:49 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Matthias Runge 2016-09-06 14:41:49 UTC
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Honza Horak 2016-09-08 05:56:44 UTC
This topic showed up on Tuesday's SCLo SIG meeting and I recommend Matthias to report it here. However, I checked quickly the nodejs package and the openssl seems to be removed even from the source, as this comment describes that:

# nodejs bundles openssl, but we use the system version in Fedora
# because openssl contains prohibited code, we remove openssl completely from
# the tarball, using the script in Source100

Matthias, why do you think openssl is bundled in nodejs SCL?

Comment 2 Matthias Runge 2016-09-08 06:22:37 UTC
At first glance, I couldn't find something in the spec to make sure bundled openssl is not linked and distributed[1] and from own experience, you'll need a patch to make node work with rhel7s version of openssl, eg. this here [2]


[1] https://github.com/nodejs/node/tree/v4.x/deps/openssl
[2] http://cbs.centos.org/koji/rpminfo?rpmID=63516

Comment 3 Matthias Runge 2016-09-08 06:26:49 UTC
OK, it looks like I looked at the wrong spec file.

This came up as follow-up question to the request to upgrade node to verrsion 6.


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