On 3/11/24 4:57 AM, Bruno Haible wrote:
> This looks good to me. 'sed' accepts POSIX BREs, where '(' and '[' stand for
> the literal '(' and '[' characters unless escaped.
Thanks for double checking for me.
> Also, I applied the attached follow-up, in order to make the logic of the code
> easier
Hi Collin,
> On 3/10/24 5:13 AM, Bruno Haible wrote:
> > automake_options = {x for y in configure_ac_automake_options for x in
> > y.split()}
>
> I've attached the fixed patch.
Thanks, applied.
> I would like to double check the regular expression I used:
>
> pattern = re.compile(r'^.*A
Hi Bruno,
On 3/10/24 5:13 AM, Bruno Haible wrote:
> automake_options = {x for y in configure_ac_automake_options for x in
> y.split()}
I've attached the fixed patch. The only change is turning this line
to:
automake_options = { x
for y in configure_ac_automake_opt
Hi Collin,
This one looks good, except please reformat the line
automake_options = {x for y in configure_ac_automake_options for x in
y.split()}
as suggested for part 51.
Bruno
On 3/9/24 6:28 PM, Collin Funk wrote:
> I think that should be a pretty easy change so I will try to get it
> done sometime today.
Not the cleanest code I have ever written but it seems to work. :)
Here is a test case:
gnulib-tool.py --create-testdir --dir test-python dummy
gnulib-tool --create-
