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

Update prebuild-install and lock runtime #3679

Merged
merged 3 commits into from
Apr 9, 2021

Conversation

takameyer
Copy link
Contributor

In preparation for github actions, we need to update prebuild-install to allow for
local installation of a prebuild package. This is provided in version 6.1.1.

Also, in the binary.package_name field the napi runtime is hardcoded.
The runtime is now hard coded into the install script for prebuild-install, as we don't support any other runtimes.

This closes #3670

☑️ ToDos

  • 📝 Changelog entry
  • 📝 Compatibility label is updated or copied from previous entry
  • 🚦 Tests
  • 📝 Public documentation PR created or is not necessary
  • 💥 Breaking label has been applied or is not necessary

If this PR adds or changes public API's:

  • typescript definitions file is updated
  • jsdoc files updated
  • Chrome debug API is updated if API is available on React Native

Copy link
Contributor

@kneth kneth left a comment

Choose a reason for hiding this comment

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

Either

  1. Target develop
  2. Revert update of vendor/realm-core

@takameyer takameyer force-pushed the andrew/prebuild-install-napi branch from 6b9625e to b54410a Compare April 8, 2021 11:16
@takameyer takameyer requested a review from kneth April 8, 2021 11:17
@takameyer
Copy link
Contributor Author

Reverted change to core version

In preparation for github actions, we need to update prebuild-install to allow for
local installation of a prebuild package.  This is provided in version 6.1.1.

Also, since in the binary.package_name field, the napi runtime is hardcoded, we have also
hardcoded the runtime into the install script for prebuild-install.
@takameyer takameyer force-pushed the andrew/prebuild-install-napi branch from b54410a to 8474734 Compare April 9, 2021 07:50
@kneth kneth mentioned this pull request Apr 9, 2021
1 task
@takameyer takameyer merged commit 019e489 into master Apr 9, 2021
@takameyer takameyer deleted the andrew/prebuild-install-napi branch April 9, 2021 15:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

npm install crashing with sh: cmake-js: command not found
3 participants