On Thursday 07 July 2016 17:30:01 Richard W.M. Jones wrote:
Change the Curl module to use an ADT to store the name of the curl
binary and the arguments.
The callers in virt-v2v are changed accordingly.
This also adds a (currently unused) ?proxy argument to allow callers
to override the proxy. It also adds some safety arguments implicitly.
---
Definitely a nice improvement, thanks!
Just a couple of notes below.
+let run { curl = curl; args = args } =
+ let config_file, chan = Filename.open_temp_file "curl" ".conf" in
I'd use "guestfs-curl" as prefix, as the location for this temporary
file is the general $TMPDIR and would be mislead as generated by curl
proper.
diff --git a/v2v/copy_to_local.ml b/v2v/copy_to_local.ml
index 717ba50..2e3b59b 100644
--- a/v2v/copy_to_local.ml
+++ b/v2v/copy_to_local.ml
@@ -199,9 +199,9 @@ read the man page virt-v2v-copy-to-local(1).
| ESXi _ ->
let curl_args = [
- "url", Some remote_disk;
- "output", Some local_disk;
- ] in
+ "url", Some remote_disk;
+ "output", Some local_disk;
+ ] in
Small unneeded indentation change.
diff --git a/v2v/vCenter.ml b/v2v/vCenter.ml
index d41f223..ed4a9b2 100644
--- a/v2v/vCenter.ml
+++ b/v2v/vCenter.ml
@@ -46,10 +46,10 @@ let get_session_cookie password scheme uri sslverify url =
Some !session_cookie
else (
let curl_args = [
- "head", None;
- "silent", None;
- "url", Some url;
- ] in
+ "head", None;
+ "silent", None;
+ "url", Some url;
+ ] in
Ditto.
Thanks,
--
Pino Toscano