Re: [libvirt-perl PATCH] Add support for VIR_SECRET_USAGE_TYPE_PASSPHRASE

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

 



On Mon, Jul 04, 2016 at 04:49:56PM +0200, Martin Kletzander wrote:
> On Mon, Jul 04, 2016 at 03:41:16PM +0100, Daniel P. Berrange wrote:
> > On Mon, Jul 04, 2016 at 04:37:27PM +0200, Martin Kletzander wrote:
> > > Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> > > ---
> > > This fixes a build for me with upstream libvirt.
> > 
> > NACK, per my reply to John's original patch, I think this should
> > not have been added to libvirt API in the first place.
> > 
> 
> I missed that.  Looking at it now I was wondering why we have special
> purpose, dedicated secrets to that they are used safely and now we have
> general-purpose secret.  OK, good to know, I haven't thought of that, I
> just wanted to fix the test.  Thanks for the info.

Oh, I guess John is not around today, due to 4th July, so we'll have to
wait for him to return to resolve this.

ACK to pushing your patch now to fix the build, and we'll just revert
it later if we revert the main libvirt.so patch.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

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