[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Curl updates
From: |
Matthew Booth |
Subject: |
Re: [Qemu-devel] Curl updates |
Date: |
Wed, 14 May 2014 12:08:59 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 14/05/14 03:48, Kevin Wolf wrote:
> Am 13.05.2014 um 21:47 hat Eric Blake geschrieben:
>> On 05/08/2014 02:42 AM, Matthew Booth wrote:
>>> [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.
>>
>> Offhand, I'm not liking this. Why not use a completely valid
>> URI, with '.../path?query&opt1=val&opt2=val'? Inventing your
>> own [opt1=val:opt2=val] on top of URI is asking for confusion.
>>
>> Are you trying to support a way to pass a query string to the
>> curl URI, in addition to local options? How often do curl URIs
>> need a query?
>
> My guess would be that you need this more often than local
> options.
>
> Anyway, let's not add new options encoded in the URL, but point
> users to separate options. We may decide that we need the support
> the old crude way of encoding local options for compatibility, but
> preferably I would make filename just a plain URL.
Agree, but only when we support giving options to a backing file.
Matt
- --
Matthew Booth
Red Hat Engineering, Virtualisation Team
Phone: +442070094448 (UK)
GPG ID: D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iEYEARECAAYFAlNzlRsACgkQNEHqGdM8NJD2gQCfcjTTNukAHagZmZ8YIl7NrKxa
n+YAoIju2DbERjV55zc5xt1TtLwOaq6u
=0n8Q
-----END PGP SIGNATURE-----
- [Qemu-devel] [PATCH 1/4] curl: Fix parsing of readahead option from filename, (continued)
- [Qemu-devel] [PATCH 3/4] curl: Add usage documentation, Matthew Booth, 2014/05/08
- [Qemu-devel] [PATCH 2/4] curl: Add sslverify option, Matthew Booth, 2014/05/08
- [Qemu-devel] [PATCH 4/4] curl: Fix build when curl_multi_socket_action isn't available, Matthew Booth, 2014/05/08
- Re: [Qemu-devel] Curl updates, Eric Blake, 2014/05/13
- Re: [Qemu-devel] Curl updates, Eric Blake, 2014/05/14
Re: [Qemu-devel] Curl updates, Matthew Booth, 2014/05/14