mv: support moving folder containing symlinks to different filesystem

[yuankunzhang]

Closes #8586.

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

[github-actions]

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/misc/stdbuf (passes in this run but fails in the 'main' branch)

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/stdbuf (passes in this run but fails in the 'main' branch)

[sylvestre]

please remove the ignore
we do run some tests as root in the ci

[yuankunzhang]

I can't seem to make the tests work. Do you mind take a look at the error outputs and provide suggestions? Thanks.

[yuankunzhang]

I think the sudo executable is not available in the test. Even a simple sudo ls would fail:

run: sudo -E --non-interactive ls

thread 'test_mv::test_mv_broken_symlink_to_another_fs' panicked at tests/uutests/src/lib/util.rs:1976:37:
called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }

https://github.com/uutils/coreutils/actions/runs/18166245128/job/51709489403?pr=8605#step:14:4625

[oech3]

Linux have /tmp by default. So sudo is not needed for the test.

[yuankunzhang]

@sylvestre Plesae suggest what we could do about the sudo issue here.

[oech3]

coreutils/tests/by-util/test_cp.rs

Lines 2443 to 2446 in b5561e7

	// Test must be run as root (or with `sudo -E`)
	if scene.cmd("whoami").run().stdout_str() != "root\n" {
	return;
	}

[oech3]

@cakebaker #8605 (comment) Is this standard way for tests with sudo at here?

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)

[github-actions]

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

[codspeed-hq]

CodSpeed Performance Report

Merging #8605 will improve performances by 2.22%

_{Comparing yuankunzhang:mv-folder-containing-symlinks-across-fs (9340bf4) with main (4ee588e)}

Summary

⚡ 1 improvement
✅ 125 untouched
⏩ 6 skipped¹

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
⚡	cp_large_file[16]	344.1 µs	336.7 µs	+2.22%

Footnotes

1. 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. ↩

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

[github-actions]

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)

[oech3]

Still failing? Would you rerun test?
[Edit] Thankyou.

[github-actions]

GNU testsuite comparison:

Congrats! The gnu test tests/cksum/cksum-base64-untagged is no longer failing!