DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org C7F4F3858D39
Content-Type: multipart/mixed; boundary="------------T0Sta3ma0oZtfkZDd8ZEc8Kn"
Message-ID: <2d41d3d5-f426-e7cc-eb07-7d3d6011f36d@Shaw.ca>
Date: Wed, 8 Feb 2023 07:20:13 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: Parameter expansion stopped working in recent ash/dash
Content-Language: en-CA
To: cygwin AT cygwin DOT com
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>
Organization: Inglis
In-Reply-To: <7890e200-4a40-5e1e-3a62-21f30d92d5c5@Shaw.ca>
Precedence: list
From: Brian Inglis via Cygwin <cygwin AT cygwin DOT com>
Reply-To: cygwin AT cygwin DOT com
Cc: Brian Inglis <Brian DOT Inglis AT Shaw DOT ca>, 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>

This is a multi-part message in MIME format.
--------------T0Sta3ma0oZtfkZDd8ZEc8Kn
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

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.

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.

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.

-- 
Take care. Thanks, Brian Inglis			Calgary, Alberta, Canada

La perfection est atteinte			Perfection is achieved
non pas lorsqu'il n'y a plus rien à ajouter	not when there is no more to add
mais lorsqu'il n'y a plus rien à retirer	but when there is no more to cut
			-- Antoine de Saint-Exupéry
--------------T0Sta3ma0oZtfkZDd8ZEc8Kn
Content-Type: text/plain; charset=UTF-8; name="test12-2.log"
Content-Disposition: attachment; filename="test12-2.log"
Content-Transfer-Encoding: base64

IyEvYmluL2Rhc2ggLXZ4Cgp0cmltKCkgewogICMgcmVtb3ZlIGxlYWRpbmcgd2hpdGVzcGFj
ZSBjaGFyYWN0ZXJzCiAgX190cmltPSIkeyojJHsqJSVbIVs6c3BhY2U6XV0qfX0iCiAgIyBy
ZW1vdmUgdHJhaWxpbmcgd2hpdGVzcGFjZSBjaGFyYWN0ZXJzCiAgcHJpbnRmICIlcyIgIiR7
X190cmltJSR7X190cmltIyMqWyFbOnNwYWNlOl1dfX0iCn0KCnRyaW0gIiBhbHBoYSIgfCBv
ZCAtdCB4MWEKKyB0cmltICBhbHBoYQorIF9fdHJpbT1hbHBoYQorIHByaW50ZiAlcyBhbHBo
YQorIG9kIC10IHgxYQowMDAwMDAwICA2MSAgNmMgIDcwICA2OCAgNjEKICAgICAgICAgIGEg
ICBsICAgcCAgIGggICBhCjAwMDAwMDUKdHJpbSAiYmV0YSAiIHwgb2QgLXQgeDFhCisgdHJp
bSBiZXRhIAorIF9fdHJpbT1iZXRhIAorIHByaW50ZiAlcyBiZXRhCisgb2QgLXQgeDFhCjAw
MDAwMDAgIDYyICA2NSAgNzQgIDYxCiAgICAgICAgICBiICAgZSAgIHQgICBhCjAwMDAwMDQK
dHJpbSAiDWdhbW1hCSIgfCBvZCAtdCB4MWEKKyB0cmltIA1nYW1tYQkKKyBfX3RyaW09Z2Ft
bWEJCisgcHJpbnRmICVzIGdhbW1hCisgb2QgLXQgeDFhCjAwMDAwMDAgIDY3ICA2MSAgNmQg
IDZkICA2MQogICAgICAgICAgZyAgIGEgICBtICAgbSAgIGEKMDAwMDAwNQoKdHJpbTIoKSB7
CiAgIyByZW1vdmUgbGVhZGluZyB3aGl0ZXNwYWNlIGNoYXJhY3RlcnMKICBfX3RyaW09IiR7
KiMjW1s6c3BhY2U6XV19IgogICMgcmVtb3ZlIHRyYWlsaW5nIHdoaXRlc3BhY2UgY2hhcmFj
dGVycwogIHByaW50ZiAiJXMiICIke19fdHJpbSUlW1s6c3BhY2U6XV19Igp9Cgp0cmltMiAi
IGFscGhhIiB8IG9kIC10IHgxYQorIHRyaW0yICBhbHBoYQorIF9fdHJpbT1hbHBoYQorIHBy
aW50ZiAlcyBhbHBoYQorIG9kIC10IHgxYQowMDAwMDAwICA2MSAgNmMgIDcwICA2OCAgNjEK
ICAgICAgICAgIGEgICBsICAgcCAgIGggICBhCjAwMDAwMDUKdHJpbTIgImJldGEgIiB8IG9k
IC10IHgxYQorIHRyaW0yIGJldGEgCisgX190cmltPWJldGEgCisgcHJpbnRmICVzIGJldGEK
KyBvZCAtdCB4MWEKMDAwMDAwMCAgNjIgIDY1ICA3NCAgNjEKICAgICAgICAgIGIgICBlICAg
dCAgIGEKMDAwMDAwNAp0cmltMiAiDWdhbW1hCSIgfCBvZCAtdCB4MWEKKyB0cmltMiANZ2Ft
bWEJCisgX190cmltPWdhbW1hCQorIHByaW50ZiAlcyBnYW1tYQorIG9kIC10IHgxYQowMDAw
MDAwICA2NyAgNjEgIDZkICA2ZCAgNjEKICAgICAgICAgIGcgICBhICAgbSAgIG0gICBhCjAw
MDAwMDUKCnRyaW0zKCkgewogIF9zcGFjZT0iJChlY2hvIC1uICdbXGJcdFxuXHZcZlxyIF0n
KSIKICAjIHJlbW92ZSBsZWFkaW5nIHdoaXRlc3BhY2UgY2hhcmFjdGVycwogIF9fdHJpbT0i
JHsqIyMkX3NwYWNlfSIKICAjIHJlbW92ZSB0cmFpbGluZyB3aGl0ZXNwYWNlIGNoYXJhY3Rl
cnMKICBwcmludGYgIiVzIiAiJHtfX3RyaW0lJSRfc3BhY2V9Igp9Cgp0cmltMyAiIGFscGhh
IiB8IG9kIC10IHgxYQorIHRyaW0zICBhbHBoYQorIG9kIC10IHgxYQorIGVjaG8gLW4gW1xi
XHRcblx2XGZcciBdCisgX3NwYWNlPVsICQoLDA0gXQorIF9fdHJpbT1hbHBoYQorIHByaW50
ZiAlcyBhbHBoYQowMDAwMDAwICA2MSAgNmMgIDcwICA2OCAgNjEKICAgICAgICAgIGEgICBs
ICAgcCAgIGggICBhCjAwMDAwMDUKdHJpbTMgImJldGEgIiB8IG9kIC10IHgxYQorIHRyaW0z
IGJldGEgCisgb2QgLXQgeDFhCisgZWNobyAtbiBbXGJcdFxuXHZcZlxyIF0KKyBfc3BhY2U9
WwgJCgsMDSBdCisgX190cmltPWJldGEgCisgcHJpbnRmICVzIGJldGEKMDAwMDAwMCAgNjIg
IDY1ICA3NCAgNjEKICAgICAgICAgIGIgICBlICAgdCAgIGEKMDAwMDAwNAp0cmltMyAiDWdh
bW1hCSIgfCBvZCAtdCB4MWEKKyB0cmltMyANZ2FtbWEJCisgb2QgLXQgeDFhCisgZWNobyAt
biBbXGJcdFxuXHZcZlxyIF0KKyBfc3BhY2U9WwgJCgsMDSBdCisgX190cmltPWdhbW1hCQor
IHByaW50ZiAlcyBnYW1tYQowMDAwMDAwICA2NyAgNjEgIDZkICA2ZCAgNjEKICAgICAgICAg
IGcgICBhICAgbSAgIG0gICBhCjAwMDAwMDUKCg==

--------------T0Sta3ma0oZtfkZDd8ZEc8Kn
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


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

--------------T0Sta3ma0oZtfkZDd8ZEc8Kn--