From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Schindelin Subject: Re: [PATCH 0/3] Call builtin functions directly, was Re: [PATCH] transport.c: call dash-less form of receive-pack and upload-pack on remote Date: Sun, 2 Dec 2007 11:35:48 +0000 (GMT) Message-ID: References: <20071127150229.GA14859@laptop> <20071127160423.GA22807@laptop> <20071128000731.GD9174@efreet.light.src> <7v8x4jb295.fsf@gitster.siamese.dyndns.org> <7vfxyq2c9b.fsf@gitster.siamese.dyndns.org> <7vd4tsvfvk.fsf@gitster.siamese.dyndns.org> <7vlk8f9m52.fsf@gitster.siamese.dyndns.org> <7vzlwu43i4.fsf@gitster.siamese.dyndns.org> <7v3aul1xmt.fsf@gitster.siamese.dyndns.org> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Eyvind Bernhardsen , Nicolas Pitre , Nguyen Thai Ngoc Duy , Jan Hudec , git@vger.kernel.org To: Junio C Hamano X-From: git-owner@vger.kernel.org Sun Dec 02 12:36:45 2007 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1Iyn7s-0008Cp-BH for gcvg-git-2@gmane.org; Sun, 02 Dec 2007 12:36:44 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752225AbXLBLgV (ORCPT ); Sun, 2 Dec 2007 06:36:21 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751592AbXLBLgV (ORCPT ); Sun, 2 Dec 2007 06:36:21 -0500 Received: from mail.gmx.net ([213.165.64.20]:53552 "HELO mail.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751316AbXLBLgU (ORCPT ); Sun, 2 Dec 2007 06:36:20 -0500 Received: (qmail invoked by alias); 02 Dec 2007 11:36:18 -0000 Received: from unknown (EHLO openvpn-client) [138.251.11.103] by mail.gmx.net (mp051) with SMTP; 02 Dec 2007 12:36:18 +0100 X-Authenticated: #1490710 X-Provags-ID: V01U2FsdGVkX19279RFudV2S52LoBQ063Nu7OQwtfeA9LUBi0JFZT aTcWT3+J/CmZ0/ X-X-Sender: gene099@racer.site In-Reply-To: <7v3aul1xmt.fsf@gitster.siamese.dyndns.org> X-Y-GMX-Trusted: 0 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Hi, On Sat, 1 Dec 2007, Junio C Hamano wrote: > Johannes Schindelin writes: > > > Okay, I bit the apple and tried to move the builtins into the library, > > and rename handle_internal_command into execv_git_builtin(), moving it > > into exec-cmd.c. > > > > Big mistake. > > I really feel this should not go in. Hmm. My rationale was not only avoiding an exec() (which I would find worth it on its own), but because this would be a non-verbal step towards libification. Ciao, Dscho