Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mkksiso: option --replace to replace arbitrary text in boot config. #1402

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

sstark
Copy link
Contributor

@sstark sstark commented Jun 24, 2024

  • It is a generalized version of the code that replaces the volume ID.

  • Test adjusted accordingly because signature changed slightly.

  • This can be used to augment/change the user visible boot menu to add more information about the kickstart installation that is going to happen.

  • Implemented for grub and isolinux, not s390 (don't know how to test).

  - It is a generalized version of the code that replaces the volume ID.

  - Test adjusted accordingly because signature changed slightly.

  - This can be used to augment/change the user visible boot menu to add
    more information about the kickstart installation that is going to
    happen.

  - Implemented for grub and isolinux, not s390 (don't know how to
    test).
@bcl
Copy link
Contributor

bcl commented Jun 24, 2024

Thanks, this looks pretty good, but please also update it for S390 and don't worry about testing -- the volid change should be enough and the signature for the functions needs to be consistent.

@sstark
Copy link
Contributor Author

sstark commented Jun 25, 2024

I added a commit on top for that, let me know if you would rather have that squashed into one.

Copy link
Contributor

@bcl bcl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@bcl bcl merged commit 62c7426 into weldr:master Jun 25, 2024
2 checks passed
@sstark sstark deleted the customize-grub-menu branch June 26, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants