search

go-debos / debos

Debian OS builder
Apache License 2.0
533 stars 136 forks source link

The MBR extended implementation causes a problem with a non-MBR configuration #473

Closed robertkirkman closed 7 months ago

robertkirkman commented 7 months ago

Hi I don't know enough about the project yet to know how to create a minimal reproducible example, but when i use this file with debos, if I use commit 03ec3cbfab6a73087ebbb58317f65e117342c1ba of debos it works but if I use commit da773ccbc899d2f5b236f80db303062693f6f2b6 or any of the recent commits after that, the debos command gets stuck inside the fakemachine because of a partition named "extended" getting incorrectly created in the wrong place even though the .yaml file specifies gpt and not mbr

If I have time I will try to create a minimal reproducible example

robertkirkman commented 7 months ago

Thank you I tested that patch and the same .yaml file I tested with the other 2 versions of debos works with your msdos-fixes branch!

sjoerdsimons commented 7 months ago

Thanks for testing!