All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: grub-devel@gnu.org
Subject: Re: [PATCH] Increase flexibility of kernel naming, allow non-versioned kernels.
Date: Sun, 29 Apr 2012 18:28:12 +0200	[thread overview]
Message-ID: <4F9D6C1C.6040202@gmail.com> (raw)
In-Reply-To: <4F9CC7A0.3090406@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 1748 bytes --]

On 29.04.2012 06:46, Mike Gilbert wrote:
> I am relaying a patch by Robin Johnson, one of the core infrastructure
> staff at Gentoo Linux. In the process of building some Gentoo servers
> utilizing GRUB 2, he has made some changes to 10_linux that should make
> grub-mkconfig "out of the box" for additional users.
>
> I did a little work to clean up the indentation and wrote a proper
> changelog. Credit should go to him.
>
> Here is his description:
>
> Increase flexibility of kernel naming, allow non-versioned kernels.
> The block that tried to find the kernels was getting unweidly long, as
> was not
> easily customizable by users or distributors. Refactor to introduce a new
> variable, GRUB_KERNEL_GLOB, that allows complete control over the naming
> used
> to search for kernel binaries.
It was well structured and its length is solely due to distros using
multiple names. We can't make it shorter and this patch surely didn't.
Also it looks like it actually degraded compatibility on POSIX systems.
I don't consider that being able to customise the kernel naming is of
any advantage. Saying otherwise is like saying "let's make location of
/etc/passwd customizable". Some objects just have to be named as given.
I don't see any advantage to renaming kernels, especially given that you
can change versionstring as you see fit.
> Add 'bzImage' to the list of default names to support more distribution
> naming
> variants.
>
> Adjust the default set of globs to look for unversioned kernels before
> versioned kernels, to find symlinked kernel names.
Additionally empty version would probably result in some quirks not
addressed in this patch.

-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]

  reply	other threads:[~2012-04-29 16:28 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-29  4:46 [PATCH] Increase flexibility of kernel naming, allow non-versioned kernels Mike Gilbert
2012-04-29 16:28 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2012-04-29 20:07   ` Mike Gilbert
2012-05-01 23:17     ` Vladimir 'φ-coder/phcoder' Serbinenko

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=4F9D6C1C.6040202@gmail.com \
    --to=phcoder@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 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.