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

vendoring: Mention rust-cargo-vendor-filterer #76

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

slyon
Copy link
Contributor

@slyon slyon commented Jan 15, 2025

The rust-cargo-vendor-filterer package is newly available as of Plucky (and soon to be in Debian), thanks to @julian-klode .

The MIR Rust vendoring docs can probably make some good use of it.

@julian-klode
Copy link
Contributor

There's a bunch of irregularieties with rust-vendor/ vs debian/rust-vendor in the commit and in the page already.

We need to sort out armhf too, it unfortunately does not end in gnu, for sqv I currently have:

env -i cargo-vendor-filterer -F crypto-openssl --no-default-features --tier 2 --platform '*-*-linux-gnu' --platform '*-*-linux-gnueabi' $(CARGO_VENDOR_DIR)

@slyon slyon marked this pull request as draft January 16, 2025 13:53
@slyon
Copy link
Contributor Author

slyon commented Jan 16, 2025

Moving to "Draft", as Julian might want to improve upon this.

@didrocks
Copy link

For reference, this is what we did in authd:

@julian-klode
Copy link
Contributor

@didrocks Ah you're missing armhf there as a platform which is *-unknown-linux-gnueabihf

@didrocks
Copy link

@julian-klode good point! Will amend this.

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