Hi Collin,

> Can you apply this patch for me?

Thanks, applied, with a tweak: I don't like the variable name 'group'
here. I know that 'tuple' is not possible either. But a couple of lines
below we call it a 'row', and that fits better (imagining the ignorelist
as a matrix).

> Then I am not convinced the check if we need to back up is correct at
> the moment:
> 
>     if test -s "$tmp"/ignore-added || test -s "$tmp"/ignore-removed;
> 
> turns into:
> 
>       if srcdata != destdata:

You're right: When ignore-added is empty and ignore-removed is non-empty,
it can happen that the .gitignore file won't change, thus gnulib-tool.sh
will advertise a file change and create a backup file when there is no
need. I'll look into fixing that.

Bruno




Reply via email to