Closed
feat(v8): consolidate wee8 packaging to single CI job producing all 4 artifacts via Starlark transitions#1090
Conversation
Signed-off-by: Ryan Northey <ryan@synca.io>
…sitions - Extend _wee8_transition_impl to set //command_line_option:platforms based on arch - Remove aarch64+libstdcxx fail() from wee8_package() - Generalise wee8_prebuilt_repo_name() for all arch×stdlib combos - Add aarch64/libstdcxx key to VERSIONS["wee8_sha256"] in versions.bzl - setup_wee8_prebuilt() now loops over all 4 arch×stdlib repos; add aarch64_libstdcxx params to extensions.bzl - Remove wee8_aarch64_gcc_unsupported and unsupported_target rule; point :linux_aarch64_gcc at new prebuilt - package/BUILD: generate all 4 wee8_package targets + wee8_packages filegroup via comprehension - bazel/v8/BUILD: replace per-arch aliases with single wee8_packages alias - Collapse package-v8 CI matrix to single job targeting //v8:wee8_packages - Update _release.yml SHA extraction to cover aarch64/libstdcxx - Update test to cover aarch64/libstdcxx and generalised repo name Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Consolidate package-v8 CI legs into a single job
feat(v8): consolidate wee8 packaging to single CI job producing all 4 artifacts via Starlark transitions
Aug 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two separate
package-v8CI matrix legs (x86_64 and aarch64) produced only 3 artifacts — aarch64+libstdcxx was explicitly blocked. This collapses everything into one job that builds all fourv8-wee8-<version>-linux-{x86_64,aarch64}[-libstdcxx].tar.xzartifacts via Starlark transitions, eliminating the--platformscommand-line flag.Transition extension (
wee8_package.bzl)_wee8_transition_implnow sets//command_line_option:platformsfrom the rule'sarchattr (x86_64→@toolchains_llvm//platforms:linux-x86_64,aarch64→…:linux-aarch64) alongside the existing stdlib/toolchain logicfail()guard for aarch64+libstdcxxPrebuilt repo machinery (
wee8_prebuilt.bzl)wee8_prebuilt_repo_name(arch, stdlib)generalised:libcxx→wee8_prebuilt_<arch>,libstdcxx→wee8_prebuilt_<arch>_libstdcxx— no more arch special-casingsetup_wee8_prebuilt()now loops over all 4 arch×stdlib combinations; addsaarch64_libstdcxx_{version,sha256}paramsunsupported_targetrule removed (no remaining callers)BUILD changes
bazel/v8/BUILD: removeswee8_aarch64_gcc_unsupported;:linux_aarch64_gccnow selects@wee8_prebuilt_aarch64_libstdcxx//:wee8; per-arch aliases replaced by single//v8:wee8_packagesbazel/v8/package/BUILD: list comprehension over_ARCHES × _STDLIBSgenerates all 4wee8_packagetargets +wee8_packagesfilegroupCI / release
bazel.yml: matrix replaced by a single job targeting//v8:wee8_packageswith globv8/v8-wee8-*.tar.xz, artifact namebazel-wee8-artifacts, no--platformsflag_release.yml: SHA extraction list extended with["aarch64", "libstdcxx"]versions.bzl: aarch64wee8_sha256dict gets"libstdcxx": ""(empty → missing-prebuilt target until first publish)wee8_prebuilt_test.bzl: assertions added for aarch64/libstdcxx repo name and archive filename