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

Release ndk-build-0.6.0, cargo-apk-0.9.2 #289

Merged
merged 2 commits into from
Jun 13, 2022
Merged

Conversation

MarijnS95
Copy link
Member

Moves the NDK r23 -lgcc workaround to ndk_build::cargo::cargo_ndk(), so that cargo apk -- blanket invocations receive it too. This should solve winit CI failures again now that GitHub Actions Virtual Enviroments are finishing up the migration to NDK r23.

Moves the NDK r23 `-lgcc` workaround to `ndk_build::cargo::cargo_ndk()`,
so that `cargo apk --` blanket invocations receive it too.  This should
solve `winit` CI failures again now that GitHub Actions Virtual
Enviroments are finishing up the migration to NDK r23.
@MarijnS95
Copy link
Member Author

@msiglreith Shall we get this out of the door now to get rid of red CI in winit? I don't have anything queued up for these crates, all my changes are targeted towards ndk and friends.

Comment on lines +14 to +22
anyhow = "1.0.57"
cargo-subcommand = "0.7"
dunce = "1"
env_logger = "0.9"
log = "0.4"
ndk-build = { path = "../ndk-build", version = "0.6.0" }
serde = "1"
thiserror = "1.0.31"
toml = "0.5"
Copy link
Member Author

Choose a reason for hiding this comment

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

This passes -Zminimal-versions, I didn't bother to find the minimal working ones for anyhow/thiserror though.

@MarijnS95 MarijnS95 merged commit 60e34d1 into master Jun 13, 2022
@MarijnS95 MarijnS95 deleted the release-cargo-apk-0.9.2 branch June 13, 2022 12:45
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.

2 participants