> shell-file-name is /bin/bash.

That's the bug.  `shell-file-name' should be /bin/sh.
As for your patch: it's an improvement because it relies less on the shell
It could actually avoid the shell altogether now, which would avoid
quoting problems.  E.g.:

  "%p -rotate %d -copy all -outfile %t \"%o\""

obviously, this assumes that quoting is done via the \"%o\" which
breaks if %o contains ".


        Stefan


PS: Also if %o assumes the quoting is done in tumme-cmd-rotate-original-options
rather than in %o itself, then I guess that if %d or %p contains metachars
like >, &, ;, SPC, etc... we may get very funny behavior.  It's probably
not a problem for those because they're under tight control (or it might
even be a desirable feature so that %p can expand to a command with some
args).


_______________________________________________
emacs-pretest-bug mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug

Reply via email to