From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Schindelin Subject: Re: [PATCH] NO_PERL support Date: Fri, 3 Apr 2009 16:51:09 +0200 (CEST) Message-ID: References: <20090403T065545Z@curie.orbis-terrarum.net> <20090403132029.GC21153@coredump.intra.peff.net> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: "Robin H. Johnson" , Johannes Sixt , Git Mailing List To: Jeff King X-From: git-owner@vger.kernel.org Fri Apr 03 16:51:00 2009 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 1LpkjK-0007yJ-FU for gcvg-git-2@gmane.org; Fri, 03 Apr 2009 16:50:50 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933535AbZDCOst (ORCPT ); Fri, 3 Apr 2009 10:48:49 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933539AbZDCOss (ORCPT ); Fri, 3 Apr 2009 10:48:48 -0400 Received: from mail.gmx.net ([213.165.64.20]:43878 "HELO mail.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S933535AbZDCOsr (ORCPT ); Fri, 3 Apr 2009 10:48:47 -0400 Received: (qmail invoked by alias); 03 Apr 2009 14:48:44 -0000 Received: from pacific.mpi-cbg.de (EHLO pacific.mpi-cbg.de) [141.5.10.38] by mail.gmx.net (mp034) with SMTP; 03 Apr 2009 16:48:44 +0200 X-Authenticated: #1490710 X-Provags-ID: V01U2FsdGVkX1/XcABkG93XnEBdIU4hA9mt9ThbPbdvYkBk0Ak/77 Do/P6Tu7tGCduk X-X-Sender: schindelin@pacific.mpi-cbg.de In-Reply-To: <20090403132029.GC21153@coredump.intra.peff.net> User-Agent: Alpine 1.00 (DEB 882 2007-12-20) X-Y-GMX-Trusted: 0 X-FuHaFi: 0.59 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Hi, On Fri, 3 Apr 2009, Jeff King wrote: > On Fri, Apr 03, 2009 at 12:03:50AM -0700, Robin H. Johnson wrote: > > > Effects: > > - No perl-based scripts or code are installed: > > $SCRIPT_PERL > > git-instaweb > > gitweb > > git-cvsserver > > git-svn > > - git-add does NOT have interactive support. > > - None of the tests for the disabled stuff gets run. > > Hmm. With this patch, a user with a NO_PERL build will just get: > > $ git svn > git: 'svn' is not a git-command. See 'git --help'. > $ git add -i > error: unknown switch `i' > usage: ... > > I wonder if we should be a little nicer and say "we know about this > command or option, but we did not build support for it". OTOH, what you > have mirrors what NO_TCLTK does, so perhaps it is not a big deal. Actually, "add -i" is a part of another program that does work without Perl, so it is quite a different kind of kettle. It is not like "git gui" works with NO_TCLTK but "git gui -i" does not. Ciao, Dscho