From mboxrd@z Thu Jan  1 00:00:00 1970
X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org
X-Spam-Level: 
X-Spam-Status: No, score=0.8 required=5.0 tests=DATE_IN_PAST_12_24,
	T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.4
Received: (qmail 5339 invoked from network); 16 Feb 2022 10:31:49 -0000
Received: from 4ess.inri.net (216.126.196.42)
  by inbox.vuxu.org with ESMTPUTF8; 16 Feb 2022 10:31:49 -0000
Received: from duke.felloff.net ([216.126.196.34]) by 4ess; Tue Feb 15 15:37:23 -0500 2022
Message-ID: <9D2350CE4852CDE2926D71A87892444D@felloff.net>
Date: Tue, 15 Feb 2022 21:37:05 +0100
From: cinap_lenrek@felloff.net
To: 9front@9front.org
In-Reply-To: <2248BB2EC411F4F96C089DB72BB66C6C@wopr.sciops.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
List-ID: <9front.9front.org>
List-Help: <http://lists.9front.org>
X-Glyph: ➈
X-Bullshit: stable GPU self-signing locator 
Subject: Re: [9front] rc(1) pattern-matching regression
Reply-To: 9front@9front.org
Precedence: bulk

i see whats wrong.

we have to propagate the glob attribute thru lists
as well. before it was only handled for single words
and propagated thru concatenations... but didnt
consider lists... sorry :(

theres a patch that should fix it:

http://felloff.net/usr/cinap_lenrek/globfix.diff

the intend of the change was to not having to
bruteforce check the argument list for each Xsimple()
command for globs (which might have very large
argument lists coming from from a variable or
`{}, but instead only do the globbing
evaulation when actually needed (the expression
containing a glob, which is what the parser
knows).

--
cinap