* [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path
@ 2020-11-03 11:23 Marc Hartmayer
2020-11-03 11:28 ` Marc Hartmayer
0 siblings, 1 reply; 4+ messages in thread
From: Marc Hartmayer @ 2020-11-03 11:23 UTC (permalink / raw)
To: qemu-devel; +Cc: Paolo Bonzini, Christian Borntraeger
The option `libexecdir` is relative to `prefix` (see
https://mesonbuild.com/Builtin-options.html), so we have to be aware
of this when creating 50-qemu-gpu.json and
50-qemu-virtiofsd.json. Otherwise, tools like libvirt will not be able
to find the executable.
Fixes: 16bf7a3326d8 ("configure: move directory options from config-host.mak to meson")
Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com>
---
contrib/vhost-user-gpu/meson.build | 2 +-
tools/virtiofsd/meson.build | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/contrib/vhost-user-gpu/meson.build b/contrib/vhost-user-gpu/meson.build
index 37ecca13cafb..c487ca72c1ff 100644
--- a/contrib/vhost-user-gpu/meson.build
+++ b/contrib/vhost-user-gpu/meson.build
@@ -9,6 +9,6 @@ if 'CONFIG_TOOLS' in config_host and 'CONFIG_VIRGL' in config_host \
configure_file(input: '50-qemu-gpu.json.in',
output: '50-qemu-gpu.json',
- configuration: { 'libexecdir' : get_option('libexecdir') },
+ configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
install_dir: qemu_datadir / 'vhost-user')
endif
diff --git a/tools/virtiofsd/meson.build b/tools/virtiofsd/meson.build
index e1a4dc98d9ec..17edecf55c0a 100644
--- a/tools/virtiofsd/meson.build
+++ b/tools/virtiofsd/meson.build
@@ -15,5 +15,5 @@ executable('virtiofsd', files(
configure_file(input: '50-qemu-virtiofsd.json.in',
output: '50-qemu-virtiofsd.json',
- configuration: { 'libexecdir' : get_option('libexecdir') },
+ configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
install_dir: qemu_datadir / 'vhost-user')
--
2.25.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path
2020-11-03 11:23 [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path Marc Hartmayer
@ 2020-11-03 11:28 ` Marc Hartmayer
2020-11-03 11:55 ` Paolo Bonzini
0 siblings, 1 reply; 4+ messages in thread
From: Marc Hartmayer @ 2020-11-03 11:28 UTC (permalink / raw)
To: Marc Hartmayer, qemu-devel; +Cc: Paolo Bonzini, Christian Borntraeger
On Tue, Nov 03, 2020 at 12:23 PM +0100, Marc Hartmayer <mhartmay@linux.ibm.com> wrote:
> The option `libexecdir` is relative to `prefix` (see
> https://mesonbuild.com/Builtin-options.html), so we have to be aware
> of this when creating 50-qemu-gpu.json and
> 50-qemu-virtiofsd.json. Otherwise, tools like libvirt will not be able
> to find the executable.
>
> Fixes: 16bf7a3326d8 ("configure: move directory options from config-host.mak to meson")
> Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com>
> ---
> contrib/vhost-user-gpu/meson.build | 2 +-
> tools/virtiofsd/meson.build | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/contrib/vhost-user-gpu/meson.build b/contrib/vhost-user-gpu/meson.build
> index 37ecca13cafb..c487ca72c1ff 100644
> --- a/contrib/vhost-user-gpu/meson.build
> +++ b/contrib/vhost-user-gpu/meson.build
> @@ -9,6 +9,6 @@ if 'CONFIG_TOOLS' in config_host and 'CONFIG_VIRGL' in config_host \
>
> configure_file(input: '50-qemu-gpu.json.in',
> output: '50-qemu-gpu.json',
> - configuration: { 'libexecdir' : get_option('libexecdir') },
> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
> install_dir: qemu_datadir / 'vhost-user')
> endif
> diff --git a/tools/virtiofsd/meson.build b/tools/virtiofsd/meson.build
> index e1a4dc98d9ec..17edecf55c0a 100644
> --- a/tools/virtiofsd/meson.build
> +++ b/tools/virtiofsd/meson.build
> @@ -15,5 +15,5 @@ executable('virtiofsd', files(
>
> configure_file(input: '50-qemu-virtiofsd.json.in',
> output: '50-qemu-virtiofsd.json',
> - configuration: { 'libexecdir' : get_option('libexecdir') },
> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
> install_dir: qemu_datadir / 'vhost-user')
> --
> 2.25.4
>
>
It’s probably not the best way to fix it, but at least a good hint
what’s wrong :)
--
Kind regards / Beste Grüße
Marc Hartmayer
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Gregor Pillen
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path
2020-11-03 11:28 ` Marc Hartmayer
@ 2020-11-03 11:55 ` Paolo Bonzini
2020-11-03 12:02 ` Marc Hartmayer
0 siblings, 1 reply; 4+ messages in thread
From: Paolo Bonzini @ 2020-11-03 11:55 UTC (permalink / raw)
To: Marc Hartmayer, qemu-devel; +Cc: Christian Borntraeger
On 03/11/20 12:28, Marc Hartmayer wrote:
> On Tue, Nov 03, 2020 at 12:23 PM +0100, Marc Hartmayer <mhartmay@linux.ibm.com> wrote:
>> The option `libexecdir` is relative to `prefix` (see
>> https://mesonbuild.com/Builtin-options.html), so we have to be aware
>> of this when creating 50-qemu-gpu.json and
>> 50-qemu-virtiofsd.json. Otherwise, tools like libvirt will not be able
>> to find the executable.
>>
>> Fixes: 16bf7a3326d8 ("configure: move directory options from config-host.mak to meson")
>> Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com>
>> ---
>> contrib/vhost-user-gpu/meson.build | 2 +-
>> tools/virtiofsd/meson.build | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/contrib/vhost-user-gpu/meson.build b/contrib/vhost-user-gpu/meson.build
>> index 37ecca13cafb..c487ca72c1ff 100644
>> --- a/contrib/vhost-user-gpu/meson.build
>> +++ b/contrib/vhost-user-gpu/meson.build
>> @@ -9,6 +9,6 @@ if 'CONFIG_TOOLS' in config_host and 'CONFIG_VIRGL' in config_host \
>>
>> configure_file(input: '50-qemu-gpu.json.in',
>> output: '50-qemu-gpu.json',
>> - configuration: { 'libexecdir' : get_option('libexecdir') },
>> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
>> install_dir: qemu_datadir / 'vhost-user')
>> endif
>> diff --git a/tools/virtiofsd/meson.build b/tools/virtiofsd/meson.build
>> index e1a4dc98d9ec..17edecf55c0a 100644
>> --- a/tools/virtiofsd/meson.build
>> +++ b/tools/virtiofsd/meson.build
>> @@ -15,5 +15,5 @@ executable('virtiofsd', files(
>>
>> configure_file(input: '50-qemu-virtiofsd.json.in',
>> output: '50-qemu-virtiofsd.json',
>> - configuration: { 'libexecdir' : get_option('libexecdir') },
>> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
>> install_dir: qemu_datadir / 'vhost-user')
>> --
>> 2.25.4
>>
>>
>
> It’s probably not the best way to fix it, but at least a good hint
> what’s wrong :)
No, it's okay. I queued it.
Paolo
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path
2020-11-03 11:55 ` Paolo Bonzini
@ 2020-11-03 12:02 ` Marc Hartmayer
0 siblings, 0 replies; 4+ messages in thread
From: Marc Hartmayer @ 2020-11-03 12:02 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel
On Tue, Nov 03, 2020 at 12:55 PM +0100, Paolo Bonzini <pbonzini@redhat.com> wrote:
> On 03/11/20 12:28, Marc Hartmayer wrote:
>> On Tue, Nov 03, 2020 at 12:23 PM +0100, Marc Hartmayer <mhartmay@linux.ibm.com> wrote:
>>> The option `libexecdir` is relative to `prefix` (see
>>> https://mesonbuild.com/Builtin-options.html), so we have to be aware
>>> of this when creating 50-qemu-gpu.json and
>>> 50-qemu-virtiofsd.json. Otherwise, tools like libvirt will not be able
>>> to find the executable.
>>>
>>> Fixes: 16bf7a3326d8 ("configure: move directory options from config-host.mak to meson")
>>> Signed-off-by: Marc Hartmayer <mhartmay@linux.ibm.com>
>>> ---
>>> contrib/vhost-user-gpu/meson.build | 2 +-
>>> tools/virtiofsd/meson.build | 2 +-
>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/contrib/vhost-user-gpu/meson.build b/contrib/vhost-user-gpu/meson.build
>>> index 37ecca13cafb..c487ca72c1ff 100644
>>> --- a/contrib/vhost-user-gpu/meson.build
>>> +++ b/contrib/vhost-user-gpu/meson.build
>>> @@ -9,6 +9,6 @@ if 'CONFIG_TOOLS' in config_host and 'CONFIG_VIRGL' in config_host \
>>>
>>> configure_file(input: '50-qemu-gpu.json.in',
>>> output: '50-qemu-gpu.json',
>>> - configuration: { 'libexecdir' : get_option('libexecdir') },
>>> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
>>> install_dir: qemu_datadir / 'vhost-user')
>>> endif
>>> diff --git a/tools/virtiofsd/meson.build b/tools/virtiofsd/meson.build
>>> index e1a4dc98d9ec..17edecf55c0a 100644
>>> --- a/tools/virtiofsd/meson.build
>>> +++ b/tools/virtiofsd/meson.build
>>> @@ -15,5 +15,5 @@ executable('virtiofsd', files(
>>>
>>> configure_file(input: '50-qemu-virtiofsd.json.in',
>>> output: '50-qemu-virtiofsd.json',
>>> - configuration: { 'libexecdir' : get_option('libexecdir') },
>>> + configuration: { 'libexecdir' : get_option('prefix') / get_option('libexecdir') },
>>> install_dir: qemu_datadir / 'vhost-user')
>>> --
>>> 2.25.4
>>>
>>>
>>
>> It’s probably not the best way to fix it, but at least a good hint
>> what’s wrong :)
>
> No, it's okay. I queued it.
Thanks :)
>
> Paolo
>
--
Kind regards / Beste Grüße
Marc Hartmayer
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Gregor Pillen
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-11-03 12:04 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-03 11:23 [PATCH] meson: vhost-user-gpu/virtiofsd: use absolute path Marc Hartmayer
2020-11-03 11:28 ` Marc Hartmayer
2020-11-03 11:55 ` Paolo Bonzini
2020-11-03 12:02 ` Marc Hartmayer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).