On Mon, Jul 04, 2016 at 04:22:32PM +0100, Daniel P. Berrange wrote:
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.
OK, pushed now, feel free to revert it when appropriate. Thanks.
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
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list