From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Scott R Parish" Subject: Re: [PATCH] use only the PATH for exec'ing git commands Date: Mon, 22 Oct 2007 14:59:06 -0500 (CDT) Message-ID: <1193083146.v2.fusewebmail-240137@f> Reply-To: srp@srparish.net Mime-Version: 1.0 Content-Type: text/plain;charset=iso-8859-1 To: "Alex Riesen" , git@vger.kernel.org X-From: git-owner@vger.kernel.org Mon Oct 22 21:59:37 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 1Ik3R1-0002as-Pe for gcvg-git-2@gmane.org; Mon, 22 Oct 2007 21:59:36 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753969AbXJVT7M (ORCPT ); Mon, 22 Oct 2007 15:59:12 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753304AbXJVT7L (ORCPT ); Mon, 22 Oct 2007 15:59:11 -0400 Received: from smtp-gw6.mailanyone.net ([208.70.128.57]:56808 "EHLO smtp-gw6.mailanyone.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752477AbXJVT7K (ORCPT ); Mon, 22 Oct 2007 15:59:10 -0400 Received: from mailanyone.net by smtp-gw6.mailanyone.net with esmtpa (MailAnyone extSMTP srp) id 1Ik3Qa-0000lL-Ns; Mon, 22 Oct 2007 14:59:09 -0500 Received: from 127.0.0.1 (FuseMail web AccountID 240137) by webmail.fusemail.com with HTTP; Mon, 22 Oct 2007 14:59:06 -0500 (CDT) User-Agent: FuseMail W X-Priority: 3 Importance: Normal Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: > Alex Reisen, Mon, Oct 22, 2007 12:01: > > + strbuf_addch(out, ':'); > > Shouldn't it break MingW32 native port? > > > + if (old_path) > > + strbuf_addstr(&new_path, old_path); > > + else > > + strbuf_addstr(&new_path, "/usr/local/bin:/usr/bin:/bin"); > > the default PATH is platform-dependent. Git is multi-platform. > You should consider putting the path list somewhere in Makefile, > config.mak or configure. > The original code was already doing both of these things (see git.c's prepend_to_path()) sRp