esx: Switch to creating URLs using virURIFormat

[PATCH v3 0/3] esx: Switch to creating URLs using virURIFormat

Richard W.M. Jones via Devel posted 3 patches 6 days, 10 hours ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20260126181115.1832646-1-rjones@redhat.com

Expand all Fold all

[PATCH v3 0/3] esx: Switch to creating URLs using virURIFormat

Posted by Richard W.M. Jones via Devel 6 days, 10 hours ago

v2 was here:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/XMLWGNNW3DAIWY25PRT4DTDG52HO5WFX/

As suggested by Dan in the previous review, it'd be better to use
virURI, and indeed the code is a bit nicer this way.

Rich.

Re: [PATCH v3 0/3] esx: Switch to creating URLs using virURIFormat

Posted by Daniel P. Berrangé via Devel 6 days, 8 hours ago

On Mon, Jan 26, 2026 at 06:09:56PM +0000, Richard W.M. Jones wrote:
> v2 was here:
> https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/XMLWGNNW3DAIWY25PRT4DTDG52HO5WFX/
> 
> As suggested by Dan in the previous review, it'd be better to use
> virURI, and indeed the code is a bit nicer this way.

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|