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

Progress toward running the Node.js API on ancient versions of Node.js. #3764

Merged
merged 1 commit into from Jun 24, 2016
Merged

Conversation

powdercloud
Copy link
Contributor

Set the Node.js version to 0.10.25, the Node.js version from Ubuntu Trusty.

I have verified that index.js runs OK with that.
In index_test.js, fix an es6 construct to be old-style. But even with that
it won't pass with 0.10.25.
There's one remaining problem with running on Trusty: The shebang line
wants node, but on Trusty it's 'nodejs'. I will attempt to fix this in a
later revision.

…rusty.

I have verified that index.js runs OK with that.
In index_test.js, fix an es6 construct to be old-style. But even with that
it won't pass with 0.10.25.
There's one remaining problem with running on Trusty: The shebang line
wants node, but on Trusty it's 'nodejs'. I will attempt to fix this in a
later revision.
@honeybadgerdontcare
Copy link
Contributor

LGTM

@powdercloud powdercloud merged commit 550046b into ampproject:master Jun 24, 2016
@powdercloud powdercloud deleted the luddite branch July 20, 2016 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants