From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: [PATCH] Add shell-quasiquote. Date: Sat, 07 Nov 2015 09:51:50 +0200 Message-ID: <83bnb6qmax.fsf@gnu.org> References: <87si59wj42.fsf@T420.taylan> <56267302.7050606@yandex.ru> <87io61igyu.fsf@T420.taylan> <56267CDF.6010201@yandex.ru> <87wpuhh15s.fsf@T420.taylan> <562683B9.1060305@yandex.ru> <83y4exe71v.fsf@gnu.org> <87y4ewoqbs.fsf@fastmail.com> <83si54e2qx.fsf@gnu.org> <87y4ew8fue.fsf@fastmail.com> <83zizcurco.fsf@gnu.org> <87k2qgw3q1.fsf@fastmail.com> <83lhawuny4.fsf@gnu.org> <22055.54617.848927.986392@turnbull.sk.tsukuba.ac.jp> <85a8rcrs8q.fsf@iznogoud.viz> <83fv14ukna.fsf@gnu.org> <854mhkrqwe.fsf@iznogoud.viz> <838u6wujeu.fsf@gnu.org> <85eggnuq9w.fsf@iznogoud.viz> <83fv13t07m.fsf@gnu.org> <87pp07vsxr.fsf@fencepost.gnu.org> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE X-Trace: ger.gmane.org 1446882738 19767 80.91.229.3 (7 Nov 2015 07:52:18 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 7 Nov 2015 07:52:18 +0000 (UTC) Cc: emacs-devel@gnu.org To: kgrossjo@emptydomain.de (Kai =?iso-8859-1?Q?Gro=DFjohann?=) Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Nov 07 08:52:09 2015 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1ZuyIB-0003vn-HJ for ged-emacs-devel@m.gmane.org; Sat, 07 Nov 2015 08:52:07 +0100 Original-Received: from localhost ([::1]:42482 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuyIA-000595-TO for ged-emacs-devel@m.gmane.org; Sat, 07 Nov 2015 02:52:06 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:49782) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuyI7-00058t-QQ for emacs-devel@gnu.org; Sat, 07 Nov 2015 02:52:04 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZuyI2-0006gW-RK for emacs-devel@gnu.org; Sat, 07 Nov 2015 02:52:03 -0500 Original-Received: from mtaout23.012.net.il ([80.179.55.175]:46390) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuyI2-0006gS-J7 for emacs-devel@gnu.org; Sat, 07 Nov 2015 02:51:58 -0500 Original-Received: from conversion-daemon.a-mtaout23.012.net.il by a-mtaout23.012.net.il (HyperSendmail v2007.08) id <0NXF00600PJGD500@a-mtaout23.012.net.il> for emacs-devel@gnu.org; Sat, 07 Nov 2015 09:51:57 +0200 (IST) Original-Received: from HOME-C4E4A596F7 ([84.94.185.246]) by a-mtaout23.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0NXF006GGPUKDD00@a-mtaout23.012.net.il>; Sat, 07 Nov 2015 09:51:57 +0200 (IST) In-reply-to: X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: Solaris 10 X-Received-From: 80.179.55.175 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:193497 Archived-At: > From: kgrossjo@emptydomain.de (Kai Gro=DFjohann) > Date: Sat, 07 Nov 2015 00:35:35 +0100 >=20 > One thing that could be improved, imho, is that shell-quote-argumen= t > looks at the system it's running on and quotes for that. But actua= lly > it should quote for the shell that receives the result, and it's no= t easy > to know what that shell is. >=20 > For example, especially on Windows it's conceivable that some folks > would like to quote for cmd.exe, others would like to quote for > Powershell, and still others are Cygwin aficionados and would like = to > quote for bash or zsh... >=20 > One idea might be that shell-quote-argument could look at shell-fil= e-name. > I have no idea whether that would fly. This already works as you describe in the Windows build, see w32-shell-dos-semantics.