FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Release proposal: v3.0.0 by rvagg · Pull Request #2221 · nodejs/node · GitHub

/ node Public

Release proposal: v3.0.0 - #2221

Closed
rvagg wants to merge 52 commits into
masterfrom
v3.0.0-alpha
ConversationCommits (52)ChecksFiles changed
Closed

Release proposal: v3.0.0#2221
rvagg wants to merge 52 commits into
masterfrom
v3.0.0-alpha

Conversation

rvagg commented Jul 22, 2015

Copy link
Copy Markdown
Member

I'm tackling this 3.0.0 problem from a different angle, I'm sick of producing "RC" builds from a next branch that is so far behind master, meaning they are nothing like "release candidates" and catching up is next to impossible.

This branch, on nodejs/io.js so others can contribute, is the current master with changes from next cherry-picked in and I'm just going to rebase to keep it up to date and it's mergable if we decide to go that way (I'm leaning that way currently fwiw).

I made a branch-diff tool that's like doing git log b1..b2 but not as strict and uses our PR-URL metadata to help with matches. This was mainly to help identify what to cherry-pick. It also prints out changelog-maker output so I can also do this:

yosuke-furukawa commented Jul 22, 2015

Copy link
Copy Markdown
Member

I would like to know the relationship between next and next+1 branch.
io.js v3.0 is based on next branch or next+1 branch ??

rvagg commented Jul 22, 2015

Copy link
Copy Markdown
Member Author

io.js v3.0.0 will be based on what's in this PR (may not come from this PR but it's this list of changes on master), next is where this comes from but next is falling behind master at a rapid rate, hence the change of approach. This means that v3.0.0 will be with V8 v4.4 unless we can't get it done for some reason.

next+1 is where some folks are working with V8 v4.5, at the current rate I'd expect that work to form the basis of Node.js v4.0.0. Unfortunately next+1 is going to suffer from the same problem as next in falling behind master. But if you want a playground for V8 v4.5 then you should go there.

rvagg commented Jul 22, 2015

Copy link
Copy Markdown
Member Author

v3.0.0 RC 4 based on this branch: https://iojs.org/download/rc/v3.0.0-rc.4/

$ node -pe process.versions
{ http_parser: '2.5.0',
  node: '3.0.0-rc.4',
  v8: '4.4.63.12',
  uv: '1.6.1',
  zlib: '1.2.8',
  ares: '1.10.1-DEV',
  modules: '45',
  openssl: '1.0.2d' }

Includes the node-gyp changes for quick header downloads (note how it not only grabs the headers tarball but it also knows this is an RC so it has to go elsewhere for it):

$ npm i buffertools
npm http request GET https://registry.npmjs.org/buffertools
npm http 200 https://registry.npmjs.org/buffertools
npm http fetch GET https://registry.npmjs.org/buffertools/-/buffertools-2.1.3.tgz
npm http fetch 200 https://registry.npmjs.org/buffertools/-/buffertools-2.1.3.tgz
npm WARN engine buffertools@2.1.3: wanted: {"node":">=0.3.0"} (current: {"node":"3.0.0-rc.4","npm":"2.13.1"})

> buffertools@2.1.3 install /home/rvagg/git/iojs/io.js/node_modules/buffertools
> node-gyp rebuild

gyp http GET https://iojs.org/download/rc/v3.0.0-rc.4/iojs-v3.0.0-rc.4-headers.tar.gz
gyp http 200 https://iojs.org/download/rc/v3.0.0-rc.4/iojs-v3.0.0-rc.4-headers.tar.gz
gyp http GET https://iojs.org/download/rc/v3.0.0-rc.4/SHASUMS256.txt
gyp http 200 https://iojs.org/download/rc/v3.0.0-rc.4/SHASUMS256.txt
make: Entering directory '/home/rvagg/git/iojs/io.js/node_modules/buffertools/build'
  CXX(target) Release/obj.target/buffertools/buffertools.o
  SOLINK_MODULE(target) Release/obj.target/buffertools.node
  COPY Release/buffertools.node
make: Leaving directory '/home/rvagg/git/iojs/io.js/node_modules/buffertools/build'
buffertools@2.1.3 node_modules/buffertools

It also includes the process.release changes which will be used for further modifications to node-gyp which should be upstreamed if all goes well.

$ node -pe process.release
{ name: 'io.js',
  sourceUrl: 'https://iojs.org/download/rc/v3.0.0-rc.4/iojs-v3.0.0-rc.4.tar.gz',
  headersUrl: 'https://iojs.org/download/rc/v3.0.0-rc.4/iojs-v3.0.0-rc.4-headers.tar.gz' }

rvagg mentioned this pull request Jul 22, 2015
Closed

ralphtheninja commented Jul 22, 2015

Copy link
Copy Markdown

Huge +1 for process.release

ofrobots commented Jul 22, 2015

Copy link
Copy Markdown
Contributor

@rvagg I'm a little bit confused with what the difference between the next and this branch is and why it would be easier to merge master onto this branch and then cherry-pick from next rather than merging master onto next.

Going forward, should V8 upgrades (like the latest one in #2220) continue going into next?

trevnorris commented Jul 22, 2015

Copy link
Copy Markdown
Contributor

@rvagg Buffer changes look good to me. Though I would recommend we land #2195 first. Also I noticed another bug in Slice(). Easy enough to fix, but will need to submit a PR and all that.

trevnorris commented Jul 22, 2015

Copy link
Copy Markdown
Contributor

@rvagg Here's the fix for the other issue: #2225

mscdex added the meta Issues and PRs related to the general management of the project. label Jul 22, 2015

rvagg commented Jul 23, 2015

Copy link
Copy Markdown
Member Author

@ofrobots this:

$ git diff v3.0.0-alpha next --stat | tail -1
 695 files changed, 13678 insertions(+), 18437 deletions(-)

Granted a lot of that is npm, but getting out release candidates from a next branch that is so far behind master is a pain and lacks legitimacy. This is just my attempt at resetting so we have a path to getting it out.

I was rightly scolded in the TSC call today for my willingness to rebase given the pain this causes for anyone else working on the same branch, but we're starting to converge on a better process for keeping all of these things up to date. 3.0.0 is just a hump we have to get over now!

rvagg force-pushed the v3.0.0-alpha branch 2 times, most recently from d584b2b to 01336b0 Compare July 23, 2015 03:13

ofrobots commented Jul 23, 2015

Copy link
Copy Markdown
Contributor

Okay, I see. Going forward I can volunteer some time to try and keep next and next+1 up-to-date with master.

hokein commented Jul 23, 2015

Copy link
Copy Markdown
Contributor

Hi all, could you tell me when will iojs v3.0.0 release?

Fishrock123 commented Jul 23, 2015

Copy link
Copy Markdown
Contributor

Hi all, could you tell me when will iojs v3.0.0 release?

Soon™ -- Literally when it's ready. Most of everything is done now, we had to wait for NAN to catch up for native addons to be reasonable.

bnoordhuis commented Jul 23, 2015

Copy link
Copy Markdown
Member

Going forward I can volunteer some time to try and keep next and next+1 up-to-date with master.

Me too post. I've been a little overcommitted recently but I'll have more time again starting next week.

I missed the discussion on yesterday's TSC call. Is the plan to merge master frequently into next and next+1 (or master into next and next into next+1)?

Fishrock123 commented Jul 23, 2015

Copy link
Copy Markdown
Contributor

@bnoordhuis will you possibly be able to make it to #2230?

The generally discussion has shifted a bit. Turns out things like bisect and blame don't work so well with the previous proposed approach.

See also: https://gist.github.com/trevnorris/7620a64b086e95271197 and #2215

bnoordhuis commented Jul 23, 2015

Copy link
Copy Markdown
Member

will you possibly be able to make it to #2230?

I think I can make that. I'll try to.

rvagg commented Jul 24, 2015

Copy link
Copy Markdown
Member Author

https://iojs.org/download/rc/v3.0.0-rc.5/ RC 5

Starefossen commented Jul 24, 2015

Copy link
Copy Markdown
Member

Great work 👏

YurySolovyov commented Jul 27, 2015

Copy link
Copy Markdown

#2193 should probably included too, according to #2193 (comment)

Fishrock123 commented Jul 27, 2015

Copy link
Copy Markdown
Contributor

@YuriSolovyov everything in 2.x will be included. Don't worry. :)

trevnorris commented Jul 27, 2015

Copy link
Copy Markdown
Contributor

There have been additional bug fixes (e.g. 9cf36a1). Will these be taken into the v3.0.0 release, or done after in a 3.0.1?

AntouanK commented Jul 30, 2015

Copy link
Copy Markdown

Can we expect Chrome's ES6 features in v3?
( like arrow functions, spread operator and Array static methods )
see http://kangax.github.io/compat-table/es6/#chrome45

targos commented Jul 30, 2015

Copy link
Copy Markdown
Member

@AntouanK probably not, because Chrome 45 is still in beta and v3 will be released with the V8 version of Chrome 44.

bnoordhuis commented Jul 30, 2015

Copy link
Copy Markdown
Member

@AntouanK v3.x will ship with V8 4.4, the same version of V8 that's in Chrome 44.

rvagg commented Jul 30, 2015

Copy link
Copy Markdown
Member Author

Updated to latest master, included additional items in next that were not in here, performed a diff against a next + master merged and found some rebase problems that I've fixed, also discovered that the commit for #1961 hasn't been pulled in to next yet.

Merged in #2275 and #2247 prior to cutting a v3.0.0-rc.6 which is now available in https://iojs.org/download/rc/v3.0.0-rc.6/ - @fhemberger would you mind testing both Windows installers? I haven't tried them but they appear to have been generated successfully.

rvagg and others added 15 commits August 4, 2015 09:50
src: increment NODE_MODULE_VERSION to 45
35c7938 
PR-URL: #2096
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
node: do not override message/stack of error
df9cdd8 
Put the `...^` arrow string to the hidden property of the object, and
use it only when printing error to the stderr.

Fix: #2104
PR-URL: #2108
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
http: use official IANA Status Codes
fa5b871 
* adds missing HTTP status codes
* corrects those with a wrong description
* the falsely included codes have been kept

PR-URL: #1470
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
src: introduce process.release object
b096278 
PR-URL: #2154
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
node-gyp: detect RC build with x.y.z-rc.n format
6c97138 
PR-URL: #2171
Reviewed-By: Alexis Campailla <alexis@janeasystems.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
buffer: fix not return on error
8d13222 
Throwing a JS error from C++ does not mean the function will return
early. This must be done manually.

Also remove extraneous comment no longer relevant.

Fix: 2903030 "buffer: switch API to return MaybeLocal<T>"
PR-URL: #2225
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
build: first set of updates to enable PPC support
be68537 
These are the core changes that allow pLinux BE/LE compile. They
don't include all of the changes needed for AIX which will follow
once we have pLinux up and running in the builds

PR-URL: #2124
Reviewed-By: Ben Noordhuis <ben@strongloop.com>
Reviewed-By: Johan Bergstrom <bugs@bergstroem.nu>
build: add 'x86' option back in to configure
606fdfa 
Accidentally removed @ #2124
when ppc support was added

PR-URL: #2233
Reviewed-By: Johan Bergstrom <bugs@bergstroem.nu>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
buffer: fix missing null/undefined check
2f54e45 
The new implementation of Buffer missed the check for null/undefined as
the first argument to new Buffer(). Reintroduce the check and add test.

Fix: e8734c0 "buffer: implement Uint8Array backed Buffer"
Fix: #2194
PR-URL: #2195
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
node: remove redundant --use-old-buffer
050ab42 
Accidentally left when upgrading to V8 4.4.63.9

PR-URL: #2275
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
build: prepare Windows installer for i18n support
c04c190 
PR-URL: #2247
Reviewed-By: Rod Vagg <rod@vagg.org>
readline: allow tabs in input
fd8da09 
If tab completion is not being used, allow user to enter tab
characters.

PR-URL: #1761
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
http: fix agent.getName() and add tests
d390a6f 
This commit fixes agent.getName(), which returned an extra colon
according to the docs, and adds tests (it was previously not unit
tested).

PR-URL: #1617
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
deps: update V8 to 4.4.63.26
181c3bc 
Includes cherry-picks for:
 * JitCodeEvent patch: https://crrev.com/f7969b1d5a55e66237221a463daf422ac7611788
 * argparse patch: https://crrev.com/44bc918458481d60b08d5566f0f31a79e39b85d7

PR-URL: #2220
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
src: disable vector ICs on arm
fe22685 
A bug on ARMv6 makes the process crash when the V8 debugger is used.
Disabling vector ICs as a workaround.

V8-Bug: https://code.google.com/p/v8/issues/detail?id=4338
PR-URL: #2220
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Rod Vagg <rod@vagg.org>

rvagg commented Aug 4, 2015

Copy link
Copy Markdown
Member Author

thanks @mzgol, have amended the changelog

Fishrock123 and others added 5 commits August 4, 2015 10:27
doc: document repl persistent history changes
40a9cae 
PR-URL: #2224
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
test: add tests for persistent repl history
8e5eae9 
PR-URL: #2224
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
repl: default persistence to ~/.node_repl_history
17c4108 
Makes the REPL persistently save history by default to
~/.node_repl_history when in terminal mode.
This can be disabled by setting NODE_REPL_HISTORY="".

PR-URL: #2224
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
repl: persist history in plain text
f00e11b 
Persists the REPL history in plain text using the new NODE_REPL_HISTORY
environment variable. Deprecates NODE_REPL_HISTORY_FILE. The REPL will
notify the user and automatically convert the history to the new format
if files are specified.

PR-URL: #2224
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
2015-08-01 io.js v3.0.0 Release
966c977 
Notable changes:

rvagg commented Aug 4, 2015

Copy link
Copy Markdown
Member Author

https://jenkins-iojs.nodesource.com/job/node-test-commit/102/
& https://jenkins-iojs.nodesource.com/job/node-test-commit/103/

rvagg mentioned this pull request Aug 4, 2015
Merged
rvagg closed this Aug 4, 2015
rvagg deleted the v3.0.0-alpha branch August 4, 2015 19:55
snyk-bot mentioned this pull request Aug 11, 2020
Open
solebox mentioned this pull request Nov 27, 2023
Open
s4sc mentioned this pull request Nov 28, 2023
Open
enterstudio mentioned this pull request Nov 29, 2023
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Back | FazBrowse Home | New Git URL