-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Handle disk sector size #621
base: main
Are you sure you want to change the base?
Conversation
Fixes #448 |
Note to self: Here is the command I used to launch the elementary OS live ISO and test the installer, after creating a raw format qemu disk image with
I just installed the After running the installer IIRC I just changed the |
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.
I've imported a version of distinst with the associated fixes into the elementary PPAs, and the code looks pretty good here, just one small thing to change and then I'll test this out.
I don't like passing this sector_size
through all of these layers of construction since it's just a property of the disk, not of every partition and mount, but that isn't a problem with your code, it's jut the original architecture of this, so we can sort that later.
@davidmhewitt @elementary/desktop-developers I've created PR pop-os/distinst#278 and along with this PR it appears to succeed installation for the 3 test cases I tried in QEMU:
Note that I say "appears to succeed" because the installer succeeds and partitions appear to be created fine, I've not rebooted into the installed image in QEMU to verify it works post install - would appreciate if anyone else would be able to test and comment. In the mean time I'll try to more fully test in the coming days and report my findings here. |
Fixes #620
Requires pop-os/distinst#266