Re: [PATCH v3 4/5] qemu: Remove need for qemuBuildSecretInfoProps

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

 



On Fri, Jun 03, 2016 at 06:52:52 -0400, John Ferlan wrote:
> Just move the code into qemuBuildObjectSecretCommandLine.
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
>  src/qemu/qemu_command.c | 57 +++++++++++--------------------------------------
>  1 file changed, 12 insertions(+), 45 deletions(-)

Again. If this is going to be reused in the monitor then it doesn't make
sense to move it to the builder directly. If it's not going to be used
with the monitor then it doesn't make sense to use the json parser and
the object formatter to achieve the same at all since it's rather
untrivial code that can be achieved with a single printf.

Peter

--
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]