Re: [PATCH] Allow specifying the boot device by its label instead of UUID

[Oskari Pirhonen <xxc3ncoredxx@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1437 bytes --]

I know you're just learning, so here's some more tips:

Updated patches are generally sent as a "vN" where N is the revision of
the series. `git format-patch` can do this nicely with the `-v` arg.
This would make the subject line look like "[PATCH vN] Allow specifying
the boot device by its label instead of UUID".

Updated patches are also typically sent as a new thread instead of as a
reply to the old one. Makes for easier review.

On Wed, Sep 27, 2023 at 16:25:23 +0000, darkpenguin wrote:
> Here it is, as a single patch, and in reply to the corresponding thread this time!
> 

The way git handles patches, applying this would make the above sentence
the commit message. Which is probably not what you were after.

> ---
>  util/grub-mkconfig.in     |  1 +
>  util/grub-mkconfig_lib.in | 14 +++++++++++---
>  util/grub.d/10_linux.in   |  3 +++
>  3 files changed, 15 insertions(+), 3 deletions(-)
> 
> diff --git a/util/grub-mkconfig.in b/util/grub-mkconfig.in
> index 32c480dae..fb5000d3f 100644
> --- a/util/grub-mkconfig.in
> +++ b/util/grub-mkconfig.in
> @@ -244,6 +244,7 @@ export GRUB_DEFAULT \
>    GRUB_DISABLE_UUID \
>    GRUB_DISABLE_LINUX_UUID \
>    GRUB_DISABLE_LINUX_PARTUUID \
> +  GRUB_ENABLE_LINUX_LABEL \

I still think this should be GRUB_DISABLE_LINUX_LABEL to match the other
vars.

>    GRUB_DISABLE_RECOVERY \
>    GRUB_VIDEO_BACKEND \
>    GRUB_GFXMODE \

- Oskari

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 141 bytes --]

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