qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] Add HTTP protocol using curl v2


From: Daniel P. Berrange
Subject: Re: [Qemu-devel] [PATCH] Add HTTP protocol using curl v2
Date: Wed, 6 May 2009 11:24:41 +0100
User-agent: Mutt/1.4.1i

On Wed, May 06, 2009 at 12:16:01PM +0200, Kevin Wolf wrote:
> Daniel P. Berrange schrieb:
> > Oh and if you want to be really nice, you could say that any filename
> > with a leading '/' is implicitly file:///, since I can't imagine
> > any protocol name starting with a  '/'.  And this would ensure any
> > existing usage with absolute filenames 'just works' without tripping
> > up on colons.
> 
> Existing usage would mean to get an error message. I don't see any
> reason why we should change this behaviour if we have a real solution.
> Adding more exceptions to the guessing doesn't really make it more harmless.

By existing usage, I meant the practice of supplying an absolute filename
to QEMU, which works all the time, except if you're unlucky to have a colon
in the filename, in which case it breaks, even though it could  trivially
work. Getting an error for an absolute path with a colon is a bug IMHO,
rather than useful behaviour.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|




reply via email to

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