Re: Support for detached LUKS header

2018-03-14 Thread John Lane

> Could anyone give their approval for merging?

> May I ask you or the original author to rebase the patches
> on the latest master? Then I will take a look. And please
> do not forget to CC me.

> Daniel

Daniel I am the original author of the aforementioned patches.

I have reapplied my patches to the current master head
(28b0d19061d66e3633148ac8e44decda914bf266).

I had to make a slight tweak to patch 4 to change one line of context
surrounding one hunk of the patch, but I didn't need to change any of
the patch code.

There are also two new patches with changes received from contributors'
pull requests.

I will forward the patches shortly.

My patches apply cleanly to the current head.

___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel


Re: Support for detached LUKS header

2018-03-07 Thread Daniel Kiper
On Tue, Mar 06, 2018 at 11:30:14AM +0100, Yannik Sembritzki wrote:
> Hello,
>
> unfortunately, the patches for cryptomount
>
> ?? 0001-Cryptomount-support-LUKS-detached-header.patch
> ?? 0002-Cryptomount-support-key-files.patch
> ?? 0003-Cryptomount-luks-allow-multiple-passphrase-attempts.patch
> ?? 0004-Cryptomount-support-plain-dm-crypt.patch
> ?? 0005-Cryptomount-support-for-hyphens-in-UUID.patch
>
> which were submitted to grub-devel in 2015 (see
> https://lists.gnu.org/archive/html/grub-devel/2015-06/msg00109.html) and
> were again discussed on the grub-help mailing list last year, with the
> result of revisiting after the next release, have still not been merged.
>
> The author of these patches has provided a summary page about the
> patches here: http://grub.johnlane.ie/
>
> I would personally advocate for merging these patches, as I see immense
> use in them. To provide plausible deniability in a crypto setup,
> detached luks headers are very much required.
>
> Could anyone give their approval for merging?

May I ask you or the original author to rebase the patches
on the latest master? Then I will take a look. And please
do not forget to CC me.

Daniel

___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel


Support for detached LUKS header

2018-03-06 Thread Yannik Sembritzki
Hello,

unfortunately, the patches for cryptomount

    0001-Cryptomount-support-LUKS-detached-header.patch
    0002-Cryptomount-support-key-files.patch
    0003-Cryptomount-luks-allow-multiple-passphrase-attempts.patch
    0004-Cryptomount-support-plain-dm-crypt.patch
    0005-Cryptomount-support-for-hyphens-in-UUID.patch

which were submitted to grub-devel in 2015 (see
https://lists.gnu.org/archive/html/grub-devel/2015-06/msg00109.html) and
were again discussed on the grub-help mailing list last year, with the
result of revisiting after the next release, have still not been merged.

The author of these patches has provided a summary page about the
patches here: http://grub.johnlane.ie/

I would personally advocate for merging these patches, as I see immense
use in them. To provide plausible deniability in a crypto setup,
detached luks headers are very much required.

Could anyone give their approval for merging?

Best regards
Yannik


___
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel