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

2024-03-04 Thread Bruno Haible
Hi Collin, > The error should only be removed in mode == 'import' correct? Yes. Patch applied. Thanks! Bruno

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

2024-03-04 Thread Collin Funk
ode != 'import'. CollinFrom 5dd9572597ada0790d9b9d9911c433fd25dca092 Mon Sep 17 00:00:00 2001 From: Collin Funk Date: Mon, 4 Mar 2024 12:35:15 -0800 Subject: [PATCH] gnulib-tool.py: Follow gnulib-tool changes, part 39. Follow gnulib-tool change 2017-12-28 Bruno Haible gnulib-tool: Make --conditional-dependenc

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

2024-03-03 Thread Collin Funk
On 3/3/24 4:30 AM, Pádraig Brady wrote: > Right, join without options is essentially set intersection. > See https://www.pixelbeat.org/cmdline.html#sets Thanks for the examples. Bookmarked it since it will probably come handy as I work on gnulib-tool. :) Collin

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

2024-03-03 Thread Collin Funk
On 3/3/24 4:30 AM, Bruno Haible wrote: > In the original commit, I removed the error message > "option --conditional-dependencies is not supported with --with-tests" > only at one place (the command-line option checks), but left it in > func_import. > You are removing it in both places (mode ==

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

2024-03-03 Thread Bruno Haible
Hello Collin, > -for m in $modules; do echo $m; done | LC_ALL=C sort -u > "$tmp"/modules > +for m in $referenceable_modules; do echo $m; done | LC_ALL=C sort -u > > "$tmp"/modules > > Which I didn't understand until seeing this line [1]: > > deps=`for m in $deps; do echo $m; done |

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

2024-03-03 Thread Pádraig Brady
On 03/03/2024 05:33, Collin Funk wrote: On 3/2/24 7:02 PM, Collin Funk wrote: When working on the one of the TODO entries I noticed that the gnulib-comp.m4 output was incorrect. Here is the item I was working on. I _think_ I understood it correctly but you guys are much more talented at shell

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

2024-03-02 Thread Collin Funk
g something. [1] https://git.savannah.gnu.org/cgit/gnulib.git/tree/gnulib-tool?id=589e96475f8f2d21a83405ab0672ce95091b80e5#n4296 CollinFrom bf4cfc143601f17133238a32de68ca033fa1b757 Mon Sep 17 00:00:00 2001 From: Collin Funk Date: Sat, 2 Mar 2024 21:02:15 -0800 Subject: [PATCH] gnulib-tool.py: Foll