Skip to content

v4.8.3 proposal #12499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MylesBorins merged 15 commits into from
May 2, 2017
Merged

v4.8.3 proposal #12499

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6d3aaa7
tls: TLSSocket emits 'error' on handshake failure
lekoder Sep 27, 2016
07b92a3
doc: add supported platforms list for v4.x
mhdawson Mar 17, 2017
1dc6b38
test: add test for loading from global folders
richardlau Oct 20, 2016
ba91c41
module: fix loading from global folders on Windows
richardlau Oct 25, 2016
99749dc
tls: keep track of stream that is closed
jBarz Mar 9, 2017
ff1d61c
stream_base,tls_wrap: notify on destruct
trevnorris Oct 31, 2016
4426080
Partial revert "tls: keep track of stream that is closed"
trevnorris Mar 20, 2017
54f5258
tls: fix segfault on destroy after partial read
bnoordhuis Mar 17, 2017
ab3fdf5
deps: cherry-pick ca0f9573 from V8 upstream
ofrobots Mar 20, 2017
8ed18a1
src: ensure that fd 0-2 are valid on windows
bzoz Mar 14, 2017
7a1920d
test: add hasCrypto check to tls-socket-close
danbev Mar 8, 2017
eb393f9
src: fix base64 decoding
seishun Mar 22, 2017
b5b78b1
src: add fcntl.h include to node.cc
bzoz Apr 20, 2017
6040efd
test: fix flaky test-tls-wrap-timeout
Trott Jul 23, 2016
788f280
2017-05-02, Version 4.8.3 'Argon' (Maintenance)
MylesBorins Apr 19, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
module: fix loading from global folders on Windows 
Code was calculating $PREFIX/lib/node relative to process.execPath, but
on Windows process.execPath is $PREFIX\node.exe whereas everywhere else
process.execPath is $PREFIX/bin/node (where $PREFIX is the root of the
installed Node.js).

PR-URL: #9283
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
@richardlau @MylesBorins
richardlau authored and MylesBorins committed Apr 19, 2017
commit ba91c41478bfe3f3c07b56b8534a6c676ef6ae3d
11 changes: 10 additions & 1 deletion lib/module.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -453,7 +453,16 @@ Module._initPaths = function() {
homeDir = process.env.HOME;
}

var paths = [path.resolve(process.execPath, '..', '..', 'lib', 'node')];
// $PREFIX/lib/node, where $PREFIX is the root of the Node.js installation.
var prefixDir;
// process.execPath is $PREFIX/bin/node except on Windows where it is
// $PREFIX\node.exe.
if (isWindows) {
prefixDir = path.resolve(process.execPath, '..');
} else {
prefixDir = path.resolve(process.execPath, '..', '..');
}
var paths = [path.resolve(prefixDir, 'lib', 'node')];

if (homeDir) {
paths.unshift(path.resolve(homeDir, '.node_libraries'));
Expand Down