<div dir="ltr">the Android.bp file for toybox has<div><br></div><div>        "-Wno-string-plus-int",<br><br>since this is quite common in toybox code.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 23, 2024 at 1:24 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 10/18/24 18:09, Ray Gardner wrote:<br>
> The attached patches (tested with git am) bring toybox awk up to date.<br>
> The first two are the same ones I sent last time I sent patches (Sep<br>
> 7) because those have not been applied.<br>
<br>
Lost in the noise, apparently. No obvious objections.<br>
<br>
> I hope these can be applied<br>
> before the next toybox release.<br>
<br>
I applied the first 5 of this list, 6 is the compiler being stupid and <br>
I'd prefer to find a --fshut-up flag to make it stop. (I do <br>
"string"+offset in many places, and don't intend to stop. Dereference <br>
and then take the address of to humor compiler warnings is just silly.)<br>
<br>
7 starts with a comment change that seemed at first glance like part of <br>
6, thought I'd ask.<br>
<br>
My local llvm builds look like:<br>
<br>
../llwrap make clean defconfig awk<br>
<br>
(Which is warning about the extra parentheses on line 994 but not <br>
anything else for me.)<br>
<br>
My llwrap script is:<br>
<br>
CROSS_COMPILE=~/llvm/llvm- LDFLAGS+=" --static" CFLAGS+=" <br>
-Werror=format-security -Werror=unused-result" "$@"<br>
<br>
With the llvm directory being a symlink to <br>
android-ndk-r26d/toolchains/llvm/prebuilt/linux-x86_64/bin where I have <br>
added an "llvm-cc" file containing:<br>
<br>
#!/bin/bash<br>
<br>
"$(dirname "$0")"/clang ${LLTARGET:---target=x86_64-linux-android34} "$@"<br>
<br>
(Sigh, I need to reinstall the thunderbird extension that lets me <br>
disable wordwrap but they break compatability with old plugins on <br>
tuesdays...)<br>
<br>
Anyway, I'm not seeing that warning on android-ndk-r26d's llvm, do I <br>
need a newer one...? (I can apply just the parentheses removal but I <br>
didn't want to edit hunks out of your patch without asking.)<br>
<br>
Rob<br>
_______________________________________________<br>
Toybox mailing list<br>
<a href="mailto:Toybox@lists.landley.net" target="_blank">Toybox@lists.landley.net</a><br>
<a href="http://lists.landley.net/listinfo.cgi/toybox-landley.net" rel="noreferrer" target="_blank">http://lists.landley.net/listinfo.cgi/toybox-landley.net</a><br>
</blockquote></div>