On Wed, May 30 2018, David Bremner wrote: > Tomi Ollila <tomi.ollila@iki.fi> writes: > >> On Tue, May 29 2018, David Bremner wrote: >> >>> If e.g. /lib is a link to /usr/lib, then the latter may not show up in >>> the way we expect in the output of ldconfig. 'test foo -ef bar' checks >>> if foo and bar have the same device and inode numbers. Since (at least >>> in bash, ksh, and zsh) the shell dereferences symlinks before applying >> >> i think you have to add 'dash' to this list in this commit message, as it >> is default /bin/sh in debian (and many debian-derived distributions); >> that is quite common configuration there when the test line in question >> is executed > > That's right. I even tested dash, _after_ I sent the patch :P > > What do you think about the risk/benefit of including this in 0.27? I > guess we have lived with this bug for a long time. +1 for inclusion. that's how we get testers >:) > > d _______________________________________________ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch