Re: [PATCH] Add shell-quasiquote.

[Random832]

Eli Zaretskii <address@hidden> writes:

>> From: Random832 <address@hidden>
>> It's also simpler than the POSIX section of shell-quote-argument.
>
> Simpler doesn't mean correct.

But it is correct.

>> I wonder what "really weird shells" this refers to?
>
> The set of characters special to an arbitrary shell is not known in
> advance.

And neither is its quoting mechanism. That tells me that this section of
the function is not, cannot be, and should not be in the business of
supporting "arbitrary shells". I was simply wondering what shells of the
set that it can reasonably support (e.g. POSIX shells, or at the
outside, shells that are likely to be used as "sh" on a unix-like
system) it is referring to as "really weird".

>> Certainly not csh, the mechanism it uses for newlines doesn't work
>> there.
>
> What did you try that didn't work with csh?

It's not a matter of trying anything. For all I know, maybe Emacs will
never try to execute a command with csh even if it is your SHELL...
that's certainly what I'd do. The point is I know that csh doesn't
accept this syntax, and will result in an error of "Unmatched '." But,
fine, if you want a test case:

(call-process "csh" nil t "csh" "-c"
              (concat "echo " (shell-quote-argument "hello\nworld")))
Unmatched '.
Unmatched '.
1