Re: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27.

2024-02-23 Thread Collin Funk
On 2/23/24 3:51 PM, Bruno Haible wrote: > I.e. you meant to write > mode != None > not > modules != None > ? Oops. I'm not sure how I missed this in your original email as well... You are correct. It should be "mode != None". I should have a patch ready in a bit to fix another item in the gnul

Re: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27.

2024-02-23 Thread Bruno Haible
Hi Collin, > > * gnulib-tool.py line 610: > > +if modules != None and "tests" in mode and gnu_make: > > ^^^ > > What is the rationale for this condition? It is not present in the > > original. Can > > it be removed? > > I added this condition in an attempt to mirror c

Re: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27.

2024-02-23 Thread Collin Funk
Hey Bruno, thanks for the feedback. On 2/23/24 5:08 AM, Bruno Haible wrote: > Just three small nits that you might want to revisit: > > * gnulib-tool.py line 610: > +if modules != None and "tests" in mode and gnu_make: > ^^^ > What is the rationale for this condition?

Re: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27.

2024-02-23 Thread Bruno Haible
Hello Collin, Collin Funk wrote: > Hello, here is a patch implementing the --gnu-make option for > gnulib-tool.py. Thanks! Applied. This was already a major piece of work. > All of these > commits were grouped together and were similar so I felt like it > didn't make too much sense to handle the

[PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27.

2024-02-22 Thread Collin Funk
ssed. Thanks, Collin [1] https://git.savannah.gnu.org/cgit/emacs.git/tree/admin/merge-gnulibFrom edf3dd7dc7f0d8c2b4bd77213554a236e23dca4e Mon Sep 17 00:00:00 2001 From: Collin Funk Date: Thu, 22 Feb 2024 20:16:18 -0800 Subject: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 27. Follow gnuli