[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Curl updates
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] Curl updates |
Date: |
Wed, 14 May 2014 18:43:51 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 14.05.2014 um 18:08 hat Matthew Booth geschrieben:
> 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.
Right, but we want that anyway. I applied Max's patches for the json:
pseudo-protocol today, so we should have everything we need.
Kevin
pgpdBZrJ_5ccb.pgp
Description: PGP signature
- Re: [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