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:mime-version:in-reply-to:references:date
	:message-id:subject:from:to:content-type
	:content-transfer-encoding; q=dns; s=default; b=W2+zwPbcEbFVE4SB
	FtOwaY+Rnk5kkR5lC3527f4wLddoByNvAXxJP5il0D9etb3ZFHinsEz3FDgtJo4i
	r/6ef9Hwe13Co2K8H8SXVH2cIE6BMwsSzHDAuIAMYMOynh5ccoApRC/+28d8CywK
	JlzN/K/xb5FvvQY2doIS/U9Ymg0=
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:mime-version:in-reply-to:references:date
	:message-id:subject:from:to:content-type
	:content-transfer-encoding; s=default; bh=dvfgLsBhwjcVc9BAZH8xx7
	G17ok=; b=e9ln75wNtZ9dutoBowME5UCVP4ZbSUFxPD6DBqxP0WzujHkOQX2mPR
	R7lPl5oaEO5dDIisdZsvCFVq2hYhNLKcMEth4c1bdKbJM3CxW1ytiufX/jZzITtg
	Bzhrg9slrIdTo39SOI8Ewk0bIpCP5o/1H9UWxHKOvj2qrHXkcui30=
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
X-Spam-SWARE-Status:	No, score=1.5 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,KHOP_THREADED,MIME_CHARSET_FARAWAY,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE,SPF_PASS autolearn=ham version=3.3.1
MIME-Version:	1.0
X-Received:	by 10.180.36.147 with SMTP id q19mr10512497wij.64.1371639878332; Wed, 19 Jun 2013 04:04:38 -0700 (PDT)
In-Reply-To:	<20130619101227.GL1620@calimero.vinschen.de>
References:	<CABEPuQJDLjtbcLig1isTUJgb6RBCD8LNShbm9mTPcb9WM5S5fw AT mail DOT gmail DOT com> <51C0B08E DOT 8080900 AT etr-usa DOT com> <CABEPuQJJpRfPKSwZ7M0eTOdp1HxDcmvuy1=qXFHBw-8kLkZ1ZQ AT mail DOT gmail DOT com> <51C0D956 DOT 4090905 AT etr-usa DOT com> <20130619020234 DOT GA3669 AT ednor DOT casa DOT cgf DOT cx> <CABEPuQKtytz=Yjw=-S4LOKm8VGCOfa3hSgdpnhkxaWye1GoVaQ AT mail DOT gmail DOT com> <20130619101227 DOT GL1620 AT calimero DOT vinschen DOT de>
Date:	Wed, 19 Jun 2013 15:04:38 +0400
Message-ID:	<CABEPuQJF51dJ72mv=A_MS3AOR8Lq_Htt9=D98fB_xSB3jCszEA@mail.gmail.com>
Subject:	Re: Adding MSYS functionality to Cygwin
From:	Alexey Pavlov <alexpux AT gmail DOT com>
To:	cygwin AT cygwin DOT com
X-MIME-Autoconverted:	from quoted-printable to 8bit by delorie.com id r5JB4ujX024976

2013/6/19 Corinna Vinschen :
> On Jun 19 10:53, Алексей Павлов wrote:
>> Today I investigate in this direction and find that logic works well
>> except one line in spawn.cc that I think can be fixed without break
>> anything.
>>
>> Index: cygwin/spawn.cc
>> ===================================================================
>> RCS file: /cvs/src/src/winsup/cygwin/spawn.cc,v
>> retrieving revision 1.345
>> diff -u -p -r1.345 spawn.cc
>> --- cygwin/spawn.cc 3 May 2013 19:39:01 -0000 1.345
>> +++ cygwin/spawn.cc 19 Jun 2013 05:53:36 -0000
>> @@ -406,7 +406,7 @@ child_info_spawn::worker (const char *pr
>> }
>> else
>> {
>> - if (wascygexec)
>> + if (real_path.iscygexec ())
>
> Line 374:
>
>   wascygexec = real_path.iscygexec ();
>
> Do you have an example why your change should make a difference?
>
My opinion is next:
wascygexec is initialized before calling av::fixup that determine is
executable depends on Cygwin1.dll and always (for me) is 0. But in
av::fixup real_path.iscygexec () can be changed.
And code always go to condition with one_line.fromargv.

Regards, Alexey.

--
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

- Raw text -