<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 28, 2021 at 4:36 PM enh <<a href="mailto:enh@google.com">enh@google.com</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 Tue, Sep 28, 2021 at 4:31 PM Rob Landley <<a href="mailto:rob@landley.net" target="_blank">rob@landley.net</a>> wrote:<br>
><br>
> On 9/28/21 11:55 AM, enh wrote:<br>
> > Thoughts/explanation of how I misunderstood what you were asking here? (We build<br>
> > uname for macOS host builds, so I can't sync toybox if uname doesn't build.)<br>
><br>
> (Explanation: now that uname.c doesn't have any other #ifdefs in it, I wanted to<br>
> fix this in portability.h. That's why I didn't just apply your patch. I didn't<br>
> know macos #defines __linux__.)<br>
<br>
it doesn't.<br>
<br>
> For some reason, the previous commit doesn't have a little check mark OR a<br>
> little X next to it in the github web view for me...? (The new one does.)<br>
<br>
yeah, i don't know how that works. sometimes it skips commits. rate<br>
limiting? busy with other stuff?<br>
<br>
> And I pushed a 'maybe this fixes it' commit to the mactest branch and that<br>
> doesn't have a test run either? (It's had 10 minutes. make clean && time make"<br>
> on my laptop takes just under 12 seconds, and this model was introduced in 2012.)<br>
<br>
i _think_ the guy who set this up (apologies to him if he's reading<br>
--- i'm terrible with names) set it up for the master branch, not all<br>
branches? that's certainly how the toybox.yml file reads to me (search<br>
for `branches`).<br>
<br>
> Do I need to do something to convince it to run a test build? Like touch a file<br>
> in the www directory perhaps?<br>
<br>
i think you can just push to master? or give me the copy & paste to<br>
pull the other branch. is it -b or is that just repo? yeah, it's -b<br>
just like repo...<br>
<br>
no, that's still broken because it was the PER_LINUX32 typo mentioned<br>
in my previous mail that was the problem, not the #ifdef :-)<br></blockquote><div><br></div><div>...oh, and that personality() wasn't an inline. i've sent a patch to fix that (and with that, things build again on my mac). this still feels like a worse choice than just splitting linux32 out though...</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>