All of lore.kernel.org
 help / color / mirror / Atom feed
From: Glenn Washburn <development@efficientek.com>
To: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
Cc: Patrick Steinhardt <ps@pks.im>,
	Daniel Kiper <daniel.kiper@oracle.com>,
	The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: v7 for detached headers and key files
Date: Wed, 16 Dec 2020 12:35:13 -0600	[thread overview]
Message-ID: <20201216123513.1a60315d@crass-HP-ZBook-15-G2> (raw)
In-Reply-To: <20201210091459.11154-1-GNUtoo@cyberdimension.org>

On Thu, 10 Dec 2020 10:14:53 +0100
Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org> wrote:

> Hi,
> 
> I've addressed the issues found in the last round.
> 
> Like the last time I also tested it on real hardware.
> 
> I'm also sorry for the delay.

Great, thanks for the update. Since Grub is currently in a feature
freeze until the release, this won't get accepted until after. It'll
probably be less work for you to wait until after the release, then
rebase and fix any issues that might have arisen from now til then.
Though, at this point, probably not much will need to change in a
rebase.  Regardlesss, I doubt anyone will even look at them until then.
This is a feature that I really want in Grub because my testing patches
rely on this infrastructure, so I don't want it to get forgotten about
either.

Also, when updating a patch series, its nice to use the "--cover-letter"
and "--range-diff" options to git format-patch to show quickly what has
changed since the last patch series version.

Glenn



  parent reply	other threads:[~2020-12-16 18:35 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-10  9:14 v7 for detached headers and key files Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 1/6] cryptodisk: luks: unify grub_cryptodisk_dev function names Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 2/6] cryptodisk: geli: " Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 3/6] cryptodisk: enable the backends to implement detached headers Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 4/6] cryptodisk: add support for LUKS1 " Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 5/6] cryptodisk: enable the backends to implement key files Denis 'GNUtoo' Carikli
2020-12-10  9:14 ` [PATCH v7 6/6] cryptodisk: Add support for LUKS1 " Denis 'GNUtoo' Carikli
2020-12-16 18:35 ` Glenn Washburn [this message]
2020-12-17  0:27   ` v7 for detached headers and " Denis 'GNUtoo' Carikli
2021-08-17 22:30 ` Denis 'GNUtoo' Carikli
2021-08-26 18:06   ` Daniel Kiper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201216123513.1a60315d@crass-HP-ZBook-15-G2 \
    --to=development@efficientek.com \
    --cc=GNUtoo@cyberdimension.org \
    --cc=daniel.kiper@oracle.com \
    --cc=grub-devel@gnu.org \
    --cc=ps@pks.im \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.