<div dir="ltr"><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">I was grinding through my todo list last night and fixed this, and only<br>
just noticed replying to your message that you'd attached a patch.<br>
(Oops. Sorry.)</blockquote><div><br></div><div>:-|, well at least it's fixed.<br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">The fixes look fairly similar. I need to update tests/chown.test and<br></blockquote><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">
tests/chgrp.test to actually check for this usefully. (Poking at that now.) <br></blockquote><div><br></div><div>Why return `unsigned` instead of `uid_t`/`gid_t` from the get uid/gid methods? <br><br></div><div>By the way, I have another patch for the test applet:<br><a href="https://github.com/landley/toybox/pull/47" target="_blank">https://github.com/landley/<wbr>toybox/pull/47</a><br><br></div><div>~Matthias<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-19 23:56 GMT+02:00 Rob Landley <span dir="ltr"><<a href="mailto:rob@landley.net" target="_blank">rob@landley.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 08/14/2016 12:54 PM, darken wrote:<br>
> The issue is that chown/chgrp would error exit if it was passed an<br>
> unknown numeric uid/gid.<br>
<br>
</span>I was grinding through my todo list last night and fixed this, and only<br>
just noticed replying to your message that you'd attached a patch.<br>
(Oops. Sorry.)<br>
<br>
The fixes look fairly similar. I need to update tests/chown.test and<br>
tests/chgrp.test to actually check for this usefully. (Poking at that now.)<br>
<br>
Thanks,<br>
<br>
Rob<br>
</blockquote></div><br></div></div>