From mboxrd@z Thu Jan 1 00:00:00 1970 Path: main.gmane.org!not-for-mail From: David Kastrup Newsgroups: gmane.emacs.devel Subject: Re: Patch: new function process-file (call-process with file handlers) Date: Mon, 18 Oct 2004 12:24:40 +0200 Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Message-ID: References: <863c0dnxxe.fsf@ketchup.de.uu.net> <871xfxnuyv.fsf-monnier+emacs@gnu.org> NNTP-Posting-Host: deer.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1098095127 987 80.91.229.6 (18 Oct 2004 10:25:27 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Mon, 18 Oct 2004 10:25:27 +0000 (UTC) Cc: Kai Grossjohann , Stefan Monnier , emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Mon Oct 18 12:25:17 2004 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by deer.gmane.org with esmtp (Exim 3.35 #1 (Debian)) id 1CJUhZ-0003rq-00 for ; Mon, 18 Oct 2004 12:25:17 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.33) id 1CJUoq-0003xl-J2 for ged-emacs-devel@m.gmane.org; Mon, 18 Oct 2004 06:32:48 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.33) id 1CJUoc-0003xB-2g for emacs-devel@gnu.org; Mon, 18 Oct 2004 06:32:34 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.33) id 1CJUoY-0003wk-FN for emacs-devel@gnu.org; Mon, 18 Oct 2004 06:32:30 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.33) id 1CJUoY-0003wV-7Z for emacs-devel@gnu.org; Mon, 18 Oct 2004 06:32:30 -0400 Original-Received: from [199.232.76.164] (helo=fencepost.gnu.org) by monty-python.gnu.org with esmtp (Exim 4.34) id 1CJUh2-0004eM-Bs for emacs-devel@gnu.org; Mon, 18 Oct 2004 06:24:44 -0400 Original-Received: from localhost ([127.0.0.1] helo=lola.goethe.zz) by fencepost.gnu.org with esmtp (Exim 4.34) id 1CJUh1-0004L9-L3; Mon, 18 Oct 2004 06:24:43 -0400 Original-To: storm@cua.dk (Kim F. Storm) In-Reply-To: (Kim F. Storm's message of "Mon, 18 Oct 2004 10:28:40 +0200") User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/21.3.50 (gnu/linux) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 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 Xref: main.gmane.org gmane.emacs.devel:28554 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:28554 storm@cua.dk (Kim F. Storm) writes: > Stefan Monnier writes: > >>> The attached patch provides the new function process-file, adds Tramp >>> support for this new function, and changes VC to invoke process-file >>> instead of call-process. The patch also removes the ugly advice from >>> Tramp if process-file is available. >> >> I think you attached the wrong file. In any case, I think now is not the >> right moment to discuss such enhancements. > > The addition seems trivial, and since it helps tramp to avoid using > defadvice I support adding it (with proper docs of course). That's the main rub right now. The code itself seems comparatively small, but additions to the manual in sections that have already been verified need to get checked for correctness and consistency. -- David Kastrup, Kriemhildstr. 15, 44793 Bochum