Skip to content

Misc fixes for Debian#48

Open
MggMuggins wants to merge 2 commits into
awesomized:mainfrom
MggMuggins:debian-fixes
Open

Misc fixes for Debian#48
MggMuggins wants to merge 2 commits into
awesomized:mainfrom
MggMuggins:debian-fixes

Conversation

@MggMuggins

@MggMuggins MggMuggins commented May 31, 2026

Copy link
Copy Markdown

The Problem

Working on packaging crc-fast in Debian.

The Solution

  • Debian Testing already has rand = 0.10.1, so bump the dep and fix build/test failures
  • Allow the tests to run with --no-default-features --features ffi

Changes

debcargo automatically creates tests than run the test suite for each feature with all other features disabled. This fails for ffi since it doesn't pull in std. Since the function interface provided by the .so handles some file I/O for callers, it looks to me like ffi should depend on std (it fails to compile otherwise).

Planned version bump

  • Which: PATCH
  • Why: internal change

Links

https://salsa.debian.org/rust-team/debcargo-conf/-/merge_requests/1152

Notes

N/A

crc_fast_checksum_file implies that filesystem access is needed in order
to satisfy the interface defined in libcrc_fast.h.

Allows the tests to pass with default features disabled:

cargo test --no-default-features --features ffi

Signed-off-by: Wesley Hershberger <wesley.hershberger@gmail.com>
Signed-off-by: Wesley Hershberger <wesley.hershberger@gmail.com>
@MggMuggins

Copy link
Copy Markdown
Author

Previous version broke the build on aarch64; missed a Rng -> RngExt.

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.

1 participant