On Tue, Mar 25, 2025 at 05:01:02PM +0100, Daniel Kiper wrote:
> On Fri, Mar 21, 2025 at 03:59:01PM +0800, Gary Lin wrote:
> > Extract the logic to handle the file buffer from the SRK recover
> > function to prepare to load the sealed key from the NV index handle,
> > so the NV index mode can share the same code path in the later patch.
> > The SRK recover function now only reads the file and sends the file
> > buffer to the new function.
> >
> > Besides this, to avoid introducing more options for the NV index mode,
> > the file format is detected automatically before unmarshalling the data,
> > so there is no need to use the command option to specify the file format
> > anymore. In other words, '--tpm2key' and '--keyfile' are the same now.
> 
> You are referring to long options in the commit message but documentation
> updates doing some stuff around short ones. I can understand they are
> equivalent but I think you should be consistent and use short form
> everywhere or both in the commit message.
> 
Agree. Mixing the long and short options is confusing. I'll update the
commit message to use the short option.

> Additionally, it seems to me the docs mention short or long options but
> do not show there are some equivalents. Am I right? If yes I think this
> should be fixed in separate patch.
> 
When writing the document, I mainly use the short options. So yes, it
needs to address the equivalent long options. I'll update the document.

Gary Lin

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

Reply via email to