Commits · 156cd82ef4d2ff4fa291407de562c3a7c2386dc7 · github.com / v8-riscv / node

Jan 09, 2015

win,nodevars: run iojs.exe to get node version · 156cd82e

Bert Belder authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

156cd82e

win,msi: rename node.exe to iojs.exe · 0430215d

Bert Belder authored Jan 08, 2015

This change is strictly limited to the name of the binary. The shortcut
name, install folder, docs website links etc. are all unchanged.

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

0430215d

win,build: rename node.exe to iojs.exe · 92c5d32d

Bert Belder authored Jan 08, 2015

  * Sign iojs.exe
  * Run license2rtf.js through iojs

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

92c5d32d

win: regenerate perfctr provider files · facc0565

Bert Belder authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

facc0565

win,tracing: node -> iojs in perfctr/etw manifests · eab60039

Bert Belder authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

eab60039

win,src: node -> iojs in executable manifest · 889f4023

Bert Belder authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

889f4023

win,build: remove upload target from vcbuild.bat · 2e40ae67

Bert Belder authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/263


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

2e40ae67

install: install iojs -> node compat symlink · 72f1b348

Ben Noordhuis authored Jan 08, 2015

Install a symlink for compatibility with scripts that expect a binary
called `node` or have a /path/to/node shebang.  Only done on UNIX
platforms because symlinks on Windows require elevated privileges.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

72f1b348

build: rename binary from node to iojs · f17f473a

Ben Noordhuis authored Jan 08, 2015

* rename the build targets

* update the test runner to use `out/{Debug,Release}/iojs`

* update the installer to install the iojs binary

* update one test that explicitly checks for the binary name

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

f17f473a

test: remove --use-http1 test runner flag · 3e7a25dd

Ben Noordhuis authored Jan 08, 2015

The flag was used for a short while during the v0.5 development cycle,
four years ago.  Remove it, it's long overdue.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

3e7a25dd

doc: update v8 options in man page · 57a2cab7

Ben Noordhuis authored Jan 08, 2015

Update the outdated list of V8 options in the man page.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

57a2cab7

doc: s/node/iojs/ in iojs.1 · 62bd7110

Ben Noordhuis authored Jan 02, 2015

The binary is about to be renamed from `node` to `iojs`; preemptively
update the man page.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

62bd7110

doc: rename node.1 -> iojs.1 · 3f8b76eb

Ben Noordhuis authored Jan 02, 2015

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

3f8b76eb

build: remove upload targets from makefile · 73b93234

Ben Noordhuis authored Jan 02, 2015

The targets upload to nodejs.org but that is the joyent/node website.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

73b93234

build: don't run make recursively · e3f86103

Ben Noordhuis authored Jan 02, 2015

For targets that need the binary: don't invoke `make all` as part of the
recipe, just depend on the $(NODE_EXE) target.

PR-URL: https://github.com/iojs/io.js/pull/262


Reviewed-By: Bert Belder <bertbelder@gmail.com>

e3f86103

win,test: fix test-process-active-wraps.js · b5c9dcb6

Colin Ihrig authored Jan 08, 2015

b636ba81 broke this test, because it now takes a loop iteration or two
to resolve the loopback address. That consequence is that the TCPWrap
handle that we *don't* want to see is created a bit later, and also
destroyed later, so when we assert that the active handle list is empty
the TCPWrap object is still "busy" being closed.

Wait one extra loop iteration before checking there are no more active
handles. This allows name resolution and clean-up to finish before the
assertion.

BUG: https://github.com/iojs/io.js/issues/246
PR-URL: https://github.com/joyent/node/pull/8998


Reviewed-By: Bert Belder <bertbelder@gmail.com>

b5c9dcb6

dgram: make error messages more informative · c9fd9e21

Evan Lucas authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/250


Reviewed-By: Bert Belder <bertbelder@gmail.com>

c9fd9e21

util: _detailedException to _exceptionWithHostPort · b1a208c8

Evan Lucas authored Jan 08, 2015

The _detailedException() helper function used to be local to the 'net'
module, but now that it has been moved to 'util' a more descriptive name
is desirable.

PR-URL: https://github.com/iojs/io.js/pull/250


Reviewed-By: Bert Belder <bertbelder@gmail.com>

b1a208c8

util,net: move _detailedException into util · 3937e856

Evan Lucas authored Jan 07, 2015

This allows _detailedException() to be used by both the 'net' and
'dgram' modules to provide more informative error messages.

PR-URL: https://github.com/iojs/io.js/pull/250


Reviewed-By: Bert Belder <bertbelder@gmail.com>

3937e856

Jan 08, 2015

version: working on 1.0.0 · 8a0e7d65

Rod Vagg authored Jan 08, 2015

PR-URL: https://github.com/iojs/io.js/pull/253


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

8a0e7d65

src: prefix ARCH and PLATFORM with NODE_ · e1fe270f

Bert Belder authored Jan 08, 2015

The PLATFORM preprocessor symbol is defined in node.gyp, and on Windows
it's set to "win". This conflicts with a built-in preprocessor symbol
with a different value ("win32"), which makes the linker(!) complain.
Resolve this by renaming these symbols to NODE_ARCH and NODE_PLATFORM.

PR-URL: https://github.com/iojs/io.js/pull/261

Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

e1fe270f

win: fix NODE_NET_SOCKET_READ/WRITE signature · 858b5583

Bert Belder authored Jan 08, 2015

The NODE_NET_SOCKET_READ and NODE_NET_SOCKET_WRITE macros are just
no-ops on Windows, but they used to be defined as taking four parameters
while being called with five arguments. Turn them into variadic macros
to squelch a compiler warning.

PR-URL: https://github.com/iojs/io.js/pull/261


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

858b5583

win: suppress GetVersionEx deprecation warning · 09fe6fe1

Bert Belder authored Jan 08, 2015

There is no other way to retrieve the Windows version. The stated reason
this API is deprecated is that applications are not supposed to check
whether the Windows it's running on is recent enough. But that's not
what we use it for.

PR-URL: https://github.com/iojs/io.js/pull/261


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

09fe6fe1

win: use GetVersionExW instead of GetVersionExA · 0ce7b912

Bert Belder authored Jan 08, 2015

For consistency with the rest of the source code, use the wide-char
version of this API.

PR-URL: https://github.com/iojs/io.js/pull/261


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

0ce7b912

win,openssl: disable some warnings · e7056270

Bert Belder authored Jan 08, 2015

This patch disables two (categories of) warnings:

  * deprecation of GetVersionExA
  * possible loss of data in implicit conversion of scalar types

These warnings don't seem to point out serious problems, and avoiding
them in openssl is somebody else's business.

PR-URL: https://github.com/iojs/io.js/pull/261


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

e7056270

test: move broadcast/multicast test to internet/ · 7266b757

Ben Noordhuis authored Jan 08, 2015

Move parallel/test-dgram-broadcast-multi-process to test/internet.

The test does not play nice with firewalls that restrict broadcast
or multicast traffic, nor can it be rewritten to use only loopback
traffic without running into platform-specific limitations, see also
commits 52e600a9 and 236533c (TODO: update second one before landing.)

PR-URL: https://github.com/iojs/io.js/pull/259


Reviewed-By: Rod Vagg <rod@vagg.org>

7266b757

Revert "test: don't assume broadcast traffic is unfiltered" · df3c4ca1

Ben Noordhuis authored Jan 08, 2015

This reverts commit 52e600a9.

Reverted for:

* making the test fail with ENETUNREACH on OS X 10.8, and

* making the test fail with EHOSTDOWN on OS X 10.9 and 10.10 when there
  is no network connectivity, and

* leaving behind orphan processes that make subsequent tests fail with
  EADDRINUSE errors

PR-URL: https://github.com/iojs/io.js/pull/259


Reviewed-By: Rod Vagg <rod@vagg.org>

df3c4ca1

win: disable stl exception usage · 4519db04

Bert Belder authored Jan 08, 2015

Tell the stl that exception handling is unavailable. This avoids warnings
like:

  C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\include\
  xlocale(337): warning C4530: C++ exception handler used, but unwind
  semantics are not enabled. Specify /EHsc

PR-URL: https://github.com/iojs/io.js/pull/258


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

4519db04

win: disable exception handling for all builds · da9eff80

Bert Belder authored Jan 08, 2015

Previously exception handling was disabled in release builds, but
enabled in debug builds. That makes no sense.

PR-URL: https://github.com/iojs/io.js/pull/258


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

da9eff80

util: show symbol properties · c70d192a

Vladimir Kurchatkin authored Jan 08, 2015

Properties with symbol names are shown if option `showHidden` of `util.inspect`
or `console.dir` is `true`.

PR-URL: https://github.com/iojs/io.js/pull/247


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rod Vagg <rod@vagg.org>

c70d192a

src,deps: replace LoadLibrary by LoadLibraryW · 604b8761

Cheng Zhao authored Dec 31, 2014

On Windows, when compiling with `UNICODE` defined, `LoadLibrary` becomes
`LoadLibraryW`. When an ASCII string is passed to that function it
crashes.

PR-URL: https://github.com/iojs/io.js/pull/226


Reviewed-By: Bert Belder <bertbelder@gmail.com>

604b8761

src: pass Isolate to node::Utf8Value constructor · cbf76c1f

Trevor Norris authored Jan 07, 2015

Initial attempt to remove all uses of Isolate::GetCurrent(). Still
exists a few locations, but this works out a heavy usage.

PR-URL: https://github.com/iojs/io.js/pull/244


Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>

cbf76c1f

src: pass Isolate to additional locations · d5533866

Trevor Norris authored Jan 07, 2015

Due to a recent V8 upgrade, more methods require Isolate as an argument.

PR-URL: https://github.com/iojs/io.js/pull/244


Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>

d5533866

test: fix parallel/test-repl with new v8 · 6e9d1c86

Ben Noordhuis authored Jan 07, 2015

Per the ES6 spec, V8 3.31 no longer throws a SyntaxError for scoped
function declarations.

PR-URL: https://github.com/iojs/io.js/pull/243


Reviewed-By: Domenic Denicola <domenic@domenicdenicola.com>
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

6e9d1c86

test: fix parallel/test-repl-harmony with new v8 · 58f3a038

Ben Noordhuis authored Jan 07, 2015

Assignment to a const var in strict mode is a lazy TypeError now.
Update the test accordingly.

PR-URL: https://github.com/iojs/io.js/pull/243


Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

58f3a038

build: fix up build after v8 upgrade · 8a83ebae

Ben Noordhuis authored Jan 07, 2015

* Define and disable new flag `v8_use_external_startup_data`.

* Disable v8_postmortem_support, it's broken again.  Fedor (@indutny)
  has volunteered to fix it up in the near future.

PR-URL: https://github.com/iojs/io.js/pull/243


Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

8a83ebae

deps: upgrade v8 to 3.31.74.1 · dad73f64

Ben Noordhuis authored Jan 07, 2015

PR-URL: https://github.com/iojs/io.js/pull/243


Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

dad73f64

Jan 07, 2015

doc: update AUTHORS, .mailmap, add authors tool · 53ba4945

Rod Vagg authored Jan 02, 2015

PR-URL: https://github.com/iojs/io.js/pull/232


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

53ba4945

test: don't assume broadcast traffic is unfiltered · 52e600a9

Ben Noordhuis authored Dec 29, 2014

parallel/test-dgram-broadcast-multi-process assumes non-local broadcast
traffic is unfiltered, an assumption that fails with, for example, the
default Fedora firewall rules.  Use a loopback interface instead.

Fixes: https://github.com/iojs/io.js/issues/219
PR-URL: https://github.com/iojs/io.js/pull/220


Reviewed-By: Fedor Indutny <fedor@indutny.com>

52e600a9

Jan 06, 2015

deps: update libuv to 1.2.0 · eaed2a11

Ben Noordhuis authored Jan 05, 2015

PR-URL: https://github.com/iojs/io.js/pull/237


Reviewed-By: Bert Belder <bertbelder@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>

eaed2a11