Changeset 3682 for trunk/src/kmk/kmkbuiltin/rm.c

Timestamp:

Aug 13, 2025, 1:34:19 AM (3 months ago)

Author:

bird

Message:

lib/nt,kmk: Fixed around rm/unlink semantics and general support for long file names in lib/nt.

File:

• trunk/src/kmk/kmkbuiltin/rm.c (modified) (5 diffs)

trunk/src/kmk/kmkbuiltin/rm.c

@@ -82 +82 @@
 #  include "mscfakes.h"
 # endif
-# include "nt/ntunlink.h"
-  /* Use the special unlink implementation to do rmdir too. */
-# undef  rmdir
-# define rmdir(a_pszPath)       birdUnlinkForced(a_pszPath)
+# include "nt/ntunlink.h" /* redefines both unlink & rmdir */
 #endif
 #if defined(__OS2__) || defined(_MSC_VER)
@@ -430 +427 @@
 #ifdef KBUILD_OS_WINDOWS
                                 if (p->fts_parent->fts_dirfd != NT_FTS_INVALID_HANDLE_VALUE) {
-                                    rval = birdUnlinkForcedEx(p->fts_parent->fts_dirfd, p->fts_name);
+                                    rval = birdRmDirForcedEx(p->fts_parent->fts_dirfd, p->fts_name);
                                 } else {
-                                    rval = birdUnlinkForced(p->fts_accpath);
+                                    rval = birdRmDirForced(p->fts_accpath);
                                 }
 #else
@@ -480 +477 @@
                                                 rval = birdUnlinkForcedFastEx(p->fts_parent->fts_dirfd, p->fts_name);
                                         } else { /* NtDeleteFile doesn't work on directory links, so slow symlink deletion: */
-                                                rval = birdUnlinkForcedEx(p->fts_parent->fts_dirfd, p->fts_name);
+                                                if (p->fts_stat.st_isdirsymlink) {
+                                                        rval = birdRmDirForcedEx(p->fts_parent->fts_dirfd, p->fts_name);
+                                                } else {
+                                                        rval = birdUnlinkForcedEx(p->fts_parent->fts_dirfd, p->fts_name);
+                                                }
                                         }
                                 } else {
@@ -486 +487 @@
                                                 rval = birdUnlinkForcedFast(p->fts_accpath);
                                         } else { /* NtDeleteFile doesn't work on directory links, so slow symlink deletion: */
-                                                rval = birdUnlinkForced(p->fts_accpath);
+                                                if (p->fts_stat.st_isdirsymlink) {
+                                                        rval = birdRmDirForced(p->fts_accpath);
+                                                } else {
+                                                        rval = birdUnlinkForced(p->fts_accpath);
+                                                }
                                         }
                                 }
@@ -586 +591 @@
                                 operation = "unlink";
 #else
-                                if (pThis->fUseNtDeleteFile) {
+                                /*if (sb.st_isdirsymlink) {
+                                        rval = birdRmDirForced(f);
+                                        operation = "rmdir";
+                                } else*/ if (pThis->fUseNtDeleteFile /*&& S_ISREG(sb.st_mode) && !sb.st_isdirsymlink*/) {
                                         rval = birdUnlinkForcedFast(f);
                                         operation = "NtDeleteFile";