<div dir="ltr"><div dir="ltr"><span class="gmail-im" style="color:rgb(80,0,80)"><div dir="ltr">On Wed, 8 Feb 2023 at 05:23, Rob Landley <<a href="mailto:rob@landley.net" target="_blank">rob@landley.net</a>> wrote:</div></span><div class="gmail_quote"><span class="gmail-im" style="color:rgb(80,0,80)"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It's after mkfs.vfat in my todo list because of rock ridge weirdness. Right.<br></blockquote></span><div>I might take a look at that as I need mkfs.vfat (or mtools) to create the ESP for UEFI bootable images. </div><span class="gmail-im" style="color:rgb(80,0,80)"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">And now I need to go back and add a couple extra paremeters, because TEST_HOST<br>is saying...<br><br>FAIL: blkid iso<br>echo -ne '' | BLKID iso<br>--- expected    2023-02-07 23:02:05.903941946 -0600<br>+++ actual      2023-02-07 23:02:05.911941946 -0600<br>@@ -1 +1 @@<br>-temp.img: SEC_TYPE="msdos" LABEL="MYISO" UUID="1970-01-02-12-55-42-00"<br>TYPE="iso9660"<br>+temp.img: UUID="2023-02-08-04-47-27-00" LABEL="CDROM" TYPE="iso9660"<br><br>And... sigh. Why are the fields out of order?</blockquote></span><div>Let me check a couple more blkid implementations and get back to you. :)</div><span class="gmail-im" style="color:rgb(80,0,80)"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>On 2/6/23 10:11, Eric Molitor wrote:<br>> Move SEC_TYPE for vfat to not display when searching by label<br>> Remove SEC_TYPE from iso9660<br>> ---<br>>  tests/blkid.test   | 6 +++---<br>>  toys/other/blkid.c | 4 +++-<br>>  2 files changed, 6 insertions(+), 4 deletions(-)<br>><br>> diff --git a/tests/blkid.test b/tests/blkid.test<br>> index 4c385dcb..37bce67a 100755<br>> --- a/tests/blkid.test<br>> +++ b/tests/blkid.test<br>> @@ -29,10 +29,10 @@ testing "f2fs" "BLKID f2fs" \<br>>    'temp.img: LABEL="myf2fs" UUID="b53d3619-c204-4c0b-8504-36363578491c"<br>> TYPE="f2fs"\n' \<br>>    "" ""<br>>  testing "iso" "BLKID iso" \<br>> -  'temp.img: SEC_TYPE="msdos" LABEL="MYISO" UUID="1970-01-02-12-55-42-00"<br>> TYPE="iso9660"\n' \<br>> +  'temp.img: LABEL="MYISO" UUID="1970-01-02-12-55-42-00" TYPE="iso9660"\n' \<br>>    "" ""<br><br>You removed SEC_TYPE from the output but debian is still producing it. Why did<br>you do that? I'd think "adjusting the tests to match the different output my<br>patch produces" would warrant at least a comment...</blockquote></span><div>See below but I will resubmit these as two sets of patches which I believe will be clearer. I'm guilty of compounding too many changes in a single patch.</div><div><div class="gmail-adm" style="margin:5px 0px"><div id="gmail-q_184" class="gmail-ajR gmail-h4" style="background-color:rgb(232,234,237);border:none;clear:both;line-height:6px;outline:none;width:24px;color:rgb(80,0,80);font-size:11px;border-radius:5.5px"><div class="gmail-ajT" style="background:url("https://www.gstatic.com/images/icons/material/system_gm/2x/more_horiz_black_20dp.png") 50% 50%/20px no-repeat;height:11px;opacity:0.71;width:24px"></div></div></div></div><div>Just removed from the ISO9660 test, not all. I was fixing the order of the output for SEC_TYPE to be consistent with the EXT code path and other blkid implementations. Just doing the above smaller change means SEC_TYPE renders before TYPE. FWICT SEC_TYPE should always follow TYPE. But let me test a bit more with various implementations and see if there is any consistency to this. </div><font color="#888888"><div><br></div><div>- Eric<br><br></div></font></div></div></div>