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

Fix grub config #2661

Merged
merged 3 commits into from Jul 27, 2021
Merged

Fix grub config #2661

merged 3 commits into from Jul 27, 2021

Conversation

DEvil0000
Copy link
Contributor

  • Type: Bug Fix / Enhancement
  • Impact: Normal
  • Reference to related issue (URL):
  • How was this pull request tested?
    with output=usb, partitioned gpt, bootloader grub, no efi/legacy boot on apu2 hardware with firmware 4.13.0.6
  • Brief description of the changes in this pull request:
    This PR fixes two bugs in grub config while improving readability of config write function a bit.
  1. it fixes all the none efi cases for the boot menu in the grub config
  2. it fixes or actually adds serial configuration to the config

@jsmeix jsmeix self-assigned this Jul 22, 2021
@jsmeix jsmeix added the enhancement Adaptions and new features label Jul 22, 2021
@jsmeix jsmeix added this to the ReaR v2.7 milestone Jul 22, 2021
@jsmeix jsmeix requested a review from a team July 22, 2021 11:33
Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

This needs to wait until I fixed current OUTPUT=USB regressions, cf.
#2660 (comment)

Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

With #2660
and #2659 merged
this one can be also merged, cf.
#2659 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adaptions and new features fixed / solved / done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants