Skip to content
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.

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

deps: import [email protected] #22488

Closed
wants to merge 3 commits into from
Closed

Conversation

rubys
Copy link
Member

@rubys rubys commented Aug 23, 2018

Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.1.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in #22405.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.1.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in nodejs#22405.
@nodejs-github-bot nodejs-github-bot added the lib / src Issues and PRs related to general changes in the lib or src directory. label Aug 23, 2018
Copy link
Contributor

@refack refack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@refack
Copy link
Contributor

refack commented Aug 23, 2018

/CC @TimothyGu

Copy link
Member

@BridgeAR BridgeAR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

Copy link
Member

@TimothyGu TimothyGu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I admittedly haven't been too diligent on upgrading Acorn… Would you mind regenerating the LICENSE.md file by running tools/license-builder.sh?

update produced by tools/license-builder.sh
Copy link
Member

@jdalton jdalton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Holding to see if acorn can bump changes in their master so that we can upgrade to the more recent latest and greatest. (if not possible in a few days I'll release the hold)

@rubys
Copy link
Member Author

rubys commented Aug 24, 2018

@jdalton "Latest commit bba80ab 17 days ago". Are you pursuing a release or there something I should be doing to make this happen?

@jdalton
Copy link
Member

jdalton commented Aug 24, 2018

@rubys Just a release. Nothing you need to do (beside bump the acorn dep in the PR when appropriate). Their last release was mid June so if they are able to bump then we'll get more fresh bits is all.

@rubys
Copy link
Member Author

rubys commented Aug 24, 2018

@jdaldon, let me rephrase the question. Are you aware of the acorn team actively preparing a release, or has somebody asked them to produce a release, or are we merely delaying on the off chance that a release coincidentally happens in the next few days?

@jdalton
Copy link
Member

jdalton commented Aug 24, 2018

@rubys

Are you aware of the acorn team actively preparing a release, or has somebody asked them to produce a release

Yes, I've inquired about a release.

@rubys
Copy link
Member Author

rubys commented Aug 24, 2018

@rubys rubys changed the title deps: import [email protected] deps: import [email protected] Aug 24, 2018
Copy link
Contributor

@refack refack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still RSLGTM

@refack
Copy link
Contributor

refack commented Aug 25, 2018

@rubys
Copy link
Member Author

rubys commented Aug 26, 2018

Failures: https://ci.nodejs.org/job/node-test-binary-windows/19527/COMPILED_BY=vs2017,RUNNER=win2016,RUN_SUBSET=2/console

17:23:33 ERROR: Error fetching remote repo 'jenkins_tmp'
17:23:33 hudson.plugins.git.GitException: Failed to fetch from [email protected]:binary_tmp.git
17:23:33 stderr: fatal: Unable to create 'c:/workspace/node-test-binary-windows/.git/index.lock': File exists.
17:23:33 
17:23:33 Another git process seems to be running in this repository, e.g.
17:23:33 an editor opened by 'git commit'. 

Copy link
Member

@trivikr trivikr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

rubys added a commit to rubys/node that referenced this pull request Aug 26, 2018
Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.2.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in nodejs#22405.

PR-URL: nodejs#22488
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Reviewed-By: Bryan English <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
@rubys
Copy link
Member Author

rubys commented Aug 26, 2018

landed: 49bfc37

@rubys rubys closed this Aug 26, 2018
addaleax pushed a commit that referenced this pull request Aug 27, 2018
Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.2.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in #22405.

PR-URL: #22488
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Reviewed-By: Bryan English <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
targos pushed a commit that referenced this pull request Sep 3, 2018
Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.2.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in #22405.

PR-URL: #22488
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Reviewed-By: Bryan English <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
targos pushed a commit that referenced this pull request Sep 6, 2018
Imported from the tarball published on npm
(https://registry.npmjs.org/acorn/-/acorn-5.7.2.tgz).

Update to emcaScript version 10 in order to get support for
binding-less catch statements.

Also needed to parse node.js lib API in #22405.

PR-URL: #22488
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Reviewed-By: Bryan English <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lib / src Issues and PRs related to general changes in the lib or src directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants