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

fix: build cargo-c with --locked to work around rust-lang/cargo#13002 #1785

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

fdintino
Copy link
Contributor

@fdintino fdintino commented Nov 20, 2023

rust-lang/cargo#13002 is the cause of the build failures currently on the main branch.

There is a pull request to fix the issue in cargo (rust-lang/cargo#13004), but using --locked might be advisable anyway to ensure reproducible builds.

Copy link
Collaborator

@wantehchang wantehchang left a comment

Choose a reason for hiding this comment

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

Thank you for the pull request. I suggest a small change.

ext/rav1e.cmd Outdated
@@ -13,7 +13,7 @@
git clone -b v0.6.6 --depth 1 https://github.com/xiph/rav1e.git

cd rav1e
cargo install cargo-c
cargo install cargo-c --locked
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: List the "--locked" option before "cargo-c".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@vrabaud vrabaud merged commit 5ba0e14 into AOMediaCodec:main Nov 20, 2023
18 checks passed
@fdintino fdintino deleted the fix/cargo-c-locked branch November 20, 2023 14:41
wantehchang pushed a commit to wantehchang/libavif that referenced this pull request Nov 21, 2023
wantehchang pushed a commit that referenced this pull request Nov 21, 2023
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.

3 participants