Bug#595152: ZFS support

2010-11-09 Thread Robert Millan
2010/11/9 Colin Watson : > apt-install is not that cheap; it has to set up the target chroot and > then tear it all down again.  Could you please add a bit of state here > to remember when grub_probe has already been called? Ok > I think this should be OK, although I'm (possibly unjustifiably) >

Bug#595152: ZFS support

2010-11-09 Thread Colin Watson
On Wed, Sep 01, 2010 at 02:36:17PM +0200, Robert Millan wrote: > This patch adds ZFS support to grub-installer. Thanks for this patch. > diff -ur grub-installer-1.55.old/grub-installer > grub-installer-1.55/grub-installer > --- grub-installer-1.55.old/grub-installer2010-08-18 16:38:02.00

Bug#595152: ZFS support in grub-installer

2010-11-06 Thread Robert Millan
Is there any inconvenient with the patch I've sent? Note that although it introduces a direct call to grub-probe, current version already uses grub-probe indirectly. There's really no way around relying on grub-probe: every time you call grub-install or update-grub, grub-probe is running behind t

Bug#595152: ZFS support

2010-10-17 Thread Robert Millan
After #595007 and #595759 have been fixed, this is the last remaining piece for basic ZFS support in D-I to be usable. Is my patch ok? -- Robert Millan -- To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.o

Bug#595152: ZFS support

2010-09-06 Thread Otavio Salvador
Hello Robert, On Mon, Sep 6, 2010 at 7:48 AM, Robert Millan wrote: > Hi Otavio, > > 2010/9/5, Otavio Salvador : >> I didn't get the reason why you created grub_probe. Why grub-common >> wasn't available in our target? > > IIUC grub-common isn't yet installed the first time findfs() is used. > > A

Bug#595152: ZFS support

2010-09-06 Thread Robert Millan
Hi Otavio, 2010/9/5, Otavio Salvador : > I didn't get the reason why you created grub_probe. Why grub-common > wasn't available in our target? IIUC grub-common isn't yet installed the first time findfs() is used. Alternatively, one could just apt-install grub-common and then use $chroot $ROOT gr

Bug#595152: ZFS support

2010-09-05 Thread Otavio Salvador
Hello Robert, On Wed, Sep 1, 2010 at 9:36 AM, Robert Millan wrote: > This patch adds ZFS support to grub-installer. > > Note: findfs change is required because mount output only gives zpool name, > not the device associated with it.  The other changes should be obvious. I didn't get the reason w

Bug#595152: ZFS support

2010-09-01 Thread Robert Millan
Package: grub-installer Version: 1.55 Severity: wishlist Tags: patch This patch adds ZFS support to grub-installer. Note: findfs change is required because mount output only gives zpool name, not the device associated with it. The other changes should be obvious. -- System Information: Debian R