Skip to content

Switch to astral-tokio-tar as maintained alternative #276

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

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

werdahias
Copy link
Contributor

No description provided.

@lafrenierejm
Copy link
Contributor

Tests are passing for me locally:

$ cargo test
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.20s
     Running unittests src/lib.rs (target/debug/deps/ripgrep_all-d3ad1bd3d97f01e9)

running 20 tests
test adapters::decompress::tests::test_inner_filename ... ok
test adapters::decompress::tests::gz ... ok
test adapters::postproc::tests::post1 ... ok
test adapters::postproc::tests::test_binary_content ... ok
test adapters::postproc::tests::test_postproc_prefix ... ok
test adapters::postproc::tests::test_utf16 ... ok
test adapters::postproc::tests::test_with_pagebreaks ... ok
test adapters::mbox::tests::mbox_simple ... ok
test adapters::postproc::tests::test_with_pagebreaks_chunks ... ok
test adapters::mbox::tests::mail_simple ... ok
test adapters::sqlite::test::simple ... ok
test adapters::custom::test::streaming ... ok
test preproc_cache::test::test_read_write ... ok
test adapters::postproc::tests::test_pdf_twoblank ... ok
test adapters::custom::test::poppler ... ok
test adapters::zip::test::recurse ... ok
test adapters::decompress::tests::pdf_gz ... ok
test adapters::mbox::tests::mbox_attachment ... ok
test adapters::tar::tests::test_simple_tar ... ok
test adapters::zip::test::only_seek_zip_fs ... ok

test result: ok. 20 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.24s

     Running unittests src/bin/rga.rs (target/debug/deps/rga-fe6f4f63a4dec120)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running unittests src/bin/rga-fzf.rs (target/debug/deps/rga_fzf-a88414f965bc54a8)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running unittests src/bin/rga-fzf-open.rs (target/debug/deps/rga_fzf_open-69c72235161c004b)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running unittests src/bin/rga-preproc.rs (target/debug/deps/rga_preproc-399acb0f55e31d41)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

   Doc-tests ripgrep_all

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

@phiresky
Copy link
Owner

the CI is failing with:

error: builder for '/nix/store/xcpyv18b5jds950afxk26jxn40s242qx-crate-audit-0.0.0.drv' failed with exit code 2;
       last 10 log lines:
       > default configurePhase, nothing to do
       > Running phase: buildPhase
       > ++ command cargo --version
       > cargo 1.80.0 (376290515 2024-07-16)
       > ++ command cargo audit -n -d /nix/store/ncwvs3i2ywdibfgqcf00jjdyyqhifcmq-source --ignore yanked
       >       Loaded 651 security advisories (from /nix/store/ncwvs3i2ywdibfgqcf00jjdyyqhifcmq-source)
       > error: not found: Couldn't load Cargo.lock
       > Caused by:
       >   -> I/O operation failed: parse error: parse error: invalid Cargo.lock format version: `4`
       >
       For full logs, run 'nix log /nix/store/xcpyv18b5jds950afxk26jxn40s242qx-crate-audit-0.0.0.drv'.

maybe the cargo/rustc version needs to be updated?

@werdahias
Copy link
Contributor Author

The lockfile was generated using rustc 1.84 on Debian unstable

@lafrenierejm
Copy link
Contributor

the CI is failing...
maybe the cargo/rustc version needs to be updated?

Indeed. One minute and I'll submit a PR to update the Flake inputs for newer versions of cargo and rustc.

@lafrenierejm
Copy link
Contributor

@werdahias The Nix inputs have just been updated in master. Please rebase when you get a chance so CI can re-run with a supported version of Cargo.

@werdahias
Copy link
Contributor Author

done.

@lafrenierejm
Copy link
Contributor

nix flake check is passing on my local x86_64-linux system, so there's a very high probability CI will be green.

@phiresky phiresky merged commit 987648c into phiresky:master Feb 28, 2025
1 check passed
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