Tomi Ollila <tomi.ollila@iki.fi> writes: > 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 >:) Pushed as part of 0.27_rc1 d _______________________________________________ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch