Update action cache to v5

Author: oech3

.github/workflows/GnuTests.yml

@@ -75,7 +75,7 @@ jobs:
         git submodule update --init --recursive --depth 1
       working-directory: gnu
     - name: Restore files for faster configure and skipping make
-      uses: actions/cache@v4
+      uses: actions/cache@v5
       id: cache-config-gnu
       with:
         path: |
@@ -128,7 +128,7 @@ jobs:
         env PROFILE=release-small bash util/build-gnu.sh
         
     - name: Save files for faster configure and skipping make
-      uses: actions/cache/save@v4
+      uses: actions/cache/save@v5
       if: always() && steps.cache-config-gnu.outputs.cache-hit != 'true'
       with:
         path: |

src/uu/readlink/src/readlink.rs

@@ -37,11 +37,14 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
     let silent = matches.get_flag(OPT_SILENT) || matches.get_flag(OPT_QUIET);
     let verbose = matches.get_flag(OPT_VERBOSE);
 
+    // GNU readlink -f/-e/-m follows symlinks first and then applies `..` (physical resolution).
+    // ResolveMode::Logical collapses `..` before following links, which yields the opposite order,
+    // so we choose Physical here for GNU compatibility.
     let res_mode = if matches.get_flag(OPT_CANONICALIZE)
         || matches.get_flag(OPT_CANONICALIZE_EXISTING)
         || matches.get_flag(OPT_CANONICALIZE_MISSING)
     {
-        ResolveMode::Logical
+        ResolveMode::Physical
     } else {
         ResolveMode::None
     };

tests/by-util/test_readlink.rs

@@ -3,7 +3,7 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 //
-// spell-checker:ignore regfile
+// spell-checker:ignore regfile parentdir
 
 use uutests::util::{TestScenario, get_root_path};
 use uutests::{at_and_ucmd, new_ucmd, path_concat, util_name};
@@ -68,6 +68,21 @@ fn test_canonicalize_missing() {
     assert_eq!(actual, expect);
 }
 
+#[test]
+#[cfg(unix)]
+fn test_canonicalize_symlink_before_parentdir() {
+    // GNU readlink follows the symlink first and only then evaluates `..`.
+    // Logical resolution would collapse `link/..` up front and return the current directory instead.
+    let (at, mut ucmd) = at_and_ucmd!();
+    at.mkdir("real");
+    at.mkdir("real/sub");
+    at.relative_symlink_dir("real/sub", "link");
+
+    let actual = ucmd.args(&["-f", "link/.."]).succeeds().stdout_move_str();
+    let expect = format!("{}/real\n", at.root_dir_resolved());
+    assert_eq!(actual, expect);
+}
+
 #[test]
 fn test_long_redirection_to_current_dir() {
     let (at, mut ucmd) = at_and_ucmd!();