From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:46542) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TG6Zc-0007cD-2k for qemu-devel@nongnu.org; Mon, 24 Sep 2012 07:11:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TG6Za-0008A6-KU for qemu-devel@nongnu.org; Mon, 24 Sep 2012 07:11:35 -0400 Received: from mx4-phx2.redhat.com ([209.132.183.25]:37409) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TG6Za-00089o-Cx for qemu-devel@nongnu.org; Mon, 24 Sep 2012 07:11:34 -0400 Date: Mon, 24 Sep 2012 07:11:30 -0400 (EDT) From: Paolo Bonzini Message-ID: <1475119461.458336.1348485090253.JavaMail.root@redhat.com> In-Reply-To: <20120924102738.GT24071@rhmail.home.annexia.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v9 2/4] qemu: URI parsing library List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Richard W.M. Jones" Cc: Kevin Wolf , Anthony Liguori , Anand Avati , Vijay Bellur , Stefan Hajnoczi , Harsh Bora , Amar Tumballi , qemu-devel@nongnu.org, Daniel Veillard , Blue Swirl , Avi Kivity , Bharata B Rao ----- Messaggio originale ----- > Da: "Richard W.M. Jones" > A: "Bharata B Rao" > Cc: qemu-devel@nongnu.org, "Paolo Bonzini" , "Kevin = Wolf" , "Avi Kivity" > , "Anthony Liguori" , "Stefan Hajnoc= zi" , "Blue Swirl" > , "Anand Avati" , "Vijay Bellur"= , "Amar Tumballi" > , "Harsh Bora" , "Daniel Vei= llard" > Inviato: Luned=C3=AC, 24 settembre 2012 12:27:38 > Oggetto: Re: [PATCH v9 2/4] qemu: URI parsing library >=20 >=20 > On Mon, Sep 24, 2012 at 02:42:02PM +0530, Bharata B Rao wrote: > > qemu: URI parsing library > >=20 > > From: Paolo Bonzini > >=20 > > Add a new URI parsing library to QEMU. The code has been borrowed > > from libxml2 and libvirt. >=20 > Rather than duplicating the libxml2 code, I think it would be better > to depend on libxml2. While it's not ideal to add more external > dependencies, libxml2 is very commonly available. Also libxml2 gets > frequent security updates and you never know when the URI code here > might be found to contain some sort of bug / security issue. You're preaching to the choir here, but I think there are limited advantages in this case. The URI parsing has seen hardly any change in the last 2 years and is surprisingly self-contained even in libxml2: the dependencies were limited to memory allocation hooks basically. A better plan would be to incorporate this code into glib, completing the extremely sparse URI support that is already there. However, we would not be able to use it anyway, because we support compiling on old glib versions. Paolo > This will need a change to the qemu configure script of course. >=20 > Rich. >=20 > -- > Richard Jones, Virtualization Group, Red Hat > http://people.redhat.com/~rjones > virt-df lists disk usage of guests without needing to install any > software inside the virtual machine. Supports Linux and Windows. > http://et.redhat.com/~rjones/virt-df/ >=20