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
Correct boot flag #245
Correct boot flag #245
Conversation
@@ -1,4 +1,11 @@ | |||
------------------------------------------------------------------- | |||
Mon Jun 29 12:46:58 UTC 2015 - jreidinger@suse.com | |||
|
|||
- set only proper boot boot flags, otherwise it can confuse some |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicated boot
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, thanks for catch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The term proper boot flags
is a bit vague, could you be more specific if possible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly. I see that the bug report does not help much because you wrote it yourself :) but please try a user's perspective: if the old code failed, the circumstances and symptoms would be such and such. Probably include "Distinguish between MSDOS and GPT tables when setting boot flags" ?
The code looks good (but maybe I change my mind after you fix the description ;-) ) OTOH, Rubocop has no feelings: "Class definition is too long. [179/176]" |
LGTM |
No description provided.