]> git.eshelyaron.com Git - emacs.git/commit
Add ‘nofollow’ flag to set-file-times
authorPaul Eggert <eggert@cs.ucla.edu>
Sat, 7 Mar 2020 20:04:05 +0000 (12:04 -0800)
committerPaul Eggert <eggert@cs.ucla.edu>
Sat, 7 Mar 2020 20:15:43 +0000 (12:15 -0800)
commit5d4cf1fef85bc24bc4cd9705ebb14150263ad707
treeaf696ed3ba7d2d0ab31951eba9482443d36c1456
parent9f4b260c2b98ea05a02e0ab7213156ce2e60e5a9
Add ‘nofollow’ flag to set-file-times

This is a companion to the recent set-file-modes patch.
It adds support for a ‘nofollow’ flag to set-file-times (Bug#39773).
Like the set-file-modes patch, it needs work in the w32 port.
* admin/merge-gnulib (GNULIB_MODULES): Add futimens, utimensat.
Remove utimens.
* doc/lispref/files.texi (Changing Files):
* etc/NEWS: Mention the change.
* lib/gnulib.mk.in, m4/gnulib-comp.m4: Regenerate.
* lisp/files.el (copy-directory):
* lisp/gnus/gnus-cloud.el (gnus-cloud-replace-file):
* lisp/net/tramp-adb.el (tramp-adb-handle-copy-file):
* lisp/net/tramp-smb.el (tramp-smb-handle-copy-file):
* lisp/tar-mode.el (tar-copy):
* test/lisp/filenotify-tests.el (file-notify-test03-events):
* test/lisp/files-tests.el:
(files-tests-file-name-non-special-set-file-times):
* test/lisp/net/tramp-tests.el (tramp-test22-file-times):
When setting file times, avoid following symbolic links
when the file is not supposed to be a symbolic link.
* lib/futimens.c, lib/utimensat.c, m4/futimens.m4, m4/utimensat.m4:
New files, copied from Gnulib.
* lisp/gnus/gnus-cloud.el (gnus-cloud-replace-file):
When creating a file that is not supposed to exist already,
use the excl flag to check this.
* lisp/net/tramp-adb.el (tramp-adb-handle-set-file-times):
* lisp/net/tramp-sh.el (tramp-sh-handle-set-file-times):
* lisp/net/tramp-sudoedit.el (tramp-sudoedit-handle-set-file-times):
Accept an optional FLAG arg that is currently ignored,
and add a FIXME comment for it.
* lisp/net/tramp-gvfs.el (tramp-gvfs-handle-set-file-times):
* src/fileio.c (Fset_file_times):
Support an optional FLAG arg.
* src/fileio.c (Fcopy_file): Use futimens instead of set_file_times,
as it’s simpler and is a POSIX API.
* src/sysdep.c (set_file_times): Move from here ...
* src/w32.c (set_file_times): ... to here, and make it static,
since it is now used only in w32.c.  Presumably w32.c should also
add support for futimens and utimensat (the POSIX APIs, which
Emacs now uses) and it can remove fdutimens (the Gnulib API,
which Emacs no longer uses).
24 files changed:
admin/merge-gnulib
doc/lispref/files.texi
etc/NEWS
lib/futimens.c [new file with mode: 0644]
lib/gnulib.mk.in
lib/utimensat.c [new file with mode: 0644]
lisp/files.el
lisp/gnus/gnus-cloud.el
lisp/net/tramp-adb.el
lisp/net/tramp-gvfs.el
lisp/net/tramp-sh.el
lisp/net/tramp-smb.el
lisp/net/tramp-sudoedit.el
lisp/tar-mode.el
m4/futimens.m4 [new file with mode: 0644]
m4/gnulib-comp.m4
m4/utimensat.m4 [new file with mode: 0644]
src/fileio.c
src/sysdep.c
src/systime.h
src/w32.c
test/lisp/filenotify-tests.el
test/lisp/files-tests.el
test/lisp/net/tramp-tests.el