isaacs [Fri, 16 Jan 2015 00:37:47 +0000 (16:37 -0800)]
doc: Add http keepalive behavior to CHANGELOG.md
PR-URL: https://github.com/iojs/io.js/pull/458
Reviewer: Rod Vagg
Tom Hughes [Tue, 13 Jan 2015 04:31:44 +0000 (20:31 -0800)]
doc: fix author attribution
Tom Hughes and Tom Hughes-Croucher are different people.
PR: https://github.com/joyent/node/pull/9021
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Yosuke Furukawa [Thu, 15 Jan 2015 17:55:29 +0000 (02:55 +0900)]
src: fix jslint errors
PR-URL: https://github.com/iojs/io.js/pull/449
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Yosuke Furukawa [Thu, 15 Jan 2015 17:53:46 +0000 (02:53 +0900)]
tools: update closure linter to 2.3.17
PR-URL: https://github.com/iojs/io.js/pull/449
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Vladimir Kurchatkin [Fri, 9 Jan 2015 11:51:39 +0000 (14:51 +0300)]
_debug_agent: use `readableObjectMode` option
Use public `readableObjectMode` option to construct `Transform`
instead of accessing private `_readableState.objectMode`.
Partially addresses https://github.com/iojs/io.js/issues/445.
PR-URL: https://github.com/iojs/io.js/pull/270
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Bert Belder <bertbelder@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Rod Vagg [Thu, 15 Jan 2015 18:49:13 +0000 (19:49 +0100)]
doc: fix formatting in LICENSE for RTF generation
Current format does not render properly when converted to RTF by
the tools/license2rtf.js tool, specifically the wrong sections are
presented as bold, giving the wrong emphasis to the document.
This fix makes the formatting more consistent, with non-license
summary sections bold and the licenses themselves unformatted.
PR: https://github.com/iojs/io.js/pull/436
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Phil Hughes [Wed, 14 Jan 2015 16:29:52 +0000 (16:29 +0000)]
doc: fix 404s for syntax highlighting js
Updated Makefile to remove special casing for those files, and
moved the files to doc/api_assets.
Fixes: https://github.com/iojs/iojs.github.io/issues/51
PR-URL: https://github.com/iojs/io.js/pull/409
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Aaron Bieber [Wed, 14 Jan 2015 21:40:53 +0000 (14:40 -0700)]
src: define AI_V4MAPPED for OpenBSD
OpenBSD doesn't define AI_V4MAPPED by default. Defining it to 0
as done with other ports.
PR-URL: https://github.com/iojs/io.js/pull/427
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Michal Tehnik [Thu, 15 Jan 2015 12:58:31 +0000 (13:58 +0100)]
doc: extend example of http.request by end event
Added .on('end', callback) event to http.request example, because
for first sight it's not clear from http documentation, how to
handle end of request.
PR-URL: https://github.com/iojs/io.js/pull/447
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Shigeki Ohtsu [Wed, 14 Jan 2015 07:39:44 +0000 (16:39 +0900)]
src: fix documentation url in help message
Refer to https://iojs.org/ for documentation
PR-URL: https://github.com/iojs/io.js/pull/378
Reviewed-By: Rod Vagg <rod@vagg.org>
Bert Belder [Wed, 14 Jan 2015 23:42:28 +0000 (00:42 +0100)]
win,msi: warn that older io.js needs manual uninstall
Because the UpgradeCode changes with version 1.0.2, older versions won't
be uninstalled automatically. This patch puts a warning in the
installer.
PR: https://github.com/iojs/io.js/pull/431
Reviewed-by: Rod Vagg <rod@vagg.org>
Bert Belder [Wed, 14 Jan 2015 23:41:26 +0000 (00:41 +0100)]
win,msi: change UpgradeCode
This allows node.js and io.js to be installed side-by-side.
PR: https://github.com/iojs/io.js/pull/431
Reviewed-by: Rod Vagg <rod@vagg.org>
Bert Belder [Thu, 15 Jan 2015 00:39:30 +0000 (01:39 +0100)]
deps: make node-gyp work again on windows
* Fetch from the correct url.
* Link compiled addons with iojs.lib instead of node.lib.
* Disable checksum checks for iojs.lib until our website supports
them.
PR: https://github.com/iojs/io.js/pull/422
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-by: Rod Vagg <rod@vagg.org>
Saúl Ibarra Corretgé [Wed, 14 Jan 2015 19:26:02 +0000 (20:26 +0100)]
deps: update libuv to 1.2.1
PR: https://github.com/iojs/io.js/pull/423
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Mathias Bynens [Wed, 14 Jan 2015 10:18:37 +0000 (11:18 +0100)]
doc: mention io.js alongside Node in Punycode docs
PR-URL: https://github.com/iojs/io.js/pull/390
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Wed, 14 Jan 2015 13:00:26 +0000 (16:00 +0300)]
deps: update http_parser to 2.4.1
PR-URL: https://github.com/iojs/io.js/pull/397
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ben Noordhuis [Wed, 14 Jan 2015 20:23:13 +0000 (21:23 +0100)]
doc: update cluster entry in CHANGELOG
PR-URL: https://github.com/iojs/io.js/pull/425
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Mathias Buus [Wed, 14 Jan 2015 17:42:57 +0000 (09:42 -0800)]
doc: fix double smalloc example
PR-URL: https://github.com/iojs/io.js/pull/413
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ben Noordhuis [Wed, 14 Jan 2015 01:58:44 +0000 (02:58 +0100)]
crypto: add deprecated verisign cert for s3 compat
Nolens volens add back a 1024 bits Verisign Class 3 certificate that is
still in use by AWS S3.
Fixes: https://github.com/iojs/io.js/issues/402
Yosuke Furukawa [Wed, 14 Jan 2015 16:46:28 +0000 (01:46 +0900)]
benchmark: fix command name in benchmark scripts
PR-URL: https://github.com/iojs/io.js/pull/410
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Chris Alley [Wed, 14 Jan 2015 08:40:03 +0000 (21:40 +1300)]
doc: fix broken readme links to collaborator list
PR-URL: https://github.com/iojs/io.js/pull/382
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Mathias Bynens [Wed, 14 Jan 2015 08:00:59 +0000 (09:00 +0100)]
doc: fix mixed content issues
Explicitly specify "https" over "http" for linked CSS. Fixup
indentation and quoting issues.
PR-URL: https://github.com/iojs/io.js/pull/379
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Rod Vagg [Wed, 14 Jan 2015 05:15:39 +0000 (16:15 +1100)]
Working on 1.0.2
Jesse cogollo [Wed, 14 Jan 2015 04:44:16 +0000 (23:44 -0500)]
doc: change link man doc
PR-URL: https://github.com/iojs/io.js/pull/366
Reviewed-By: Rod Vagg <rod@vagg.org>
Rod Vagg [Wed, 14 Jan 2015 04:31:29 +0000 (15:31 +1100)]
2015-01-14 io.js v1.0.1 Release
Bert Belder [Wed, 14 Jan 2015 04:20:52 +0000 (05:20 +0100)]
Merge tag 'v1.0.0-release' into v1.x
2015-01-14 io.js v1.0.0 Release
Rod Vagg [Wed, 14 Jan 2015 04:15:10 +0000 (15:15 +1100)]
Working on 1.0.1
Rui Marinho [Wed, 14 Jan 2015 03:07:11 +0000 (03:07 +0000)]
doc: improve write style consistency
Add more consistent verbs to each changelog entry, add missing periods,
trim white-space and fix sorting of entries.
PR-URL: https://github.com/iojs/io.js/pull/360
Reviewed-By: Rod Vagg <rod@vagg.org>
Bert Belder [Wed, 14 Jan 2015 03:43:21 +0000 (04:43 +0100)]
win,msi: correct doc website link
Rod Vagg [Wed, 14 Jan 2015 03:07:28 +0000 (14:07 +1100)]
2015-01-14 io.js v1.0.0 Release
Shigeki Ohtsu [Wed, 14 Jan 2015 02:36:13 +0000 (11:36 +0900)]
doc: add missed new features in CHANGELOG
Some new features were missed in Buffer and process.
PR: https://github.com/iojs/io.js/pull/359
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Rui Marinho [Wed, 14 Jan 2015 02:26:08 +0000 (02:26 +0000)]
doc: fix deps versions on changelog
PR-URL: https://github.com/iojs/io.js/pull/358
Reviewed-By: Rod Vagg <rod@vagg.org>
Rod Vagg [Wed, 14 Jan 2015 02:16:16 +0000 (13:16 +1100)]
build: rename ChangeLog to CHANGELOG.md in build
This only matters for the Makefile tarball target
as we aren't auto-building changelog.html at the moment
so the changes in tools/ are for completeness, but
not functional with the Markdown CHANGELOG.md.
PR-URL: https://github.com/iojs/io.js/pull/357
no review
Bert Belder [Wed, 14 Jan 2015 02:21:20 +0000 (03:21 +0100)]
win: use the new io.js icon
PR: https://github.com/iojs/io.js/pull/356
Reviewed-by: Rod Vagg <rod@vagg.com>
Domenic Denicola [Wed, 14 Jan 2015 02:12:46 +0000 (21:12 -0500)]
doc: fix Node versions in changelog's v8 paragraph
PR-URL: https://github.com/iojs/io.js/pull/355
Reviewed-By: Rod Vagg <rod@vagg.org>
Bert Belder [Wed, 14 Jan 2015 02:00:34 +0000 (03:00 +0100)]
win: update icon for io.js
PR: https://github.com/iojs/io.js/pull/354
Reviewed-by: Rod Vagg <rod@vagg.com>
Bert Belder [Wed, 14 Jan 2015 01:56:51 +0000 (02:56 +0100)]
build: add iojs installer assets for win & osx
Added visual assets used by WiX on Windows and PackageMaker on OS X to
build the native installers. Added node.ico file with the new io.js logo
for Windows.
PR: https://github.com/iojs/io.js/pull/344
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Tiago Ribeiro [Wed, 14 Jan 2015 01:48:09 +0000 (01:48 +0000)]
doc: fix typo in changelog
PR-URL: https://github.com/iojs/io.js/pull/353
Reviewed-By: Rod Vagg <rod@vagg.org>
Johan Bergström [Wed, 14 Jan 2015 01:30:20 +0000 (12:30 +1100)]
test: introduce a test for process.versions output
make sure that process.versions contains an expected list of software to avoid
potential mistakes with refactoring.
PR-URL: https://github.com/iojs/io.js/pull/352
Reviewed-By: Rod Vagg <rod@vagg.org>
Rod Vagg [Wed, 14 Jan 2015 01:22:04 +0000 (12:22 +1100)]
doc: added io.js v1.0.0 ChangeLog as a summary
Collaboratively edited by many people in
https://code.stypi.com/domenic/iojs-1.0-changelog.md
and includes a summary of changes in each major section of the
project from a user-facing perspective from Node.js v0.10.34
to io.js v1.0.0.
Renamed file to CHANGELOG.md from ChangeLog.md
Fixes: https://github.com/iojs/io.js/issues/339
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Rod Vagg [Wed, 14 Jan 2015 01:21:05 +0000 (12:21 +1100)]
doc: convert ChangeLog to Markdown
Conversion of the ChangeLog includes headings, compacted lists,
links for commits to GitHub (iojs/io.js) and links to issues on
GitHub (joyent/node)
Fixes: https://github.com/iojs/io.js/issues/339
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Fedor Indutny [Wed, 14 Jan 2015 01:34:19 +0000 (04:34 +0300)]
Revert "deps: update http_parser to 2.4.0"
The commit breaks windows build and brings some test failures that needs
to be debugged.
This reverts commit
d790f612218bb34580bfbc6a11027d69b7c148fe.
Brian White [Wed, 14 Jan 2015 01:19:26 +0000 (20:19 -0500)]
doc: update util methods
PR-URL: https://github.com/iojs/io.js/pull/350
Reviewed-By: Rod Vagg <rod@vagg.org>
Mikeal Rogers [Wed, 1 Oct 2014 20:40:32 +0000 (13:40 -0700)]
First draft of a new contribution policy.
PR-URL: https://github.com/node-forward/node/pull/1
Signed-off-by: isaacs <i@izs.me>
PR-URL: https://github.com/iojs/io.js/pull/348/files
Signed-off-by: isaacs <i@izs.me> (again)
Bert Belder [Wed, 14 Jan 2015 00:11:51 +0000 (01:11 +0100)]
win,msi: link node.exe to iojs.exe
Add a feature to the MSI installer that, when selected, hard-links
node.exe to iojs.exe.
PR: https://github.com/iojs/io.js/pull/346
Reviewed-by: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-by: Rod Vagg <rod@vagg.com>
Johan Bergström [Wed, 14 Jan 2015 00:12:05 +0000 (11:12 +1100)]
process: reintroduce ares to versions
ares version info was accidentally removed in
c80f8fa8f.
PR-URL: https://github.com/iojs/io.js/pull/347
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Fedor Indutny [Tue, 13 Jan 2015 23:57:45 +0000 (02:57 +0300)]
deps: update http_parser to 2.4.0
PR-URL: https://github.com/iojs/io.js/pull/345
Reviewed-By: Rod Vagg <rod@vagg.org>
Ben Noordhuis [Tue, 13 Jan 2015 22:22:49 +0000 (23:22 +0100)]
deps: make node-gyp fetch tarballs from iojs.org
Apply a small patch that makes node-gyp fetch the tarballs from
https://iojs.org/ instead of http://nodejs.org/
A patch better suited for inclusion upstream will be put together
shortly.
PR-URL: https://github.com/iojs/io.js/pull/343
Reviewed-By: Rod Vagg <rod@vagg.org>
Rod Vagg [Tue, 13 Jan 2015 21:39:45 +0000 (08:39 +1100)]
doc: update AUTHORS list
Update AUTHORS file using tools/update-authors.sh
PR-URL: https://github.com/iojs/io.js/pull/342
Reviewed-By: Ben Noorhduis <info@bnoordhuis.nl>
Rod Vagg [Sun, 11 Jan 2015 23:59:30 +0000 (10:59 +1100)]
doc: add io.js LICENSE block
PR-URL: https://github.com/iojs/io.js/pull/294
Reviewed-By: Ben Noorhduis <info@bnoordhuis.nl>
Reviewed-By: Bert Belder <bertbelder@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Discussed in TC meeting 2015-01-13, merge but accept changes
if better advice comes along
Fedor Indutny [Tue, 13 Jan 2015 13:35:08 +0000 (16:35 +0300)]
test: disable fs-readfile-error on FreeBSD
FreeBSD does not return EISDIR when reading "/".
PR-URL: https://github.com/iojs/io.js/pull/332
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Tue, 13 Jan 2015 13:28:56 +0000 (16:28 +0300)]
test: fix setproctitle on FreeBSD
PR-URL: https://github.com/iojs/io.js/pull/332
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Tue, 13 Jan 2015 13:23:21 +0000 (16:23 +0300)]
test: loosen timeout in spawnsync-test for FreeBSD
PR-URL: https://github.com/iojs/io.js/pull/332
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Tue, 13 Jan 2015 13:16:15 +0000 (16:16 +0300)]
dns: remove AI_V4MAPPED hint flag on FreeBSD
FreeBSD does not support V4MAPPED.
PR-URL: https://github.com/iojs/io.js/pull/332
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Tue, 13 Jan 2015 15:54:17 +0000 (15:54 +0000)]
deps: fix v8 build on FreeBSD
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>
Yosuke Furukawa [Tue, 13 Jan 2015 15:10:44 +0000 (00:10 +0900)]
doc: remove tracing link in toc
PR-URL: https://github.com/iojs/io.js/pull/334
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Rod Vagg [Tue, 13 Jan 2015 11:31:41 +0000 (22:31 +1100)]
src: limit .gitattributes eol to vcbuild.bat
Fixes: https://github.com/iojs/io.js/issues/330
PR-URL: https://github.com/iojs/io.js/pull/331
Johan Bergström [Tue, 13 Jan 2015 02:52:44 +0000 (13:52 +1100)]
build: initialize variable before use
found with make --warn-undefined-variables
PR-URL: https://github.com/iojs/io.js/pull/320
Reviewed-By: Rod Vagg <rod@vagg.org>
Johan Bergström [Tue, 13 Jan 2015 01:51:34 +0000 (12:51 +1100)]
build: create smaller build artifacts
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>
Ben Noordhuis [Mon, 12 Jan 2015 22:40:51 +0000 (23:40 +0100)]
test: create test-ci, modify test-all, fix cpplint
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 3e1b1dd ("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>
Sam Roberts [Tue, 13 Jan 2015 02:21:29 +0000 (03:21 +0100)]
test: use assert.throw to test exceptions
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
Timothy J Fontaine [Sat, 10 Jan 2015 20:36:16 +0000 (21:36 +0100)]
test: check for multi-localhost support
Cherry-picked-from: https://github.com/joyent/node/commit/
2b7c8a2f02c2e132c2dbc59bd05e0e7246e10196
Wyatt Preul [Tue, 13 Jan 2015 00:41:07 +0000 (18:41 -0600)]
sys: Mark as deprecated
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
Bert Belder [Tue, 13 Jan 2015 01:32:09 +0000 (02:32 +0100)]
http: optimize on_headers_complete
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>
Forrest L Norvell [Tue, 13 Jan 2015 01:21:49 +0000 (17:21 -0800)]
deps: don't build docs on 'make npm-test'
PR-URL: https://github.com/iojs/io.js/pull/318
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ben Noordhuis [Mon, 12 Jan 2015 23:45:31 +0000 (00:45 +0100)]
build,src: remove sslv3 support
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>
Ben Noordhuis [Tue, 13 Jan 2015 00:04:59 +0000 (01:04 +0100)]
test: fix message tests regression
Commit 3e1b1dd ("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>
isaacs [Mon, 12 Jan 2015 19:59:19 +0000 (11:59 -0800)]
Remove excessive copyright/license boilerplate
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.
Vladimir Guguiev [Wed, 10 Dec 2014 23:06:01 +0000 (00:06 +0100)]
docs: improve Writable/Transform stream docs
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>
Rod Vagg [Sun, 11 Jan 2015 05:49:45 +0000 (05:49 +0000)]
src: rename node to io.js for Windows installer
* 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>
Ben Noordhuis [Mon, 12 Jan 2015 22:10:34 +0000 (23:10 +0100)]
test: fix `make test-addons` target
I broke the python script in commit edaf7af 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>
Steven Rockarts [Mon, 12 Jan 2015 22:06:06 +0000 (23:06 +0100)]
win,msi: create npm folder in AppData directory
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
Ben Noordhuis [Mon, 12 Jan 2015 21:01:27 +0000 (22:01 +0100)]
addon: update NODE_MODULE_VERSION macro
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>
Chris Dickinson [Mon, 12 Jan 2015 05:40:45 +0000 (21:40 -0800)]
doc: update style for iojs
* 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>
cjihrig [Mon, 12 Jan 2015 14:24:30 +0000 (09:24 -0500)]
fs: undeprecate exists() and existsSync()
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>
cjihrig [Mon, 12 Jan 2015 16:13:18 +0000 (11:13 -0500)]
assert: throw when block is not a function
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>
Fedor Indutny [Sun, 11 Jan 2015 15:33:19 +0000 (18:33 +0300)]
gyp: remove vanished masm file from openssl build
PR-URL: https://github.com/iojs/io.js/pull/289
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Sun, 11 Jan 2015 13:08:27 +0000 (16:08 +0300)]
test: fix crypto-stream after openssl update
PR-URL: https://github.com/iojs/io.js/pull/289
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fedor Indutny [Thu, 8 Jan 2015 19:05:43 +0000 (19:05 +0000)]
openssl: fix build failure on windows
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>
Fedor Indutny [Wed, 11 Dec 2013 17:19:04 +0000 (21:19 +0400)]
openssl: fix keypress requirement in apps on win32
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>
Fedor Indutny [Sun, 11 Jan 2015 13:08:14 +0000 (16:08 +0300)]
deps: update openssl to 1.0.1k
PR-URL: https://github.com/iojs/io.js/pull/289
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ben Noordhuis [Mon, 12 Jan 2015 13:14:23 +0000 (14:14 +0100)]
test: move sequential/test-debug-port-cluster
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>
Ben Noordhuis [Mon, 12 Jan 2015 13:12:50 +0000 (14:12 +0100)]
test: move sequential/test-debug-signal-cluster
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>
Ben Noordhuis [Mon, 12 Jan 2015 13:07:55 +0000 (14:07 +0100)]
lib: make --debug-port work with cluster
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>
Ben Noordhuis [Mon, 12 Jan 2015 10:58:17 +0000 (11:58 +0100)]
test: delete sequential/test-debug-cluster
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>
Rod Vagg [Wed, 7 Jan 2015 23:06:31 +0000 (10:06 +1100)]
doc: add TC meeting 2015-01-07 minutes
PR-URL: https://github.com/iojs/io.js/pull/248
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Rod Vagg [Fri, 2 Jan 2015 11:52:50 +0000 (22:52 +1100)]
doc: clarified & split up contribution docs
- 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.
Ben Noordhuis [Mon, 12 Jan 2015 10:13:49 +0000 (11:13 +0100)]
Revert "url: support `path` for url.format"
This reverts commit
d312b6d15c69cf4c438ed7d884e6396c481a57f6.
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>
Evan Lucas [Mon, 12 Jan 2015 06:13:17 +0000 (00:13 -0600)]
url: fix parsing of ssh urls
Fix regression introduced in
61204720361824881aefd64f5bccda7d7be6617a
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>
Rod Vagg [Mon, 12 Jan 2015 04:49:55 +0000 (15:49 +1100)]
build: move armv6 snapshot test to configure_arm()
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>
Cydox [Sat, 10 Jan 2015 22:32:40 +0000 (23:32 +0100)]
build: Warn when using snapshot on Armv6
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>
Evan Lucas [Sun, 11 Jan 2015 20:33:08 +0000 (14:33 -0600)]
net: remove TODO for moving isIP to dns
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>
Ben Noordhuis [Sun, 11 Jan 2015 00:53:20 +0000 (01:53 +0100)]
build,src: remove sslv2 support
SSLv2 has been deprecated and known broken for nearly twenty years now.
I made SSLv2 support opt-in well over a year ago in commit 39aa894 and
now this commit removes it entirely.
PR-URL: https://github.com/iojs/io.js/pull/290
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Rod Vagg [Sun, 11 Jan 2015 06:29:36 +0000 (17:29 +1100)]
src: only append "-pre" to version if no tag
removed because nightly builds use the TAG make variable
to create a prerelease tag after the version yet `-pre` is
also appended beyond that. `-pre` is unnecessary if another
prerelease tag is being used.
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Rod Vagg [Sun, 11 Jan 2015 02:15:30 +0000 (13:15 +1100)]
doc: remove changelog.html from doc make target
not essential for doc build and currently broken because
tools/build_changelog.sh requires ./node.
leaving as separate make target so it can be invoked
separately in future.
PR-URL: https://github.com/iojs/io.js/pull/285
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ben Noordhuis [Sat, 10 Jan 2015 17:06:34 +0000 (18:06 +0100)]
test: fix bad assumption in pummel/test-vm-memleak
pummel/test-vm-memleak is an old test that assumes the fairly
aggressive heuristics that were common with the old collector.
The current garbage collector has a more laissez-faire attitude.
Put an upper limit on the size of the old space and update the
test's expectations.
PR-URL: https://github.com/iojs/io.js/pull/280
Reviewed-By: Bert Belder <bertbelder@gmail.com>
Bert Belder [Sun, 11 Jan 2015 00:49:33 +0000 (01:49 +0100)]
win,test: try again if file unlink fails
Now that parallel tests are enabled, the test runner spits out a ton of
'access denied' errors while running the tests. These happen because a
virus scanner or the indexing service temporarily open the file after it
has been updated, and the test runner tries to unlink() them at the same
time.
This patch resolves this issue by attempting to unlink the file until it
succeeds.
PR-URL: https://github.com/iojs/io.js/pull/284
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
Vladimir Kurchatkin [Sat, 10 Jan 2015 20:12:37 +0000 (23:12 +0300)]
fs: use ES6 octal literals for mode
Update docs, comments and code to use ES6 octal literals instead of
decimal + comment.
PR-URL: https://github.com/iojs/io.js/pull/281
Reviewed-by: Bert Belder <bertbelder@gmail.com>
Reviewed-by: Colin Ihrig <cjihrig@gmail.com>
Fedor Indutny [Thu, 6 Nov 2014 19:03:08 +0000 (22:03 +0300)]
zlib: do not unref() if wasn't ref()ed
In very unlikely case, where `deflateInit2()` may return error (right
now happening only on exhausting all memory), the `ZCtx::Error()` will
be called and will try to `Unref()` the handle. But the problem is that
this handle was never `Ref()`ed, so it will trigger an assertion error
and crash the program.
Reviewed-by: Ben Noordhuis <info@bnoordhuis.nl>
PR-URL: https://github.com/joyent/node/pull/8687
Cherry-picked-from: https://github.com/joyent/node/commit/
8c868989be778246bc4b9558f087d0f93724b83d
Trevor Norris [Mon, 15 Dec 2014 23:36:00 +0000 (15:36 -0800)]
src: remove icu_config
ICU support in v0.12 generates a new icu_config.gypi. This was
accidentally committed after switching branches. The file has been removed
and added to .gitignore.
Cherry-picked-from: https://github.com/joyent/node/commit/
813114dab05231b71f3cdc4f5889b9833d9a1d06