core2 icon indicating copy to clipboard operation
core2 copied to clipboard

Drop `memchr` dependency

Open TheBlueMatt opened this issue 2 years ago • 2 comments

While memchr has served us well, it finally bumped its MSRV substantially such that its no longer really practical in a particularly MSRV-constrained environment (e.g. when building on machines with validated toolchains or Debian stable).

Luckily, its barely used, and not really worth retaining. A trivial scan should be something LLVM can optimize at least somewhat reasonably.

TheBlueMatt avatar Aug 28 '23 21:08 TheBlueMatt

Many more things with this crate are broken.

The CI runs the following for no-std tests:

 /home/runner/.cargo/bin/cargo test --tests --no-default-features --features nightly

If you removes the extra --tests flag, a large number of other tests fail. This is independent of the changes in this PR.

A potential fix for the macos-latest, 1.47 build could be to change the CI to only build 1.47 with a fixed older version of macOS.

The fixes for the <platform>-latest, latest builds are discussed in https://github.com/technocreatives/core2/pull/21.

rmsyn avatar Aug 30 '23 04:08 rmsyn

Indeed. I'll clean it up if/when the maintainer shows up again :)

TheBlueMatt avatar Aug 30 '23 14:08 TheBlueMatt