[Libguestfs] [PATCH] Change fallback name for external supermin helper

Olaf Hering olaf at aepfle.de
Fri May 31 12:00:47 UTC 2013


On Fri, May 31, Richard W.M. Jones wrote:

> On Fri, May 31, 2013 at 01:38:06PM +0200, Olaf Hering wrote:
> > But this requires that febootstrap or supermin is in BuildRequires:,
> > while this change removes the hard dependency of BuildRequires:. But I
> > see, if the appliance is enabled then it would now fail with this
> > change. In my packaging the appliance is not enabled because zypp
> > support in supermin is not yet complete, so I did not spot this part. 
> 
> Actually I thought zypp support was complete?  Oh well, how far off is
> complete zypp support in supermin?

supermin can not be used in a chroot when called as builduser, because
there is no repo configured inside that chroot.

supermin need to handle the case '--root X --reposd-dir Y', which is
triggered by the '--packager-config <conf>' option. Currently the <conf>
file is expected in X//<conf>, which will fail. 
The new code has to parse <conf>, look for reposdir=<dir> and replace
the current hardcode path Y with the one from <conf> so that libzypp
continues to oerate below X (--root X --repos-dir <dir>).

I think there is some odd ocaml-iniparser package, which I could use.
Or I just grep for reposdir= and use that.

> On the slightly related subject, I need to add dnf/libsolv support to
> supermin.  I don't know how much code could be shared.  Unfortunately
> I suspect not very much since although both zypper and dnf are using
> the same underlying library, there's not much else in common.

I dont know much about the underlying libsolv, its very likely that in
the end libzypp does all the required work.

Olaf




More information about the Libguestfs mailing list