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
stages/sfdisk: support changing GPT partition attribute bits #966
Conversation
06a8ace
to
41a3ba5
Compare
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.
Hi @eballetbo! Thanks for your contribution. It looks reasonable to me, I only have one style comment.
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.
Thanks so much! In addition to what @teg said, just a small print statement that I think we don't need.
When is specified change the GPT partition attribute bits. The attributes is a comma list of bits numbers or bit names. You can take a look at the manual page of sfdisk to see the supported attribute bits.
41a3ba5
to
ee9f1a4
Compare
Pushed a new branch with the requested changes. Using an array of integers and removed the debug message introduced by mistake. |
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.
Looks good to me!
Restarted the test. |
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.
Thanks a lot!
The GPT attribute field is a 64-bit field that contains two subfields. The higher field is interpreted only in the context of the partition ID, while the lower field is common to all partition IDs. sfdisk supports to change this fields, and also a osbuild user might be interested on change these bits. This patchset adds support to change these bits in the sfdisk stage.
The meaning of these bits (from Wikipedia) :