X-Recipient: archive-cygwin AT delorie DOT com
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
	=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; 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;
	 s=default; bh=px/dtMGv3XT3kKlWzdz5V6Ha7zY=; b=IDcAIhAJoGa/lDRMx
	Z0h2eiXOBmxZT8JyqU5UqDYePXsN6ZwGuHQA2uJijRMe9HCDbItA736/M5NxAr0G
	Qy5ZJSeJFHFRbK1nLvOyTzKoJMfgoiorvocY7RHD5EO5iXuOWtKQPb/yhN9S2kl6
	shuOe07JIbrjqCX/VONmpKGsKs=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=0.8 required=5.0 tests=AWL,BAYES_40,RDNS_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no version=3.3.2
X-HELO: out1-smtp.messagingengine.com
Message-ID: <529CB5A9.5030305@cwilson.fastmail.fm>
Date: Mon, 02 Dec 2013 11:30:33 -0500
From: Charles Wilson <cygwin AT cwilson DOT fastmail DOT 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 AT cygwin DOT com>
Subject: Re: [patch] cygwin-apps/run 1.0.3
References: <529AA451 DOT 4050408 AT gmail DOT com> <20131201030140 DOT GB2506 AT ednor DOT casa DOT cgf DOT 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