Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-16 Thread Mauricio Faria de Oliveira
On 11/16/2016 05:42 PM, Guido Günther wrote: No, you need to use getent or similar since the user/group might not be in /etc/{passwd,group} (e.g. ldap). Oh, right. Thanks for this; getent turned out to be a very good option; no group changes required. -- Mauricio Faria de Oliveira IBM Linux

Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-16 Thread Guido Günther
On Wed, Nov 16, 2016 at 04:31:34PM -0200, Mauricio Faria de Oliveira wrote: > Hi Guido, > > On 11/15/2016 08:00 AM, Guido Günther wrote: > > > What do you think of a debconf warning message/prompt, which > > > > asks the user to confirm that it's OK not to use the uid/gid values, > > > > and

Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-16 Thread Mauricio Faria de Oliveira
Hi Guido, On 11/15/2016 08:00 AM, Guido Günther wrote: What do you think of a debconf warning message/prompt, which > asks the user to confirm that it's OK not to use the uid/gid values, > and explains about the potential problem w/ guest migration over NFS? Although it adds some complexity

Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-15 Thread Guido Günther
Hi Mauricio, On Mon, Nov 14, 2016 at 05:10:01PM -0200, Mauricio Faria de Oliveira wrote: > Hi Guido, > > Thanks for the review. > > On 11/14/2016 03:54 PM, Guido Günther wrote: > > > +# Allocated UID and GID for libvirt-qemu > > > > +libvirt_qemu_uid=64055 > > > > +libvirt_qemu_gid=64055 > >

Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-14 Thread Mauricio Faria de Oliveira
Hi Guido, Thanks for the review. On 11/14/2016 03:54 PM, Guido Günther wrote: +# Allocated UID and GID for libvirt-qemu > +libvirt_qemu_uid=64055 > +libvirt_qemu_gid=64055 Please use all caps for the variable names. Okay; including the "parameter_(u|g)id" variables below. (like another

Bug#844339: [Pkg-libvirt-maintainers] Bug#844339: patch for sid

2016-11-14 Thread Guido Günther
controL: tags -1 +moreinfo H Mauricio, On Mon, Nov 14, 2016 at 01:53:19PM -0200, Mauricio Faria de Oliveira wrote: > Patch attached now. Thanks for your patch, see my questions below: > > -- > Mauricio Faria de Oliveira > IBM Linux Technology Center > diff -Nru libvirt-2.3.0/debian/changelog