dundargoc requested wasmtime-core-reviewers for a review on PR #8496.
dundargoc requested alexcrichton for a review on PR #8496.
dundargoc opened PR #8496 from dundargoc:c-api/cmake
to bytecodealliance:release-20.0.0
:
This is a backport of PR https://github.com/bytecodealliance/wasmtime/pull/8490
build: make the c-api CMakeLists.txt easier to import
Currently the CMakeLists.txt is designed to only be consumed by a parent
project, which limits its flexibility. Adding aproject()
function
allows it to also be built as its own project.Also set
USES_TERMINAL_BUILD
toTRUE
inExternalProject_Add()
,
which allows getting the installation progress output fromcargo build
.
dundargoc commented on PR #8496:
Unsure about how backporting procedures are done in wasmtime, lemme know if this isn't correct and I can close it. Didn't find specific info about it in the contributing docs.
alexcrichton commented on PR #8496:
No worries this is indeed the procedure! The only other step is that a maintainer will need to trigger the patch release but that's just hitting a button and then hitting the PR merge button.
To confirm, is this something you'd want in a point-release of wasmtime? This, as far as I would know, would only change the source tarball in our release artifacts. If you're not using that then it may not affect too much. Or are you mostly interested in seeing this on the
release-20.0.0
branch in git?
dundargoc commented on PR #8496:
Oh, I see. We are not using the tarballs, just the commits pointed by the git tags. Yes, I mostly wanted to backport it to the release branch.
alexcrichton submitted PR review:
Ok sounds good! I'll work on pushing the point release buttons tomorrow
alexcrichton merged PR #8496.
alexcrichton commented on PR #8496:
Ok 20.0.1 should be released with https://github.com/bytecodealliance/wasmtime/pull/8538 which will have a tag for you to use
Last updated: Nov 22 2024 at 16:03 UTC