Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Giuseppe Iuculano
On 08/19/2010 04:29 PM, Bastian Blank wrote: > Sorry, this is not acceptable. The patch > - differes in comments, > - used path, > - removes autoconf parts without reason, autoreconf is called anyway, and > - is incomplete. Well, FWIW this is instead acceptable from a NMUer point of view, anyway t

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Bastian Blank
On Thu, Aug 19, 2010 at 01:29:18PM +0200, Giuseppe Iuculano wrote: > On 08/19/2010 12:54 PM, Bastian Blank wrote: > > Please describe the changes you made. It even differs in the comments. > > This only shows the annoncement, the patch is in > > https://bugzilla.redhat.com/attachment.cgi?id=434982

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Giuseppe Iuculano
On 08/19/2010 01:29 PM, Giuseppe Iuculano wrote: > It is the same patch without the configure and Makefile stuff (upstream > added --with-default-run-dir configure argument, I instead hardcoded it > to /var/run/clvmd.sock ). I removed that part to avoid autoreconf I just noted I forgot check_permi

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Giuseppe Iuculano
On 08/19/2010 12:54 PM, Bastian Blank wrote: > Please describe the changes you made. It even differs in the comments. > This only shows the annoncement, the patch is in > https://bugzilla.redhat.com/attachment.cgi?id=434982 It is the same patch without the configure and Makefile stuff (upstream

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Bastian Blank
On Thu, Aug 19, 2010 at 12:39:28PM +0200, Giuseppe Iuculano wrote: > On 08/19/2010 12:26 PM, Bastian Blank wrote: > > Where does this patch come from? It is not included into the upstream > > source this way. As long as this is not known: NACK. > It comes from upstream, I used the essential part of

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Giuseppe Iuculano
On 08/19/2010 12:26 PM, Bastian Blank wrote: > Where does this patch come from? It is not included into the upstream > source this way. As long as this is not known: NACK. It comes from upstream, I used the essential part of the patch. Please see: https://www.redhat.com/archives/linux-lvm/2010-Ju

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Bastian Blank
On Thu, Aug 19, 2010 at 12:06:34PM +0200, Giuseppe Iuculano wrote: > I've prepared an NMU for lvm2 (versioned as 2.02.66-2.1) and > uploaded it to DELAYED/1. Please feel free to tell me if I > should delay it longer. Where does this patch come from? It is not included into the upstream source this

Bug#591204: lvm2: diff for NMU version 2.02.66-2.1

2010-08-19 Thread Giuseppe Iuculano
tags 591204 + patch tags 591204 + pending thanks Dear maintainer, I've prepared an NMU for lvm2 (versioned as 2.02.66-2.1) and uploaded it to DELAYED/1. Please feel free to tell me if I should delay it longer. Regards. Giuseppe diff -Nru lvm2-2.02.66/debian/changelog lvm2-2.02.66/debian/changelo