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.0

Bug#595152: ZFS support

2010-11-09 Thread Robert Millan
2010/11/9 Colin Watson cjwat...@debian.org: 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

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

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-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact

Bug#595152: ZFS support

2010-09-06 Thread Robert Millan
Hi Otavio, 2010/9/5, Otavio Salvador ota...@ossystems.com.br: 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

Bug#595152: ZFS support

2010-09-06 Thread Otavio Salvador
Hello Robert, On Mon, Sep 6, 2010 at 7:48 AM, Robert Millan r...@debian.org wrote: Hi Otavio, 2010/9/5, Otavio Salvador ota...@ossystems.com.br: 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

Bug#595152: ZFS support

2010-09-05 Thread Otavio Salvador
Hello Robert, On Wed, Sep 1, 2010 at 9:36 AM, Robert Millan r...@debian.org 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

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