<div dir="ltr"><div>Thanks for the quick reply and updates. Works great now!</div><div><br></div><div>Luke</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 25, 2020 at 3:53 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 11/24/20 5:06 PM, Luke Frankart wrote:<br>
> I found that `toybox test -w` returned true for a file with 0440 mode bits, and<br>
> was able to find failures for other cases (below). <br>
> <br>
> I think the root cause and fix is to use octal 0111 instead of hexadecimal 0x111<br>
> on this line:<br>
> <a href="https://github.com/landley/toybox/blob/master/toys/posix/test.c#L74" rel="noreferrer" target="_blank">https://github.com/landley/toybox/blob/master/toys/posix/test.c#L74</a> <br>
<br>
Oops. Reflex. I should add a test for this...<br>
<br>
And "make test_test" fails because it complains there's no trailing ']' which it<br>
should only do when called as '[', and that's because a standalone build of test<br>
doesn't include the command multiplexer so it doesn't dispatch on the filename,<br>
instead it hardwired calls the first entry in toy_list which is sorted and '['<br>
comes before 't'. The fix is to USE_SH() instead of USE_TEST() around the<br>
OLDTOY() since it's a nofork so the [ version can _only_ be called from the<br>
shell (unlike busybox which installs it in the filesystem for some reason).<br>
<br>
Ah, and the _other_ reason tests/test.test hasn't got chmod +x on it (so it gets<br>
called by "make tests", which is plural because "make test" builds _this_<br>
command standalone.) is that the second half of the test file is a bunch of<br>
comments of tests I should add to it.<br>
<br>
Specifically, THIS one falls under:<br>
<br>
# TODO: Test rwx gu t<br>
<br>
And it's not -t (that's is a TTY), it's -k to test the sticky bit, which I don't<br>
seem to have implemented?<br>
<br>
Ok, try now?<br>
<br>
Thanks,<br>
<br>
Rob</blockquote></div></div>