[PATCH] config: Remove unnecessary setting/unsetting of prefix in profiles

Dmitry V. Levin ldv@altlinux.org
Wed Jun 29 20:08:58 GMT 2022


Hi,

On Wed, Jun 29, 2022 at 08:33:19PM +0200, Mark Wielaard wrote:
> Hi Frank,
> 
> On Wed, Jun 29, 2022 at 01:29:04PM -0400, Frank Ch. Eigler wrote:
> > > In both profile.csh.in and profile.sh.in we set and then unset
> > > prefix, but never use it.
> > 
> > It is used, because the autoconf @sysconfdir@ macro expands to a
> > string like "$prefix/...", which requires the shell variable $prefix
> > to be set.
> 
> O, I see. Indeed the default is '${prefix}/etc'. But I never saw that
> because in a package install configure is called with everything
> expanded [*]. Should have tried by hand.

This is not the first time when a patch removing these definitions of
"prefix" variable is proposed.  In fact, I applied a similar patch in my
tree last year because I always use --sysconfdir=/etc and "prefix" is a
very generic name so I prefer not to tamper with it in profile scripts.


-- 
ldv


More information about the Elfutils-devel mailing list