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

Upgrade V8 binaries for 10.9.194.9 version **working** #359

Closed
wants to merge 10 commits into from

Conversation

jacques-n
Copy link
Contributor

Use existing workflows to build latest V8 artifacts as well as code changes to ensure v8go continues to work.

Includes:

  • Update github workflow to use go 1.8 & 1.9
  • Update github workflow to use macos-latest
  • Update github build workflow to use ubuntu 22.04
  • Add gitignore for jetbrains and .gclient_previous files
  • Switch cgo build to C++17 and enable sandbox at build time
  • Update test with update to date error message
  • Remove no longer supported build flag.
  • Move initialization to v8go.go and include flag set to avoid flag
    freezing
  • Reorder initialization so allocator is initialized after v8 (required
    by latest v8)

Commits are kept separate with merges to certify provenance of all library objects built using existing rogchap/v8go mechanisms.

jacques-n and others added 10 commits January 12, 2023 23:11
- Update github workflow to use go 18 & 19
- Update github workflow to use macos-latest
- Update github build workflow to use ubuntu 22.04
- Add gitignore for jetbrains and .gclient_previous files
- Switch cgo build to C++17 and enable sandbox at build time
- Update test with update to date error message
- Remove no longer supported build flag.
- Move initialization to v8go.go and include flag set to avoid flag freezing
- Reorder initialization so allocator is initialized after v8 (required by latest v8)
Auto-generated pull request to build V8 for ubuntu-22.04 x86_64
Auto-generated pull request to build V8 for macos-11 x86_64
Auto-generated pull request to build V8 for macos-11 arm64
Auto-generated pull request to build V8 for ubuntu-22.04 arm64
@fizx
Copy link
Collaborator

fizx commented Jan 14, 2023

@dylanahsmith 🙏

@jacques-n
Copy link
Contributor Author

Quick fyi. The automated update on our repo seems to have overwritten this pr branch. Will fix shortly.

@jacques-n jacques-n changed the base branch from v8_10.9.194.9_upgrade to master January 14, 2023 19:54
@jacques-n jacques-n closed this Jan 14, 2023
@jacques-n jacques-n deleted the v8_10.9.194.9_upgrade branch January 14, 2023 19:55
@jacques-n
Copy link
Contributor Author

FYI, this was reopened as #363 since the automation was overwriting a branch with this name (as best I can tell).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants