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
Content-Type: text/plain; charset=KOI8-R
Content-Transfer-Encoding: 8bit
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