X-Spam-Check-By: sourceware.org
Message-ID: <43E8E00F.2060903@mscha.org>
Date: Tue, 07 Feb 2006 18:59:43 +0100
From: Michael Schaap <cygwin AT mscha DOT org>
Reply-To: cygwin AT cygwin DOT com
User-Agent: Thunderbird 1.5 (Windows/20051201)
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: strange cygstart bug with current Cygwin versions
References: <2fc802c00602031305p7574f255g5e67e44cd8b754e AT mail DOT gmail DOT com> <Pine DOT GSO DOT 4 DOT 63 DOT 0602031614330 DOT 3425 AT access1 DOT cims DOT nyu DOT edu> <2fc802c00602061329x56784831mb5ab16a12191b2e AT mail DOT gmail DOT com> <43E7D056 DOT 10505 AT mscha DOT org> <20060207023104 DOT GA2484 AT efn DOT org>
In-Reply-To: <20060207023104.GA2484@efn.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
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

On 7-Feb-2006 3:31, Yitzchak Scott-Thoennes wrote:
> On Mon, Feb 06, 2006 at 11:40:22PM +0100, Michael Schaap wrote:
>   
>> What we basically need to do, is copy the Cygwin environment to the
>> Windows environment, taking care of path conversion for all the
>> appropriate variables.
>>     
>
> Maybe start with:
>
> http://www.cygwin.com/ml/cygwin-patches/2005-q4/msg00009.html
>   

Not really, I think, especially since that code didn't make it into
Cygwin.  ;-)
(Also, Corinna suggested in that thread that an application should
simply convert the necessary variables itself.)

I did notice, though, that a new version, setup_winenv(), externally
callable as cygwin_internal(CW_SETUP_WINENV), was made available ...
last Thursday.  (Looks like the code was adapted from the cygstart code,
actually. :-) )
So, ideally, that function would be fixed to do POSIX to Windows
conversion on the necessary variables, and cygstart can then be changed
to call it, instead of its own setup_win_environ() function.

That would mean that the bug won't be fixed until 1.5.20 is released,
though ...
So, I guess I'll just add some path conversion handling to the cygstart
code itself, for now.  Then if at some point the Cygwin setup_winenv()
function is fixed in a similar way, and released, I might take it out
and call cygwin_internal(CW_SETUP_WINENV) instead.

If I don't see any other suggestions or objections, watch this space for
a patch.

 - Michael

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