[libvirt] [libvirt-sandbox] virt-sandbox-service: fixed /lib/ into /usr/lib for searching unit files

[Christophe Fergeau]

On Wed, Jun 04, 2014 at 01:24:17PM +0200, Cedric Bosdonnat wrote:
> On Wed, 2014-06-04 at 11:24 +0200, Christophe Fergeau wrote:
> > On Tue, May 13, 2014 at 10:10:36AM +0200, Cédric Bosdonnat wrote:
> > > ---
> > >  bin/virt-sandbox-service | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/bin/virt-sandbox-service b/bin/virt-sandbox-service
> > > index 2dcbfb8..9ed37e0 100755
> > > --- a/bin/virt-sandbox-service
> > > +++ b/bin/virt-sandbox-service
> > > @@ -1071,7 +1071,7 @@ class CheckUnit(argparse.Action):
> > >              src = "/etc/systemd/system/" + unit
> > >              if os.path.exists(src):
> > >                  return src
> > > -            src = "/lib/systemd/system/" + unit
> > > +            src = "/usr/lib/systemd/system/" + unit
> > 
> > This makes sense to me as systemd installs unit files in
> > $rootdir/lib/systemd, with $rootdir set to /usr at least upstream and in
> > fedora.
> > Are you having issues withusing /lib instead of /usr/lib? Or is
> > it just cleaner this way?
> 
> Yes, we don't have a /lib symlink pointing to /usr/lib: so failing here
> on openSUSE.

Ah, this would be useful info to have in the commit log imo. ACK from
me, though I'm not familiar at all with all of this ;)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140604/35276ad8/attachment-0001.sig>