Pass given file names to key-handler instead of real paths

Misbehaving command lines in the key-handler that rewrite the given files and
thus replace symbolic links instead of their targets shall call realpath(1).

Fixes issue #205 and reverts commit 92e3b578.
This commit is contained in:
Bert Münnich 2017-01-07 14:44:33 +01:00
parent 1c260e701b
commit 5c607ad495
3 changed files with 3 additions and 3 deletions

View file

@ -1,4 +1,4 @@
VERSION := git-20161228 VERSION := git-20170107
PREFIX := /usr/local PREFIX := /usr/local
MANPREFIX := $(PREFIX)/share/man MANPREFIX := $(PREFIX)/share/man

View file

@ -18,7 +18,7 @@ readonly TMPFILE="/tmp/sxiv.$$"
rotate() { rotate() {
degree="$1" degree="$1"
while read file; do tr '\n' \0' | xargs -0 realpath | sort | uniq | while read file; do
case "$(file -b -i "$file")" in case "$(file -b -i "$file")" in
image/jpeg*) jpegtran -rotate "$degree" -copy all -outfile "$file" "$file" ;; image/jpeg*) jpegtran -rotate "$degree" -copy all -outfile "$file" "$file" ;;
*) mogrify -rotate "$degree" "$file" ;; *) mogrify -rotate "$degree" "$file" ;;

2
main.c
View file

@ -525,7 +525,7 @@ void run_key_handler(const char *key, unsigned int mask)
for (f = i = 0; f < fcnt; i++) { for (f = i = 0; f < fcnt; i++) {
if ((marked && (files[i].flags & FF_MARK)) || (!marked && i == fileidx)) { if ((marked && (files[i].flags & FF_MARK)) || (!marked && i == fileidx)) {
stat(files[i].path, &oldst[f]); stat(files[i].path, &oldst[f]);
fprintf(pfs, "%s\n", files[i].path); fprintf(pfs, "%s\n", files[i].name);
f++; f++;
} }
} }