Gabriel,

> > Both "apt install bash-completion" and "apt purge bash-completion"
> > generate the following warning:
> >       find: '/etc/bash_completion.d/': No such file or directory

> > The cause is that postinst and postrm assume that
> > /etc/bash_completion.d exists.

> Is it safe to upload such a change so close to the freeze?  I'm always
> worried about unforeseen side-effects.

The patch might be easier to understand by ignoring indentation:

diff -br a/debian/postinst b/debian/postinst
6a7
>         if [ -d /etc/bash_completion.d ]; then
19a21
>             fi
diff -br a/debian/postrm b/debian/postrm
7a8
>     if [ -d /etc/bash_completion.d ]; then
11a13
>     fi

I think it is safe, but I defer to you.

> ...
> Is this hunk needed?  The test (-d /etc/bash_completion.d/helpers) is
> not likely to produce the warnings you mentioned.

Absolutely.  The warnings come from the following line in both
post{inst,rm}:
        for f in $(find /etc/bash_completion.d/ -type f -name "*.dpkg-*");
do

Thank you!
Dam

Reply via email to