Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-19 Thread Guilhem Moulin
Control: severity -1 wishlist On Mon, 19 Sep 2016 at 08:21:13 -0500, Richard Laager wrote: > On 09/19/2016 07:21 AM, Guilhem Moulin wrote: >> Interesting, worksformeâ„¢ both with gawk(1) and mawk(1). Perhaps an >> issue with TAB vs spaces? Could you redirect the output of `zfs status >> -P $pool`

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-19 Thread Richard Laager
On 09/19/2016 07:21 AM, Guilhem Moulin wrote: > On Mon, 19 Sep 2016 at 06:39:35 -0500, Richard Laager wrote: >> On 09/19/2016 04:48 AM, Guilhem Moulin wrote: >>> On Mon, 19 Sep 2016 at 00:51:45 -0500, Richard Laager wrote: On 09/18/2016 02:13 PM, Guilhem Moulin wrote: >>> Alright, thanks! I

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-19 Thread Guilhem Moulin
On Mon, 19 Sep 2016 at 06:39:35 -0500, Richard Laager wrote: > On 09/19/2016 04:48 AM, Guilhem Moulin wrote: >> On Mon, 19 Sep 2016 at 00:51:45 -0500, Richard Laager wrote: >>> On 09/18/2016 02:13 PM, Guilhem Moulin wrote: >> Alright, thanks! I think a more robust AWK or sed script is desired to

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-19 Thread Richard Laager
On 09/19/2016 04:48 AM, Guilhem Moulin wrote: > On Mon, 19 Sep 2016 at 00:51:45 -0500, Richard Laager wrote: >> On 09/18/2016 02:13 PM, Guilhem Moulin wrote: > Fair enough it's not the first field, but by looking at the output I > have the impression that the header is pretty much free form.

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-19 Thread Guilhem Moulin
On Mon, 19 Sep 2016 at 00:51:45 -0500, Richard Laager wrote: > On 09/18/2016 02:13 PM, Guilhem Moulin wrote: >> The kernel parameters >> approach and the /etc/fstab approach are both FS-agnostic, and I'd >> rather not have FS-specific code to find out which device is mounted on >> / or /usr. > >

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-18 Thread Richard Laager
On 09/18/2016 02:13 PM, Guilhem Moulin wrote: > On Sat, 17 Sep 2016 at 19:37:23 -0500, Richard Laager wrote: >> I have changes all ready-to-go for supporting LUKS, but this "bug" >> (read: feature request) is a blocker. It is possible to work-around this >> with (otherwise unused) /etc/fstab

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-18 Thread Guilhem Moulin
On Sat, 17 Sep 2016 at 19:37:23 -0500, Richard Laager wrote: > I have changes all ready-to-go for supporting LUKS, but this "bug" > (read: feature request) is a blocker. It is possible to work-around this > with (otherwise unused) /etc/fstab entries, but I don't want to > encourage that in the

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-17 Thread Richard Laager
On 09/17/2016 06:45 PM, Guilhem Moulin wrote: > On Fri, 16 Sep 2016 at 04:32:24 -0500, Richard Laager wrote: >> The attached patch adds ZFS support to cryptsetup. > > Thanks the patch. Unfortunately I can't test it easily, and I know > nothing about ZFS, but I'll try to review your patch anyway.

Bug#838001: [pkg-cryptsetup-devel] Bug#838001: cryptsetup does not support ZFS

2016-09-17 Thread Guilhem Moulin
Hi Richard, On Fri, 16 Sep 2016 at 04:32:24 -0500, Richard Laager wrote: > The attached patch adds ZFS support to cryptsetup. Thanks the patch. Unfortunately I can't test it easily, and I know nothing about ZFS, but I'll try to review your patch anyway. > + zfs list -H -o