qemu-devel
[Top][All Lists]
Advanced

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

[Qemu-devel] Curl updates


From: Matthew Booth
Subject: [Qemu-devel] Curl updates
Date: Thu, 8 May 2014 09:42:16 +0100

[PATCH 1/4] curl: Fix parsing of readahead option from filename
[PATCH 2/4] curl: Add sslverify option
[PATCH 3/4] curl: Add usage documentation

The first 3 patches are reposted with updates following discussion of the option
syntax. With this patch I've decided to break entirely with the previous syntax.
Given that option parsing was previously both broken and undocumented, this is
hopefully a forgivable sin.

The new syntax is:

  http://user:address@hidden/path?query[opt1=val:opt2=val]

I've bounded the option block in square brackets as these have no semantic
meaning in any of the supported URI formats. Consequently the user can escape
them if they're unfortunate enough to have a URI which looks like it contains an
option block.

I decided to separate options with colons rather than commas because commas play
havoc with qemu's command line parsing. There's presumably a way round this, but
I couldn't guess it and I was too lazy to look it up, so I assume users would
feel the same.

As options are now unambigous, invalid options now result in an error.

[PATCH 4/4] curl: Fix build when curl_multi_socket_action isn't

The last patch is unrelated. It should fix build against old curl, although I
don't have an old curl kicking around to test it against.

I still have a couple of patches in my local tree which:

* Remove blocking behaviour in curl_open
* Don't send EIO when a read connection hits a timeout

I'm also planning to add another option for timeout length, and to implement
write support.

Matt



reply via email to

[Prev in Thread] Current Thread [Next in Thread]