Re: [PATCH] Fix file name transformations (mostly)

[Lubomir Kundrak <lkundrak@redhat.com>]

On Tue, 2008-04-15 at 15:19 -0400, Pavel Roskin wrote:
> On Tue, 2008-04-15 at 00:12 +0200, Lubomir Kundrak wrote:
> > Hi,
> > 
> > Attaching the patch that fixes ./configure's --program-transform-name
> > option. Tested with --program-transform-name=s,grub,grub2, quite
> > extensively. Only one more modification is needed to completely support
> > this -- define DEFAULT_DIRECTORY with -D option of the preprocessor
> > instead of hardcoding in include/grub/util/misc.h. Should be fairly
> > trivial though.
> 
> I would prefer if we don't rush with "program-transform-name" at the
> expense of readability, but rather make other simplifications what would
> have a similar effect.  I think ${libdir}/grub should be given a name
> (e.g. grublibdir) in configure.ac and substituted in the scripts.

That makes sense. Another solution would be to have shell library to
include by all scripts and define variables with substituted names
there.

> As for /boot/grub, we are not installing anything there on "make
> install".  I don't think it should be transformed.  But we could accept
> a configuration option to change that directory.

We should have a sensible default that wouldn't clash with GRUB Legacy's
default.

Thanks,
-- 
Lubomir Kundrak (Red Hat Security Response Team)