On Wed, 6 Mar 2013, Mikołaj Izdebski wrote:

lbzip2 development (git master)
                        -z   -zc  -zf  -zcf  -d  -dc -df  -dcf -t  -tf
regular file             1    1    1    1     1   1   1    1    1   1
symlink to regular file       M    1    1         M   1    1    M   1
device                        M    1    1         M   1    1    M   1
symlink to device             M    1    1         M   1    1    M   1

Gentoo lbzip2
                        -z   -zc  -zf  -zcf  -d  -dc -df  -dcf -t  -tf
regular file             1    1    1    1     1   1   1    1    1   1
symlink to regular file            1    1     G   G   1    1    G   1
device                             1    1     G   G   1    1    G   1
symlink to device                  1    1     G   G   1    1    G   1

These help the discussion greatly, thanks!

On the surface, neither "relaxation patch" seems to be a subset of the other. However, what Gentoo actually cares about is -dc (IIRC); they need it for some kind of automated installation procedure (part of emerge or some such). So, Gentoo's relaxation of the plain -d case is actually wrong (I recall noticing that when I had seen their patch for the first time), but in practice they don't invoke plain -d on non-regular files, so it doesn't bother them probably.

Hence my observation: you shouldn't need to adopt anything from the Gentoo patch (your patch is correct & complete), but your commit actually covers the one case Gentoo cares about, so they could drop their out-of-tree patch when upgrading their package next time.

(Ugh, apologies, Debian, we should really be discussing this in the Gentoo bugzilla.)

Laszlo


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to