From: Andrei Borzenkov <arvidjaar@gmail.com>
To: grub-devel@gnu.org
Subject: Re: [PATCH] Allow to add/change menu entry class defaults.
Date: Sun, 27 Dec 2015 20:03:36 +0300 [thread overview]
Message-ID: <568019E8.2040301@gmail.com> (raw)
In-Reply-To: <567F03CE.9050007@riseup.net>
[-- Attachment #1: Type: text/plain, Size: 1512 bytes --]
27.12.2015 00:17, Robin Schneider пишет:
> I am sorry for the misunderstanding. I should have explained the indention
> behind my patch a bit better then just linking to another patch which makes use
> of the newly introduced variables by this patch.
>
> My indented use case is to allow to add options like '--unrestricted' or
> '--users "Jane"' to each menuentry generated by grub-mkconfig without altering
> the scripts itself.
Oh, no, sorry. CLASS is for adding --class option and --class option is
for defining icon used to represent menu entry. Please do not misuse it
for something else.
I try to understand possible use cases.
Please get a look at
https://lists.gnu.org/archive/html/grub-devel/2015-05/msg00170.html
thread. SUSE has actually implemented my suggestion. This gives us "all
menu entries unrestricted" case.
Do you really have situation where you need separate category of users
that won't have access to CLI but will be the *only* users allowed to
select non-default menu entry? Moreover, do you really need to allow
different users to boot different categories of menu entries?
...
>
> BTW: The efi menuentry has the class 'windows'. Is that correct? My patch
> assumes that this menuentry is indented for UEFI applications.
>
Well, so far upstream os-prober only detects Windows on EFI. But yes,
SUSE includes additional script.
See https://lists.gnu.org/archive/html/grub-devel/2015-12/msg00103.html
- does it address your concern?
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2015-12-27 17:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-22 15:19 [PATCH] Allow to add/change menu entry class defaults Robin Schneider
2015-12-23 6:58 ` Andrei Borzenkov
2015-12-23 20:54 ` Robin Schneider
2015-12-24 8:21 ` Andrei Borzenkov
2015-12-26 21:17 ` Robin Schneider
2015-12-27 17:03 ` Andrei Borzenkov [this message]
2015-12-27 20:44 ` Robin Schneider
2016-01-13 12:12 ` Robin Schneider
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=568019E8.2040301@gmail.com \
--to=arvidjaar@gmail.com \
--cc=grub-devel@gnu.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).