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:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type
	:content-transfer-encoding; q=dns; s=default; b=qok4V8H1CKr5G6pJ
	gKTZJN704PdXTzLec6YGb/1r8Xo2LpRxRF+VpSm1iO5fLi2jnS2TLsiMvTKuA7kx
	fDBsTg1cKflQqgUCZ36lu0ROkoAVpMRY8j7laLrZtWzOPCRzltuUKXfdTRiBdlAG
	Bdc4cbXooVfVtsVM0UstnWoWLKY=
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:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type
	:content-transfer-encoding; s=default; bh=/Sv/HgrmYxCeb7rhW5og9+
	RjlXg=; b=tza16KAlax4qtoUbDfiz84ZIGfjLoFL3p7kxaxxA+7f+YLHAJPNBjy
	4dtYajOAPlWJkcSt0zyMvoxiVcH0jgMrUw4We/+hcI+BEyDN3RGahOYQsw4XPcIE
	tUF1VwA+mzOCreG9O3UDblOOXuuWJxkZGpO8kLy/3KX8p0tT+dsAE=
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-Spam-SWARE-Status:	No, score=1.5 required=5.0 tests=BAYES_20,KAM_LAZY_DOMAIN_SECURITY,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE autolearn=no version=3.3.2 spammy=strongly, ANNOUNCEMENT, andrey, Repin
X-HELO:	mout.kundenserver.de
Subject:	Re: [ANNOUNCEMENT] Updated: mintty 2.9.5
To:	cygwin AT cygwin DOT com
References:	<announce DOT ba14b773-7e0a-32fd-7a01-4979c60eeb4c AT towo DOT net> <87woon9133 DOT fsf AT Rainer DOT invalid> <a753939f-9724-296a-8ec5-643cd09b4f93 AT towo DOT net> <87in06la12 DOT fsf AT Rainer DOT invalid> <94206790 DOT 20181207003223 AT yandex DOT ru> <2f3cf57e-0b44-8c4a-b075-24743e4443d2 AT towo DOT net> <199122924 DOT 20181208004157 AT yandex DOT ru>
From:	Thomas Wolff <towo AT towo DOT net>
Message-ID:	<482194f6-37e5-6892-f63f-d7474cdab694@towo.net>
Date:	Mon, 10 Dec 2018 00:22:33 +0100
User-Agent:	Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.3.3
MIME-Version:	1.0
In-Reply-To:	<199122924.20181208004157@yandex.ru>
X-IsSubscribed:	yes

Am 07.12.2018 um 22:41 schrieb Andrey Repin:
> Greetings, Thomas Wolff!
>
>> Am 06.12.2018 um 22:32 schrieb Andrey Repin:
>>> Greetings, Achim Gratz!
>>>
>>>> a) Just warn about the missing PATH component without changing the PATH.
>>>> b) Give the user an option to let the command run with a separate PATH.
>>>> Indeed there might be other things that are missing in the environment,
>>>> so instead of just fixing up PATH you might consider using a wrapper
>>>> that the user can change.
>>> This is an interesting point.
>>> Probably letting the user specify custom $PATH for quick commands (and save
>>> current %PATH% as $ORIGINAL_PATH) may be the best choice.
>>> The default can be %Cygwin%\bin:%PATH% or %PATH%:%Cygwin%\bin, I'm not very
>>> concerned as long as it's explicit and customizable.
>> I didn't expect the setup of this specific feature to be so strongly
>> advocated :)
>> In any case, things shouldn't become too complex. What I can image is an
>> option UserCommandsPATH
>> with one optional placeholder %s which would be replaced with the
>> environment PATH;
>> default /bin:%s, corresponding to current behaviour.
> Using native paths and environment variable syntax, you can always use
> ExpandEnvironmentString to get the %PATH% you need.
> Cygwin DLL will then convert it to own version, avoiding you mangling it
> yourself.
> And I would strongly discourage from using placeholders familiar for you, but
> unexpected by the end user and taken from entirely different area of knowledge.
>
> BTW, Cygwin itself does it differently. %Cygwin%\bin is converted to /usr/bin.
This setup should be handled in the Posix path domain. I've uploaded a 
patch for evaluation, to the github repository.
Thomas

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