Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-30 Thread David Härdeman
David Härdeman wrote: I've attached a patch which details what this could look like... And here's an updated patch. This one is tested and works with one exception: if you do an automated partman-auto-lvm install two times in a row, the parted server seems to become confused and the manual

Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-30 Thread David Härdeman
On Wed, Aug 30, 2006 at 08:39:58AM +0200, David Härdeman wrote: And here's an updated patch. This one is tested and works with one exception: if you do an automated partman-auto-lvm install two times in a row, the parted server seems to become confused and the manual partitioning screen is

Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-29 Thread Ronny Aasen
David Härdeman wrote: On Thu, Aug 24, 2006 at 09:27:49PM +0200, Ronny Aasen wrote: Index: debian/partman-auto-lvm.templates === --- debian/partman-auto-lvm.templates(revision 40192) +++ debian/partman-auto-lvm.templates

Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-29 Thread David Härdeman
On Thu, Aug 24, 2006 at 09:27:49PM +0200, Ronny Aasen wrote: Index: debian/partman-auto-lvm.templates === --- debian/partman-auto-lvm.templates (revision 40192) +++ debian/partman-auto-lvm.templates (working copy) @@ -37,6

Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-28 Thread Geert Stappers
On Thu, Aug 24, 2006 at 09:27:49PM +0200, Ronny Aasen wrote: snip/ this works interactivly, and can preseed it as normal i expect. ^^^ Is it 'can preseed' or 'can be preseeded' ? hope someone can review and judge this Ronny ---

Bug#375491: yet another lvm_removal_on_demand patch suggestion

2006-08-24 Thread Ronny Aasen
here is attached my 3rd attempth for lvm removal on installations. It's basicaly Frans Pops suggestions. with a new template if lvm is detected on the disk, ask user if it's ok to delete them. if yes ; delete if no ; same error as before. If this is acceptable ill see about adding something to