X-Recipient:	archive-cygwin AT delorie DOT com
DKIM-Filter:	OpenDKIM Filter v2.11.0 sourceware.org 58EA53858425
DKIM-Signature:	v=1; a=rsa-sha256; c=relaxed/relaxed; d=cygwin.com;
	s=default; t=1675945223;
	bh=9LqUFFkQmuE5ZI7lNBrhlCH6z4pNN+1JyHKxO/5g0Jk=;
	h=Date:To:Subject:In-Reply-To:References:List-Id:List-Unsubscribe:
	List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:
	From;
	b=acFv/r7hVUu2W/32jzBfa7m2RqGuc0qhAz+PajcjZjtb//BJaJL1Q3cuhL/cOMoZd
	5ExEVUZ0IKZtPha7n67v29VJKHO5IVfX1zhJIb2sp9mTmFIoUvy8cQwOAHuHkFdnmc
	HPgfZhRnjAlAtYBdT5Xf3/HT7bhwKpw35wimN86s=
X-Original-To:	cygwin AT cygwin DOT com
Delivered-To:	cygwin AT cygwin DOT com
DMARC-Filter:	OpenDMARC Filter v1.4.2 sourceware.org 726C53858428
X-Yandex-Fwd:	1
Date:	Thu, 9 Feb 2023 15:08:45 +0300
X-Mailer:	The Bat! (v9.3.4) Professional
Message-ID:	<355862424.20230209150845@yandex.ru>
To:	Brian Inglis <cygwin AT cygwin DOT com>, cygwin AT cygwin DOT com
Subject:	Re: Parameter expansion stopped working in recent ash/dash
In-Reply-To:	<2d41d3d5-f426-e7cc-eb07-7d3d6011f36d@Shaw.ca>
References:	<1552195602 DOT 20230207150759 AT yandex DOT ru>
	<230ea647-5228-46e1-f6cb-954db39d20c2 AT Shaw DOT ca>
	<7890e200-4a40-5e1e-3a62-21f30d92d5c5 AT Shaw DOT ca>
	<2d41d3d5-f426-e7cc-eb07-7d3d6011f36d AT Shaw DOT ca>
MIME-Version:	1.0
X-Spam-Status:	No, score=-1.3 required=5.0 tests=BAYES_00, DKIM_SIGNED,
	DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, KAM_THEBAT,
	NICE_REPLY_A, SPF_HELO_NONE, SPF_PASS,
	TXREP autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version:	SpamAssassin 3.4.6 (2021-04-09) on
	server2.sourceware.org
X-BeenThere:	cygwin AT cygwin DOT com
X-Mailman-Version:	2.1.29
List-Id:	General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe:	<https://cygwin.com/mailman/options/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive:	<https://cygwin.com/pipermail/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe:	<https://cygwin.com/mailman/listinfo/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
From:	Andrey Repin via Cygwin <cygwin AT cygwin DOT com>
Reply-To:	cygwin AT cygwin DOT com
Cc:	Andrey Repin <anrdaemon AT yandex DOT ru>
Errors-To:	cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com
Sender:	"Cygwin" <cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com>
X-MIME-Autoconverted:	from base64 to 8bit by delorie.com id 319CKkcj021932

Greetings, Brian Inglis!

> On 2023-02-07 12:01, Brian Inglis via Cygwin wrote:
>> On 2023-02-07 10:19, Brian Inglis via Cygwin wrote:
>>> On 2023-02-07 05:07, Andrey Repin via Cygwin wrote:
>>>> In the attached script, that I always run using dash, the expansion is
>>>> supposed to strip leading/trailing spaces from provided parameter.
>>>> The upgrade that took place after which the expansion stopped working is
>>>> libsolv:   - dash-0.5.11.5-1.any -> dash-0.5.12-1.any
>>>> Downgrading to 0.5.11.5 restored the expected behavior.
>>>> I failed to find anything relevant in the dash 0.5.12 patch notes, but perhaps
>>>> community could help?

>>> Looks like something in dash broke space trimming of any sort: see attached >> script and logs.
>>> I will see what upstream has to say for themselves, or about Cygwin.

>> Stupid errors in test script, didn't actually run trim3 anyway, redone, and that > and updated logs attached.
>> It appears the locale dependent [[:space:]] regexp no longer works but using > escapes generated by echo still does, see corrected trim3, so perhaps use those > for now, if you can, while I follow this upstream.
>> You may need to add escape sequences for any locale dependent spacing characters > to the space string, so please let us know if there are.

> Hi Andrey,

> A new test release 0.5.12-2 has been uploaded so please install and test
> that behaves as expected in past releases.

At least it no longer clears the input in real world tests.
So that's good for me now.

> The issue was caused as glibc fnmatch and glob now support locale dependent
> named character classes, equivalents, etc. as in grep, so the build now
> defaults to using the local libc fnmatch and glob, without any config test for which libc or supported features.

That's proving importance of proper testing yet again. >.<

> Cygwin winsup and newlib libc fnmatch and glob do not support those locale
> dependent named character classes, equivalents, etc. as in grep, so those
> features are no longer provided in dash by default.

> The new test release disables the build from using libc fnmatch and glob
> and use those provided by dash (possibly by or from gnulib), so those
> classes again appear to work as expected: see attached log.

> If there are no further negative reports by this weekend, I will replace
> the current stable release with this latest test release.


-- 
With best regards,
Andrey Repin
Thursday, February 9, 2023 15:05:35

Sorry for my terrible english...

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

- Raw text -