[Libguestfs] [PATCH] osinfo: revamp db reading (RHBZ#1359652)

Pino Toscano ptoscano at redhat.com
Tue Jul 26 11:19:30 UTC 2016


On Tuesday, 26 July 2016 11:16:07 CEST Richard W.M. Jones wrote:
> On Mon, Jul 25, 2016 at 05:47:05PM +0200, Pino Toscano wrote:
> > +  if (path == NULL)
> > +    path = "/usr/share/osinfo";
> 
> I wonder if we should use $prefix here.  I guess there are
> arguments both ways.

The specification defines that directory as fallback.  Also, $prefix
would be libguestfs' own, not osinfo's.

> Should we just run the external osinfo-query program?

Unfortunately, it does not expose all the information we need.

> Anyway it's fine as it stands, so ACK.

Thanks, pushed.
-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20160726/c7dc3b0b/attachment.sig>


More information about the Libguestfs mailing list