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

Remove Node.js as an installed pre-requisite for the jsii runtime #380

Closed
1 of 11 tasks
nija-at opened this issue Sep 24, 2021 · 4 comments
Closed
1 of 11 tasks

Remove Node.js as an installed pre-requisite for the jsii runtime #380

nija-at opened this issue Sep 24, 2021 · 4 comments
Labels
bar-raiser/needed management/tracking status/stale The RFC did not get any significant enough progress or tracking and has become stale.

Comments

@nija-at
Copy link
Contributor

nija-at commented Sep 24, 2021

Description

Today, the jsii runtime expects Node.js to be installed. It would be useful to remove this dependency and get the jsii-runtime to work out of the box.

Roles

Role User
Proposed by @nija-at
Author(s) @alias, @alias, @alias
API Bar Raiser @alias
Stakeholders @alias, @alias, @alias

See RFC Process for details

Workflow

  • Tracking issue created (label: status/proposed)
  • API bar raiser assigned (ping us at #aws-cdk-rfcs if needed)
  • Kick off meeting
  • RFC pull request submitted (label: status/review)
  • Community reach out (via Slack and/or Twitter)
  • API signed-off (label api-approved applied to pull request)
  • Final comments period (label: status/final-comments-period)
  • Approved and merged (label: status/approved)
  • Execution plan submitted (label: status/planning)
  • Plan approved and merged (label: status/implementing)
  • Implementation complete (label: status/done)

Author is responsible to progress the RFC according to this checklist, and
apply the relevant labels to this issue so that the RFC table in README gets
updated.

@nija-at
Copy link
Contributor Author

nija-at commented Sep 24, 2021

Some approaches we can take -

  • embedded v8
  • embedding Node.js runtime in the jsii-runtime
  • downloading Node.js runtime on the first run of jsii-runtime.

@Chriscbr
Copy link
Contributor

Could you speak to some of the motivations of removing Node as a prerequisite (minimizing bugs caused by using different Node versions, reducing the setup needed for integration test runners, etc.)? I'm also wondering if this would change the amount of setup needed by users or not, since I think the AWS CDK toolkit still requires Node.

@nija-at
Copy link
Contributor Author

nija-at commented Sep 27, 2021

The main motivation is that users building a CDK app in Java, should feel like they are using native Java libraries, and not have to install Node.js.

This also moves jsii as a possible polyglot build tool beyond just the CDK.

@mrgrain mrgrain added status/stale The RFC did not get any significant enough progress or tracking and has become stale. and removed status/proposed Newly proposed RFC labels Oct 13, 2023
@awsmjs
Copy link
Contributor

awsmjs commented Dec 15, 2023

Closing this ticket as it has gone stale and no longer reflects current priorities. We suggest you pursue experimentation in a separate package or a fork if needed at this time.

@awsmjs awsmjs closed this as completed Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bar-raiser/needed management/tracking status/stale The RFC did not get any significant enough progress or tracking and has become stale.
Projects
None yet
Development

No branches or pull requests

5 participants