Mailing-List: contact cygwin-help@sourceware.cygnus.com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@sources.redhat.com Delivered-To: mailing list cygwin@sources.redhat.com Date: Tue, 26 Jun 2001 15:37:00 -0400 From: Christopher Faylor To: cygwin@cygwin.com Subject: Re: Patch: Bug: Shell fork/exec avoidance in make-3.79.1-3 Message-ID: <20010626153700.A15819@redhat.com> Reply-To: cygwin@cygwin.com Mail-Followup-To: cygwin@cygwin.com References: <3B38CBFA.97932AC8@agilent.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.3.11i In-Reply-To: <3B38CBFA.97932AC8@agilent.com>; from earl_chew@agilent.com on Tue, Jun 26, 2001 at 10:52:58AM -0700 On Tue, Jun 26, 2001 at 10:52:58AM -0700, Earl Chew wrote: >I've included a patch below that changes all the char[] definitions >in job.c to char*. Thanks for the patch. I think that the simple solution of defining default_shell as a 'char *' under cygwin is the correct approach to this, though. Looking at the rest of the code in make, it seems like the intent is for this to be either an array or a pointer, depending on various ifdefs. That's what I've done to the code. I'll be announcing a new version of make shortly. I very much appreciate your tracking this down. It eases a maintainer's load considerably when people are willing to debug problems like this themselves. Thanks again, cgf -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Bug reporting: http://cygwin.com/bugs.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/