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

[20.0.0]: Backport fix from main to the release branch #8496

Merged

Conversation

dundargoc
Copy link
Contributor

This is a backport of PR #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 a project() function
allows it to also be built as its own project.

Also set USES_TERMINAL_BUILD to TRUE in ExternalProject_Add(),
which allows getting the installation progress output from cargo build.

Currently the CMakeLists.txt is designed to only be consumed by a parent
project, which limits its flexibility. Adding a `project()` function
allows it to also be built as its own project.

Also set `USES_TERMINAL_BUILD` to `TRUE` in `ExternalProject_Add()`,
which allows getting the installation progress output from `cargo
build`.
@dundargoc dundargoc requested a review from a team as a code owner April 27, 2024 11:15
@dundargoc dundargoc requested review from alexcrichton and removed request for a team April 27, 2024 11:15
@github-actions github-actions bot added the wasmtime:c-api Issues pertaining to the C API. label Apr 27, 2024
@dundargoc
Copy link
Contributor Author

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
Copy link
Member

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
Copy link
Contributor Author

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.

Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Ok sounds good! I'll work on pushing the point release buttons tomorrow

@alexcrichton alexcrichton merged commit b875189 into bytecodealliance:release-20.0.0 May 2, 2024
43 checks passed
@alexcrichton
Copy link
Member

Ok 20.0.1 should be released with #8538 which will have a tag for you to use

@dundargoc dundargoc deleted the c-api/cmake branch May 3, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:c-api Issues pertaining to the C API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants