Re: [PATCH 1/3] Use virFileFindResource to locate libvirtd daemon

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/25/2014 08:25 AM, Daniel P. Berrange wrote:
> On Fri, Apr 25, 2014 at 08:01:58AM -0600, Eric Blake wrote:
>> On 04/25/2014 05:28 AM, Daniel P. Berrange wrote:
>>> Make the remote driver use virFileFindResource to find the
>>> libvirt daemon path, so that it executes the in-builddir
>>> daemon if run from source tree.
>>>

>>
>> the new code assumes LIBVIRTD_PATH provides "/path/to" which contains
>> libvirtd.
>>
>> Don't know how many users this will impact, but the only former
>> documentation of LIBVIRTD_PATH was in the error message.  I can live
>> with the change in semantics, since it is not formally documented on the
>> web page as something users would normally fiddle with.
> 
> The 'run' script uses this, but we can just delete those lines from
> the 'run' script now, since we'll "do the right thing" automatically

In fact, you HAVE to fix the run script, or using it will break :) (I
just tested it now, on a virgin VM with no libvirt installed and just an
in-tree build: 'tools/virsh list' works now where it previously didn't,
but without a tweak to run.in, './run tools/virsh list' started failing.

So amendment to my earlier posting: my ACK is conditional on this being
squashed in (and similarly for 2/3):

diff --git i/run.in w/run.in
index 2211f24..c8f12a5 100644
--- i/run.in
+++ w/run.in
@@ -1,6 +1,6 @@
 #!/bin/sh
 # libvirt 'run' programs locally script
-# Copyright (C) 2012-2013 Red Hat, Inc.
+# Copyright (C) 2012-2014 Red Hat, Inc.
 #
 # This library is free software; you can redistribute it and/or
 # modify it under the terms of the GNU Lesser General Public
@@ -56,7 +56,6 @@ export LD_LIBRARY_PATH
 export LIBVIRT_DRIVER_DIR="$b/src/.libs"
 export LIBVIRT_LOCK_MANAGER_PLUGIN_DIR="$b/src/.libs"
 export VIRTLOCKD_PATH="$b/src/virtlockd"
-export LIBVIRTD_PATH="$b/daemon/libvirtd"

 # This is a cheap way to find some use-after-free and uninitialized
 # read problems when using glibc.


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]