tsort: gnu misc tsort.pl
Summary
Align tsort with GNU behavior: silently accept -w, reorder successors to produce GNU’s deterministic output, and surface loop errors with the new util: error: … prefix via clap_localization::print_prefixed_error. Adjust uniq, comm, tsort, and chroot tests to expect the prefixed error strings so they continue to pass with the updated localization handling. Keep the test fixtures and assertions consistent with the updated message format across the suite.
Tests
cargo test test_tsort cargo test test_uniq (including gnu_tests) cargo test test_chroot --features chroot
GNU testsuite comparison:
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)
CodSpeed Performance Report
Merging #9289 will improve performances by 2.03%
Comparing mattsu2020:tsort_compatibility (7d21d9c) with main (0f5ab81)
Summary
⚡ 1 improvement
✅ 125 untouched
⏩ 6 skipped[^skipped]
Benchmarks breakdown
| Benchmark | BASE |
HEAD |
Change | |
|---|---|---|---|---|
| ⚡ | du_max_depth_balanced_tree[(6, 4, 10)] |
33.1 ms | 32.4 ms | +2.03% |
| [^skipped]: 6 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. |
GNU testsuite comparison:
Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)
is it expected that tsort.pl doesn't pass yet? thanks
GNU testsuite comparison:
Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/misc/tsort is no longer failing!
GNU testsuite comparison:
Congrats! The gnu test tests/misc/tsort is no longer failing!
GNU testsuite comparison:
Congrats! The gnu test tests/misc/tsort is no longer failing!