Skip to content

Corrected bug reported in issue #866 #955

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.

Sign up for GitHub

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

Jump to bottom
Merged
jsmeix merged 1 commit into from
Aug 9, 2016
Merged

Corrected bug reported in issue #866 #955

jsmeix merged 1 commit into from
Aug 9, 2016

Conversation

@gozora
Copy link
Member

@gozora gozora commented Aug 8, 2016

No description provided.

@gozora
Corrected bug reported in issue #866 where distributions without grub…
b81693f 
… modules

stored in /usr/lib/grub/x86_64-efi got incomplete bootx64.efi loader.

Corrected bug (not yet reported) when SLES12 SP1 boot ends with
kernel panic due missing root variable in grub2.
@gozora gozora mentioned this pull request Aug 8, 2016
Closed
@jsmeix jsmeix added bug The code does not do what it is meant to do fixed / solved / done labels Aug 9, 2016
@jsmeix jsmeix added this to the Rear v1.19 milestone Aug 9, 2016
@jsmeix jsmeix self-assigned this Aug 9, 2016
@jsmeix
Copy link
Member

jsmeix commented Aug 9, 2016

@gozora
your comment in lib/bootloader-functions.sh about
"SLES12 SP1 boot throw kernel panic without root= set"
shows that you have tested it at least on SLES12 SP1
which is sufficient for me to just merge it so that @abbbi
can easily test if now #866
works for him.

@gozora
Copy link
Member Author

gozora commented Aug 9, 2016

@jsmeix Yes, correct.
My UEFI test vector currently consists from testing on SLES12 SP1, Debian and Centos.

@jsmeix jsmeix merged commit 5873085 into rear:master Aug 9, 2016
@jsmeix
Copy link
Member

jsmeix commented Aug 9, 2016

@gozora
and once more many thanks for your
valuable contribution to Relax-and-Recover!

@jsmeix jsmeix mentioned this pull request Aug 9, 2016
Closed
@gozora
Copy link
Member Author

gozora commented Aug 9, 2016

no problem ;-)

@gozora gozora deleted the efiboot branch August 9, 2016 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jsmeix jsmeix

Labels

bug The code does not do what it is meant to do fixed / solved / done

Projects

None yet

Milestone

Rear v1.19

Development

Successfully merging this pull request may close these issues.

2 participants

@gozora @jsmeix