From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from vmicros1.altlinux.org (vmicros1.altlinux.org [194.107.17.57]) by sourceware.org (Postfix) with ESMTP id 4AECB3858D28 for ; Wed, 10 Nov 2021 22:43:00 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 4AECB3858D28 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=altlinux.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=altlinux.org Received: from imap.altlinux.org (imap.altlinux.org [194.107.17.38]) by vmicros1.altlinux.org (Postfix) with ESMTP id A39D772C8B8; Thu, 11 Nov 2021 01:42:59 +0300 (MSK) Received: from altlinux.org (sole.flsd.net [185.75.180.6]) by imap.altlinux.org (Postfix) with ESMTPSA id 8F3354A46EC; Thu, 11 Nov 2021 01:42:59 +0300 (MSK) Date: Thu, 11 Nov 2021 01:42:59 +0300 From: Vitaly Chikunov To: "Frank Ch. Eigler" , elfutils-devel@sourceware.org Subject: Re: [patch] PR27783: default debuginfod-urls profile rework Message-ID: <20211110224259.ltidgisedhyb4k25@altlinux.org> References: <20211003213333.GC21634@redhat.com> <20211110214247.GA3988@altlinux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <20211110214247.GA3988@altlinux.org> X-Spam-Status: No, score=-3.9 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: elfutils-devel@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Elfutils-devel mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Nov 2021 22:43:02 -0000 Frank, On Thu, Nov 11, 2021 at 12:42:47AM +0300, Dmitry V. Levin wrote: > On Sun, Oct 03, 2021 at 05:33:33PM -0400, Frank Ch. Eigler via Elfutils-devel wrote: > > commit 0c634f243d266ce8841fd311433d5d79555fabf9 > > Author: Frank Ch. Eigler > > Date: Sun Oct 3 17:04:24 2021 -0400 > > > > PR27783: switch default debuginfod-urls to drop-in style files > > > > Rewrote and commented the /etc/profile.d csh and sh script fragments > > to take the default $DEBUGINFOD_URLS from the union of drop-in files: > > /etc/debuginfod/*.urls. Hand-tested with csh and bash, with > > conditions including no prior $DEBUGINFOD_URLS, nonexistent .urls > > files, multiple entries in .urls files. > [...] > > +# $HOME/.profile* or similar files may first set $DEBUGINFOD_URLS. > > +# If $DEBUGINFOD_URLS is not set there, we set it from system *.url files. > > +# $HOME/.*rc or similar files may then amend $DEBUGINFOD_URLS. > > +# See also [man debuginfod-client-config] for other environment variables > > +# such as $DEBUGINFOD_MAXSIZE, $DEBUGINFOD_MAXTIME, $DEBUGINFOD_PROGRESS. > > + > > +if [ -z "$DEBUGINFOD_URLS" ]; then > > + prefix="@prefix@" > > + debuginfod_urls=`find "@sysconfdir@/debuginfod/" -name '*.urls' | xargs cat | tr '\n' ' '` If *.urls file does not contain "\n" at the EOF its url will be concatenated to another without separator making multiple urls invalid. I'd suggest you don't use find (supporting subdirectories, why?) with xargs cat ..., but just `for x in *.urls` cycle. Also, running find, xargs, cat, tr, always (even if there is no urls, you can optimize calling cat with xargs -r) Calling so much binaries on login will slow down login to a loaded box, so 'for' cycle perhaps would be faster. If you still going to use so complicated processing would be beneficial to support commentaries in .urls files too, so people can add some description, or comment out some url (there can be many implied by 'urls' name). Thanks,