From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 73020 invoked by alias); 18 Nov 2019 16:27:30 -0000 Mailing-List: contact elfutils-devel-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Post: List-Help: List-Subscribe: Sender: elfutils-devel-owner@sourceware.org Received: (qmail 73006 invoked by uid 89); 18 Nov 2019 16:27:30 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Checked: by ClamAV 0.100.3 on sourceware.org X-Virus-Found: No X-Spam-SWARE-Status: No, score=-6.5 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.1 spammy=HX-Languages-Length:1799 X-Spam-Status: No, score=-6.5 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on sourceware.org X-Spam-Level: X-HELO: gnu.wildebeest.org Received: from wildebeest.demon.nl (HELO gnu.wildebeest.org) (212.238.236.112) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 18 Nov 2019 16:27:29 +0000 Received: from tarox.wildebeest.org (tarox.wildebeest.org [172.31.17.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id D1CBB300099F; Mon, 18 Nov 2019 17:27:26 +0100 (CET) Received: by tarox.wildebeest.org (Postfix, from userid 1000) id B9B02413CEAA; Mon, 18 Nov 2019 17:27:26 +0100 (CET) Message-ID: <24757660e072ccdc1be8dc27ec4bce6316e3cffe.camel@klomp.org> Subject: Re: patch 4 debuginfod: symlink following mode From: Mark Wielaard To: "Frank Ch. Eigler" Cc: elfutils-devel@sourceware.org, amerey@redhat.com Date: Mon, 18 Nov 2019 16:27:00 -0000 In-Reply-To: <20191115183132.GB15272@redhat.com> References: <20191028190438.GC14349@redhat.com> <20191028190602.GD14349@redhat.com> <20191028190726.GE14349@redhat.com> <20191104214823.GA17633@redhat.com> <20191107090732.GA19337@redhat.com> <6c66a738f766fc7ab4fc312de3a406285589dbd0.camel@klomp.org> <20191115183132.GB15272@redhat.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.28.5 (3.28.5-5.el7) Mime-Version: 1.0 X-Spam-Flag: NO X-IsSubscribed: yes X-SW-Source: 2019-q4/txt/msg00169.txt.bz2 Hi, On Fri, 2019-11-15 at 13:31 -0500, Frank Ch. Eigler wrote: > > > In order to support file/rpm archives that are organized via symlink > > > trees, add an "-L" option to debuginfod, meaning about the same as for > > > find(1) or ls(1): to traverse rather than ignore symlinks. > >=20 > > Could you give an example of when exactly this is necessary? >=20 > Because some file/rpm archievs are organized via symlink trees. :-) > For example, a build system may have >=20 > /packages/N/V/R/ARCH/*.rpm files, which are a legion > plus /compose/F30/SPIN/ARCH/ directories, which contain=20 > symlinks into the former > or package archives stored over multiple NFS mounts, whose > contents are interleaved via symlinks. >=20 > > I assume that it isn't really necessary for rpm archives, you probably > > don't want to follow any symlinks from an archive that point to > > something outside the archive, and you will see all files in the > > archive anyway. So following symlinks doesn't seem helpful there. >=20 > These are not symlinks WITHIN rpms. These are symlinks encountered > during filesystem traversal looking for RPMs. >=20 > > Also why combine symlink following with cross-device searches? > > Shouldn't that be separate options? >=20 > They could be. They seemed to go well together in the cases I've seen > and thought about. Would you like another option now? No, I understand the use case now. And you are right it makes sense to combine them for such cases. But I still think it would be better to split the directory full of rpms case from the directory full of individual file cases. The option makes perfect sense for the case you show, but I think it is not really appropriate to mix it with the individual exec/debug and source files case. Cheers, Mark