Some clarifications for the BLS #129
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
That paragraph was in "Boot Loader Entries", but "The Partitions"
seems more appropriate. Use semantic line breaks and remove some
hedging ("is not supposed to describe" → "does not describe", etc.).
This is ambiguous, because "kernel binary" could be understood to be the .linux part by itself. And an initrd or commandline are not really metadata. Instead say that the things are combined into a single file without trying to give them a common name.
There was a discussion on fedora-devel whether grub2's addition of interpolation of variables is in agreement with the spec. Let's make it clearer that the value is just a value that should be used as described. For 'linux', do not say that the distro is supposed to create a single directory for all kernels. It's supposed to be a single directory per kernel version… Also extend the description of 'initrd' to have the same format as 'linux'. One could think that they're somehow different. Extend the example to show how two initrds can be combined.
bluca
approved these changes
Nov 5, 2024
behrmann
approved these changes
Nov 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.