>-----Original Message-----
>From: Peter Krempa <pkre...@redhat.com>
>Subject: Re: [PATCH] meson: Add back prefix path for runstatedir
>
>On Wed, Apr 02, 2025 at 15:58:28 +0800, Zhenzhong Duan wrote:
>> Currently libvirt favors /run instead of /var/run, but for local build
>> run test, a prefix path is still needed to avoid interoperating with OS
>> vendor provided binaries.
>
>Could you please elaborate?

The OS vendor provided binaries generates sockets in /run/libvirt/,
The local run './build/src/libvirtd -d' also generates sockets in /run/libvirt/.

If prefix path is used, local run generates sockets in 
/root/libvirt_build/run/libvirt/,
then there is no conflict.

>
>>
>> Signed-off-by: Zhenzhong Duan <zhenzhong.d...@intel.com>
>> ---
>>  meson.build | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meson.build b/meson.build
>> index bf4a245dd3..84c9796c2f 100644
>> --- a/meson.build
>> +++ b/meson.build
>> @@ -82,7 +82,7 @@ endif
>>
>>  runstatedir = get_option('runstatedir')
>>  if runstatedir == ''
>> -  runstatedir = '/run'
>> +  runstatedir = prefix / 'run'
>
>If you need anything non-default you can always set it via
>-Drunstatedir= when configuring the project.

Yes, I have to specify it explicitly:

meson build --prefix /root/libvirt_build 
-Drunstatedir=/root/libvirt_build/var/run

In early days, I remember no need to use -Drunstatedir,
'meson build --prefix /root/libvirt_build' just work for me.

Thanks
Zhenzhong

Reply via email to