Commits · ccc91aea35c70fbae152a09253879da96f41dfd9 · github.com / v8-riscv / node

Jan 14, 2015

test: loosen timeout in spawnsync-test for FreeBSD · ccc91aea

Fedor Indutny authored Jan 13, 2015

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


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

ccc91aea

dns: remove AI_V4MAPPED hint flag on FreeBSD · 04bea9f9

Fedor Indutny authored Jan 13, 2015

FreeBSD does not support V4MAPPED.

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


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

04bea9f9

deps: fix v8 build on FreeBSD · b949437c

Fedor Indutny authored Jan 13, 2015

clang++ on FreeBSD was blaming v8 for using invalid casts from nullptr:

    reinterpret_cast from 'nullptr_t' to '...' is not allowed

Replace casts with NULL, or NULL with 0 where applicable.

Fix #324
PR-URL: https://github.com/iojs/io.js/pull/332


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

b949437c

Jan 13, 2015

doc: remove tracing link in toc · fa004f3c

Yosuke Furukawa authored Jan 14, 2015

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


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

fa004f3c

src: limit .gitattributes eol to vcbuild.bat · e25a1751

Rod Vagg authored Jan 13, 2015

Fixes: https://github.com/iojs/io.js/issues/330
PR-URL: https://github.com/iojs/io.js/pull/331

e25a1751

build: initialize variable before use · 5ea716d8

Johan Bergström authored Jan 13, 2015

found with make --warn-undefined-variables

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


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

5ea716d8

build: create smaller build artifacts · 17217c37

Johan Bergström authored Jan 13, 2015

check if xz exists on the host system and use if available, which makes
xz-tarballs available for distribution. also remove deps/zlib/contrib
since it's not in use -- shaves additional size from the tarballs.

also, slightly modify the .gitignore file to ignore these new archives.

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


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

17217c37

test: create test-ci, modify test-all, fix cpplint · 7861ff47

Ben Noordhuis authored Jan 12, 2015

test: add test/addons to default test list

`make test-all` and `python tools/test.py` will now also run the addon
tests in test/addons.

test: remove test-npm from test-all make target

The test-npm target builds documentation, changes versioned files,
clutters the current working directory with artifacts, etc.  In short,
it doesn't seem quite ready for inclusion in `make test-all`.

test: add test-ci target, reduced test-all

Add a test-ci target that is like test-all minus the (slow) pummel and
gc test suites.

This is primarily intended for the continuous integration, where we want
decent coverage but don't want to wait for ages for tests to complete.

cpplint: add -license/copyright to default filters

Commit 3e1b1dd4 ("Remove excessive copyright/license boilerplate") trips
up the copyright boilerplate style check.  Disable it.

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


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

7861ff47

test: use assert.throw to test exceptions · f7672268

Sam Roberts authored Jan 13, 2015

The test wasn't checking directly that an assertion was thrown. Instead,
it was checking that spawn did not sucessfully spawn a non-existent
command.

However, the command chosen, dir, exists in GNU coreutils, so it exists
on Linux (though not on BSD derived OS X). The test as written passed on
Linux, even with the TypeError it is supposed to be checking for deleted
from spawn(). It would also pass on Windows if a ls.exe existed.

The approach is unnecessarily obscure, assert.throw() is for asserting
code throws, using it is more clear and works regardless of what
commands do or do not exist.

PR-URL: https://github.com/joyent/node/pull/8454


Reviewed-by: Trevor Norris <trev.norris@gmail.com>

Cherry-picked-from: https://github.com/joyent/node/commit/2ff29cc7e35f486daf86710fd2db48275442c788

Conflicts:
	test/parallel/test-child-process-spawn-typeerror.js

f7672268

test: check for multi-localhost support · 22d20582

Timothy J Fontaine authored Jan 10, 2015

Cherry-picked-from: https://github.com/joyent/node/commit/2b7c8a2f02c2e132c2dbc59bd05e0e7246e10196

22d20582

sys: Mark as deprecated · dc42e1fa

Wyatt Preul authored Jan 12, 2015

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


Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

discussed at TC meeting, deprecate, don't remove

dc42e1fa

http: optimize on_headers_complete · 0bf1d124

Bert Belder authored Jan 13, 2015

Use an array instead of an object to pass a parsed header chunk from c++
to javascript. This offers a 5-10% speedup on the http_simple benchmark,
as evidenced by running:

  ab -k -t 100 -c 100 http://127.0.0.1:8000/bytes/100

PR: https://github.com/iojs/io.js/pull/292


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

0bf1d124

deps: don't build docs on 'make npm-test' · f468e5f1

Forrest L Norvell authored Jan 12, 2015

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


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

f468e5f1

build,src: remove sslv3 support · 5165d710

Ben Noordhuis authored Jan 13, 2015

SSLv3 is susceptible to downgrade attacks.  Provide secure defaults,
disable v3 protocol support entirely.

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


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

5165d710

test: fix message tests regression · 635337f9

Ben Noordhuis authored Jan 13, 2015

Commit 3e1b1dd4 ("Remove excessive copyright/license boilerplate") broke
some of the message tests because without the license boilerplate at
the top, the line numbers no longer match up.

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


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

635337f9

Remove excessive copyright/license boilerplate · 3e1b1dd4

isaacs authored Jan 12, 2015

The copyright and license notice is already in the LICENSE file.  There
is no justifiable reason to also require that it be included in every
file, since the individual files are not individually distributed except
as part of the entire package.

3e1b1dd4

docs: improve Writable/Transform stream docs · 68a2d8f0

Vladimir Guguiev authored Dec 11, 2014

Slightly fix and supplement the documentation for Writable#write and
Transform#transform methods. Make it more consistent. Mention passing
'buffer' as an encoding param in case chunk is a Buffer.

PR: https://github.com/iojs/io.js/pull/142


Reviewed-by: Bert Belder <bertbelder@gmail.com>
Reviewed-by: Rod Vagg <rod@vagg.org>

68a2d8f0

src: rename node to io.js for Windows installer · 43e4c906

Rod Vagg authored Jan 11, 2015

* quote "NODE_VERSION_STRING" in node.rc to allow for complex version
  strings
* change user-facing strings
* make sure .bat files are crlf

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


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

43e4c906

test: fix `make test-addons` target · dd260d21

Ben Noordhuis authored Jan 12, 2015

I broke the python script in commit edaf7af3 but because test-all does
not run test-addons, it slipped under the radar.  Mea culpa.

Fixes the following test runner error:

    Traceback (most recent call last):
      File "tools/test.py", line 1522, in <module>
        sys.exit(Main())
      File "tools/test.py", line 1454, in Main
        test_list = root.ListTests([], path, context, arch, mode)
      File "tools/test.py", line 720, in ListTests
        test.AddTestsToList(result, full_path, path, context, arch, mode)
      File "tools/test.py", line 690, in AddTestsToList
        arch, mode)
      File "/home/bnoordhuis/src/v1.x/test/gc/../testpy/__init__.py", line 176, in ListTests
        result.append(SimpleTestCase(test, file_path, mode, self.context, self))
    TypeError: __init__() takes at least 7 arguments (6 given)

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


Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>

dd260d21

win,msi: create npm folder in AppData directory · e431cae7

Steven Rockarts authored Jan 12, 2015

Create the empty npm folder in Roaming\Appdata so that non-Administrator
users have a place to store global packages. This fixes the error Error:
ENOENT, stat error that occurs when a user tries to run the npm install
<package> command.

Bug: https://github.com/joyent/node/issues/8141
PR: https://github.com/joyent/node/pull/8838


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

Please enter the commit message for your changes. Lines starting

e431cae7

addon: update NODE_MODULE_VERSION macro · 42ebdcbf

Ben Noordhuis authored Jan 12, 2015

io.js v1.0.0 is not ABI-compatible with joyent/node@v0.12, update
the NODE_MODULE_VERSION macro to reflect that.  Update to a much
larger value to avoid future clashes with joyent/node.

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


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

42ebdcbf

doc: update style for iojs · 9120f2b1

Chris Dickinson authored Jan 11, 2015

* updates the styling for the iojs docs
* pulls the processing step for markdown files into
  a separate module
* adds the ability to insert comments into the markdown

PR-URL: https://github.com/iojs/io.js/pull/297
Fixes: https://github.com/iojs/iojs.github.io/issues/23


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

9120f2b1

fs: undeprecate exists() and existsSync() · 3a85eac4

cjihrig authored Jan 12, 2015

fs.exists() and fs.existsSync() were deprecated in #103. This
commit removes the deprecation message, in order to stay more
in sync with joyent/node for the io.js 1.0.0 release.

Fixes: https://github.com/iojs/io.js/issues/257
PR-URL: https://github.com/iojs/io.js/pull/307


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Bert Belder <bertbelder@gmail.com>

3a85eac4

assert: throw when block is not a function · 14dc9175

cjihrig authored Jan 12, 2015

Currently, anything passed as the block argument to throws()
and doesNotThrow() is interpreted as a function, which can
lead to unexpected results. This commit checks the type of
block, and throws a TypeError if it is not a function.

Fixes: https://github.com/iojs/io.js/issues/275
PR-URL: https://github.com/iojs/io.js/pull/308


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

14dc9175

gyp: remove vanished masm file from openssl build · 7c4a50dd

Fedor Indutny authored Jan 11, 2015

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


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

7c4a50dd

test: fix crypto-stream after openssl update · a76811c8

Fedor Indutny authored Jan 11, 2015

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


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

a76811c8

openssl: fix build failure on windows · ced41b07

Fedor Indutny authored Jan 08, 2015

Backport of: https://github.com/openssl/openssl/commit/5c5e7e



Original commit message:

    Fix build failure on Windows due to undefined cflags identifier

Reviewed-by: Tim Hudson <tjh@openssl.org>

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


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

ced41b07

openssl: fix keypress requirement in apps on win32 · b9106137

Fedor Indutny authored Dec 11, 2013

Original source:

http://openssl.6102.n7.nabble.com/PATCH-s-client-Fix-keypress-requirement-with-redirected-input-on-Windows-td46787.html



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

b9106137

deps: update openssl to 1.0.1k · eebdf7ac

Fedor Indutny authored Jan 11, 2015

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


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

eebdf7ac

test: move sequential/test-debug-port-cluster · c8676cb3

Ben Noordhuis authored Jan 12, 2015

Move sequential/test-debug-port-cluster to test/parallel.  This test
is safe to run in parallel with other debugger tests, it doesn't use
fixed port numbers.

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


Reviewed-By: Miroslav Bajtoš <miroslav@strongloop.com>

c8676cb3

test: move sequential/test-debug-signal-cluster · 3da4f87b

Ben Noordhuis authored Jan 12, 2015

Move sequential/test-debug-signal-cluster to test/parallel.  Per the
previous commit, it can now run in parallel with other debugger tests.

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


Reviewed-By: Miroslav Bajtoš <miroslav@strongloop.com>

3da4f87b

lib: make --debug-port work with cluster · 11c1bae7

Ben Noordhuis authored Jan 12, 2015

Make the cluster module intercept the `--debug-port=<port>` command line
switch and replace it with the debug port of the child process.

A happy coincidence of this change is that it finally makes it possible
to run the sequential/test-debug-signal-cluster in parallel, it now no
longer needs the default port numbers.

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


Reviewed-By: Miroslav Bajtoš <miroslav@strongloop.com>

11c1bae7

test: delete sequential/test-debug-cluster · 5ec5fd83

Ben Noordhuis authored Jan 12, 2015

The test does not work well with concurrent invocations of the test
runner because it uses fixed port numbers. The functionality it tests
is covered by sequential/test-debug-port-cluster, a verbatim copy with
the only difference being that it doesn't use fixed port numbers.

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

Reviewed-By: Miroslav Bajtoš <miroslav@strongloop.com>

5ec5fd83

Jan 12, 2015

doc: add TC meeting 2015-01-07 minutes · f0c6fe53

Rod Vagg authored Jan 08, 2015

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


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

f0c6fe53

doc: clarified & split up contribution docs · c52e43d9

Rod Vagg authored Jan 02, 2015

- clarified the role of "collaborators"
- split out a governance doc
- split out a collaborator guide
- cleaned up the contributing doc
- cleaned up the readme & added collaborators list

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


Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Mikeal Rogers <mikeal.rogers@gmail.com>

(Note: no explicit "LGTM" from Mikeal)

Discussed at TC meeting 2015-01-07, agreed to push further
amendments to governance to future PRs.

c52e43d9

Revert "url: support `path` for url.format" · 913addbf

Ben Noordhuis authored Jan 12, 2015

This reverts commit d312b6d1.

Reverted for breaking `npm install` with git+ssh URLs.

Conflicts:
    doc/api/url.markdown

Fixes: https://github.com/iojs/io.js/issues/295
PR-URL: https://github.com/iojs/io.js/pull/303


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

913addbf

url: fix parsing of ssh urls · a1e54d6f

Evan Lucas authored Jan 12, 2015

Fix regression introduced in 61204720
that broke parsing of some ssh: urls.

An example url is ssh://git@github.com:npm/npm.git

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


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

a1e54d6f

build: move armv6 snapshot test to configure_arm() · 70c25013

Rod Vagg authored Jan 12, 2015

fails on windows because is_arch_armv6() calls CC to get properties,
so moved to a scope where we know we're at least on ARM

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


Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>

70c25013

build: Warn when using snapshot on Armv6 · 0cca6fe7

Cydox authored Jan 10, 2015

Building io.js on armv6 with snapshot enabled does not work due to a bug in v8,
so it now warns when building on armv6 and having snapshot enabled.

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


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

0cca6fe7

net: remove TODO for moving isIP to dns · d51efd07

Evan Lucas authored Jan 11, 2015

It is not really DNS-specific so keeping it in net makes more sense.

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


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

d51efd07