13560 2019-09-27 18:40:04 +0000 Partition numbering is broken for MBR primary partition #4 2019-10-17 15:15:09 +0000 1 1 7 Build System, Metadata & Runtime Meta-yocto meta-yocto 2.7.2 x86 Multiple RESOLVED FIXED Medium+ normal 2.8 M4 1 michaelcooper81 richard.purdie akuster poky.bs.watcher poky.watcher randy.macleod --- --- No (bug/feature does not impact docs) oldest_to_newest 85123 0 4572 michaelcooper81 2019-09-27 18:40:04 +0000 Created attachment 4572 Proposed patch When wks describes extra partitions that aren't in the partition table (e.g. boot loader) and describes four proper MBR partitions, the last partition gets added to fstab as partition #5 instead of #4. Repro wks: part bootloader --no-table part /boot --ondisk mmcblk0 --label boot part / --ondisk mmcblk0 --label root1 part / --ondisk mmcblk0 --label root2 part /data --ondisk mmcblk0 --label data bootloader --ptable msdos Generated fstab: /dev/mmcblk0p1 /boot ext4 defaults 0 0 /dev/mmcblk0p5 /data (/dev/mmcblk0p5 should be /dev/mmcblk0p4) Proposed patch attached. 85187 1 randy.macleod 2019-10-03 14:37:13 +0000 Is there a test that we can add to ensure that this doesn't happen in the future? Are you able to send a patch to the email list? https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded#Sending_using_git-send-email 85365 2 akuster 2019-10-17 14:52:27 +0000 http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/scripts?id=7a1d68d136c9746242d2243c7e9959758437a3fa 85368 3 akuster 2019-10-17 14:53:30 +0000 made it into zeus 85369 4 richard.purdie 2019-10-17 14:54:06 +0000 Fixed with the above commit. 85375 5 richard.purdie 2019-10-17 15:15:09 +0000 Mark as resolved 4572 2019-09-27 18:40:04 +0000 2019-09-27 18:40:04 +0000 Proposed patch fix_mbr_primary_part_4_numbering.patch application/octet-stream 1034 michaelcooper81 ZGlmZiAtLWdpdCBhL3NjcmlwdHMvbGliL3dpYy9wbHVnaW5zL2ltYWdlci9kaXJlY3QucHkgYi9z Y3JpcHRzL2xpYi93aWMvcGx1Z2lucy9pbWFnZXIvZGlyZWN0LnB5CmluZGV4IDY3YWRkZWZmM2Uu LjRhNTM0ZWEzNjcgMTAwNjQ0Ci0tLSBhL3NjcmlwdHMvbGliL3dpYy9wbHVnaW5zL2ltYWdlci9k aXJlY3QucHkKKysrIGIvc2NyaXB0cy9saWIvd2ljL3BsdWdpbnMvaW1hZ2VyL2RpcmVjdC5weQpA QCAtMzI2LDYgKzMyNiw3IEBAIGNsYXNzIFBhcnRpdGlvbmVkSW1hZ2UoKToKICAgICAgICAgc2Vs Zi5zZWN0b3Jfc2l6ZSA9IFNFQ1RPUl9TSVpFCiAgICAgICAgIHNlbGYubmF0aXZlX3N5c3Jvb3Qg PSBuYXRpdmVfc3lzcm9vdAogCisgICAgICAgIG51bV9yZWFsX3BhcnRpdGlvbnMgPSBsZW4oW3Ag Zm9yIHAgaW4gc2VsZi5wYXJ0aXRpb25zIGlmIG5vdCBwLm5vX3RhYmxlXSkKICAgICAgICAgIyBj YWxjdWxhdGUgdGhlIHJlYWwgcGFydGl0aW9uIG51bWJlciwgYWNjb3VudGluZyBmb3IgcGFydGl0 aW9ucyBub3QKICAgICAgICAgIyBpbiB0aGUgcGFydGl0aW9uIHRhYmxlIGFuZCBsb2dpY2FsIHBh cnRpdGlvbnMKICAgICAgICAgcmVhbG51bSA9IDAKQEAgLTMzNCw3ICszMzUsNyBAQCBjbGFzcyBQ YXJ0aXRpb25lZEltYWdlKCk6CiAgICAgICAgICAgICAgICAgcGFydC5yZWFsbnVtID0gMAogICAg ICAgICAgICAgZWxzZToKICAgICAgICAgICAgICAgICByZWFsbnVtICs9IDEKLSAgICAgICAgICAg ICAgICBpZiBzZWxmLnB0YWJsZV9mb3JtYXQgPT0gJ21zZG9zJyBhbmQgcmVhbG51bSA+IDMgYW5k IGxlbihwYXJ0aXRpb25zKSA+IDQ6CisgICAgICAgICAgICAgICAgaWYgc2VsZi5wdGFibGVfZm9y bWF0ID09ICdtc2RvcycgYW5kIHJlYWxudW0gPiAzIGFuZCBudW1fcmVhbF9wYXJ0aXRpb25zID4g NDoKICAgICAgICAgICAgICAgICAgICAgcGFydC5yZWFsbnVtID0gcmVhbG51bSArIDEKICAgICAg ICAgICAgICAgICAgICAgY29udGludWUKICAgICAgICAgICAgICAgICBwYXJ0LnJlYWxudW0gPSBy ZWFsbnVtCgo=