[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add shell-quasiquote.
From: |
Taylan Ulrich Bayırlı/Kammer |
Subject: |
Re: [PATCH] Add shell-quasiquote. |
Date: |
Sun, 18 Oct 2015 15:35:07 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eli Zaretskii <address@hidden> writes:
>> From: address@hidden (Taylan Ulrich Bayırlı/Kammer)
>> Date: Sun, 18 Oct 2015 00:22:33 +0200
>> Cc: address@hidden
>>
>> > On POSIX shells, shell-quote-argument is just as safe as
>> > shqq--quote-string, and on non-POSIX shells it works better. So it's a
>> > win, in both readability and in portability, to use
>> > shell-quote-argument.
>>
>> Fixing it does not seem easy at all given I can't trust
>> shell-quote-argument.
>
> You can trust it.
>
>> And please be realistic in the amount of trust we can put on the
>> complicated implementations for non-Unix shells. I can't judge them
>> myself since I don't know the syntax of those shells at all. Does
>> anyone here know their syntax comprehensively, or checked the
>> implementation against the documentation of those shells?
>
> Yes, we do. Yes, we have.
>
>> (Actually, those things should be documented for shell-quote-argument,
>> and my documentation would point there.)
>
> What do you think should be documented there that is missing?
I just filed bug #21702 about this.
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=21702
Taylan
- Re: [PATCH] Add shell-quasiquote., (continued)
- Re: [PATCH] Add shell-quasiquote., Paul Eggert, 2015/10/17
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/17
- Re: [PATCH] Add shell-quasiquote., Paul Eggert, 2015/10/17
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/17
- Re: [PATCH] Add shell-quasiquote., Paul Eggert, 2015/10/17
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/18
- Re: [PATCH] Add shell-quasiquote., Eli Zaretskii, 2015/10/18
- Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/18
- Re: [PATCH] Add shell-quasiquote., John Wiegley, 2015/10/18
- Re: [PATCH] Add shell-quasiquote., Eli Zaretskii, 2015/10/17
- Re: [PATCH] Add shell-quasiquote.,
Taylan Ulrich Bayırlı/Kammer <=
Re: [PATCH] Add shell-quasiquote., Random832, 2015/10/17
Re: [PATCH] Add shell-quasiquote., Artur Malabarba, 2015/10/17
Re: [PATCH] Add shell-quasiquote., Taylan Ulrich Bayırlı/Kammer, 2015/10/19