DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:mime-version:to:subject
	:references:in-reply-to:content-type:content-transfer-encoding;
	 q=dns; s=default; b=njH9Rv4U+XeQp7TxWtyXGk36O3I0jCbV3KgxB6Ob3Z9
	KBuSOysU0gXO8sMLwwemgVrVncIR1kGglT5VULM3LHs8jL5cVRnb0wqyvn9K8XqY
	yH0eAxuCWN9k4d+Rw7LUf+5Qpz56Fjmf3rAFd6QHc5E63irPkCAH4aAm1W5mDQhk
	=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Message-ID: <529CB5A9.5030305@cwilson.fastmail.fm>
Date: Mon, 02 Dec 2013 11:30:33 -0500
From: Charles Wilson <cygwin@cwilson.fastmail.fm>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.1.1
MIME-Version: 1.0
To: The Cygwin Mailing List <cygwin@cygwin.com>
Subject: Re: [patch] cygwin-apps/run 1.0.3
References: <529AA451.4050408@gmail.com> <20131201030140.GB2506@ednor.casa.cgf.cx>
In-Reply-To: <20131201030140.GB2506@ednor.casa.cgf.cx>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 11/30/2013 10:01 PM, Christopher Faylor wrote:
> On Sat, Nov 30, 2013 at 09:52:01PM -0500, Max Polk wrote:
>> Follow-up to run 1.0.3 crashing and leaving run.exe.stackdump, first
>> reported at http://cygwin.com/ml/cygwin/2013-08/msg00169.html
>>
>> The list wouldn't let me send a patch because it had an email address in
>> the Changelog, so I had to put it here: http://pastebin.com/4SSPDGQh
>
> Actually the problem is that you shouldn't submit ChangeLog entries as
> diffs since they often don't apply cleanly.  Just send the ChangeLog
> entry.  Your email address wouldn't be blocked since it doesn't include
> 'cygwin'.
>
> Thanks for the patch though.  I'm sure Chuck will be happy to consider
> applying it.

Thanks for the patch.  I'm concerned that always quoting every argument 
might hurt file globbing (e.g. when you pass foo.* as an argument). 
Howevr, since what you have is clearly better than status quo, and fixes 
an actual regression where older versions of run /used/ to handle this 
situation fine...I'll roll a new update fairly soon.

--
Chuck


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple