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 <f...@redhat.com> > > 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,
