From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id 0F1646DE0A9A for ; Sat, 2 Jun 2018 04:12:56 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: -0.001 X-Spam-Level: X-Spam-Status: No, score=-0.001 tagged_above=-999 required=5 tests=[AWL=0.010, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xS2KUkZhBIxX for ; Sat, 2 Jun 2018 04:12:54 -0700 (PDT) Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197]) by arlo.cworth.org (Postfix) with ESMTPS id B51046DE0A89 for ; Sat, 2 Jun 2018 04:12:54 -0700 (PDT) Received: from remotemail by fethera.tethera.net with local (Exim 4.89) (envelope-from ) id 1fP4Sq-0002F8-7f; Sat, 02 Jun 2018 07:12:52 -0400 Received: (nullmailer pid 29460 invoked by uid 1000); Sat, 02 Jun 2018 11:12:51 -0000 From: David Bremner To: Tomi Ollila , notmuch@notmuchmail.org Subject: Re: [PATCH] configure: check for links to/from libdir in ldconfig output In-Reply-To: References: <20180530030414.18785-1-david@tethera.net> <87wovltjp9.fsf@tethera.net> Date: Sat, 02 Jun 2018 08:12:51 -0300 Message-ID: <876031qw5o.fsf@tethera.net> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 02 Jun 2018 11:12:56 -0000 Tomi Ollila writes: > On Wed, May 30 2018, David Bremner wrote: > >> Tomi Ollila 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