On 07/20/22 17:06, Eric Blake wrote:
On Wed, Jul 20, 2022 at 01:53:22PM +0200, Laszlo Ersek wrote:
> ./configure expands @datadir@ and @libdir@ to "${prefix}/share" and
> "${exec_prefix}/lib" by default (verbatim); $prefix and $exec_prefix are
> supposed to be substituted by the shell when the generated shell scripts
> are executed.
>
> For this, capture the prefix and exec_prefix values at the time
> ./configure runs.
>
> This fixes failures such as
>
>> $ virt-p2v-make-kickstart fedora
>> base64: /share/virt-p2v/issue: No such file or directory
>
> for such scripts that were installed with "make install".
>
> +++ b/virt-p2v-make-disk.in
> @@ -20,6 +20,8 @@ unset CDPATH
>
> program="virt-p2v-make-disk"
> version="@PACKAGE_VERSION@"
> +prefix="@prefix@"
> +exec_prefix="@exec_prefix@"
I'm late to the conversation, but I also recommend adding a third line:
datarootdir="@datarootdir@"
as it will then silence the autoconf warnings about not honoring
--datarootdir in a file that dereferenced @datadir@.
With that addition in all relevant .in files,
Acked-by: Eric Blake <eblake(a)redhat.com>
Commit bde5d11566f7.
Thanks!
Laszlo