<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 7, 2023 at 5:28 PM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 6/7/23 14:40, enh wrote:<br>
> indeed. i don't know _why_ this is happening,<br>
<br>
Race condition setting up the VM? Permissions hiccup? No idea, but I've been<br>
rereading the Discworld "watch" books recently and... let's just say by halfway<br>
through Jingo prolonged exposure to Sam Vimes is not making me LESS inclined to<br>
track things down. :P<br>
<br>
In the "I break everything" department: your description has a patch in it, so<br>
"git am" on the attached file is barfing because the file "actual" is not in the<br>
repo. (I would have assumed git format-patch had an existing escape syntax for<br>
the in-band signalling the way "From:" lines in mbox files do, but apparently not?)<br>
<br>
> but i think that means the fix (attached) is obvious at least.<br>
<br>
Your sort returns modules from the newest kernel installed, but I let apt-get<br>
upgrade install new kernels regularly-ish, but sometimes don't reboot my laptop<br>
for months at a time, so the newest kernel installed in the filesystem and the<br>
one I'm running aren't necessarily the same thing. (And sort isn't version<br>
sort...) Also, in _theory_ you could have one module each installed from two<br>
kernel versions (but a _different_ single module installed from each kernel) and<br>
then the two modules test would fail...<br>
<br>
Instead I'm adding $(uname -r) to $MODULE_ROOT in the non-android case to avoid<br>
duplicates, and... </blockquote><div><br></div><div>good point. (i get forcibly rebooted rather regularly by security policy, so i'm rarely in this situation.)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">why are we discarding stderr on the calls to head and tail?<br>
(It's not going to discard it from find, it's in the wrong place in the pipeline<br>
for that...)<br>
<br>
I also added comments about why we DON'T have to quote stuff, because it was<br>
either that or add the quotes to get my shoulders to relax...<br>
<br>
Try commit a5b5749cc7f0<br></blockquote><div><br></div><div>that actually seems to have _broken_ things more reliably? looking at your change, it seems like you forgot to actually touch the -F test?</div><div><br></div><div>-- modinfo<br>basename: Needs 1 argument (see "basename --help")<br>basename: Needs 1 argument (see "basename --help")<br>basename: Needs 1 argument (see "basename --help")<br>basename: Needs 1 argument (see "basename --help")<br>PASS: modinfo missing<br>modinfo: Needs 1 argument (see "modinfo --help")<br>PASS: modinfo treats - and _ as equivalent<br>modinfo: Needs 1 argument (see "modinfo --help")<br>PASS: modinfo treats _ and - as equivalent<br>modinfo: Needs 1 argument (see "modinfo --help")<br>FAIL: modinfo -F filename gets absolute path<br>echo -ne '' | modinfo -F filename <br>--- expected 2023-06-08 17:21:10.863977489 +0000<br>+++ actual 2023-06-08 17:21:10.871977488 +0000<br>@@ -1 +0,0 @@<br>-<br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Rob<br>
</blockquote></div></div>