uutils / findutils

Rust implementation of findutils
MIT License
314 stars 38 forks source link

samefile: Cache FileInformation for the target #409

Closed tavianator closed 5 months ago

tavianator commented 5 months ago

And don't follow symlinks.

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 63.63636% with 4 lines in your changes missing coverage. Please review.

Project coverage is 65.55%. Comparing base (f145469) to head (7c8f6f3).

Files Patch % Lines
src/find/matchers/samefile.rs 50.00% 1 Missing and 2 partials :warning:
src/find/matchers/mod.rs 66.66% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #409 +/- ## ========================================== - Coverage 65.59% 65.55% -0.05% ========================================== Files 34 34 Lines 3941 3942 +1 Branches 901 901 ========================================== - Hits 2585 2584 -1 - Misses 1001 1002 +1 - Partials 355 356 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 5 months ago

GNU testsuite comparison:

Run BFS tests: Changes from main: PASS +4 / SKIP +0 / FAIL -4
Run BFS tests: BFS tests summary = TOTAL: 288 / PASS: 190 / SKIP: 1 / FAIL: 97
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O0 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O1 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O2 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O3 is now passing!
Run GNU findutils tests: GNU tests summary = TOTAL: 702 / PASS: 445 / FAIL: 254 / ERROR: 2
Run GNU findutils tests: Changes from main: PASS +4 / FAIL -8 / ERROR +0 / SKIP +0 
sylvestre commented 5 months ago

@hanbings wdyt ? :)

github-actions[bot] commented 5 months ago

GNU testsuite comparison:

Run BFS tests: Changes from main: PASS +4 / SKIP +0 / FAIL -4
Run BFS tests: BFS tests summary = TOTAL: 288 / PASS: 190 / SKIP: 1 / FAIL: 97
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O0 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O1 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O2 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O3 is now passing!
Run GNU findutils tests: GNU tests summary = TOTAL: 702 / PASS: 445 / FAIL: 254 / ERROR: 2
Run GNU findutils tests: Changes from main: PASS +4 / FAIL -8 / ERROR +0 / SKIP +0 
github-actions[bot] commented 5 months ago

Commit 7c8f6f346ca426485be910a55e20470bbd378b67 has GNU testsuite comparison:

Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O0 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O1 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O2 is now passing!
Run GNU findutils tests: Congrats! The GNU test samefile-p-brokenlink.new-O3 is now passing!
Run GNU findutils tests: GNU tests summary = TOTAL: 702 / PASS: 445 / FAIL: 254 / ERROR: 2
Run GNU findutils tests: Changes from main: PASS +4 / FAIL -8 / ERROR +0 / SKIP +0 
Run BFS tests: Changes from main: PASS +4 / SKIP +0 / FAIL -4
Run BFS tests: BFS tests summary = TOTAL: 288 / PASS: 190 / SKIP: 1 / FAIL: 97