forks/toybox/toys, branch 0.8.2

ls: Ensure file names are separated by 2 spaces

2019-10-17T21:03:18+00:00

We need two spaces between filenames because that is the convention followed by other implementations. More importantly, if we do not have two spaces, certain Unicode file names cause filenames to run together. In Unicode, combining characters come before the character they modify. If a filename ends in a combining character, the combining character attaches to the space that follows it, causing the space not to be visible. Having a two-space gap stops the above issue from happening. For context and a bit more information, see mailing list link below. https://www.mail-archive.com/toybox@lists.landley.net/msg05986.html

In-passing cleanup and add a NOP -P.

2019-10-16T01:00:51+00:00

Make cmp work with 1 argument (implicit - as second argument).

2019-10-14T21:13:01+00:00

losetup: Fix null-termination of src string instead of dest after copy

2019-10-14T20:28:28+00:00

The function loopback_setup(), after copying the loopback device name with xstrncpy(), ensures the null-termination of the string by forcing its last byte to 0. Unfortunately, this operation: - was probably intended to null-terminate dest instead; - does not affect the program execution because src is free()d right after; - if the size of src is smaller than the offset of the written zero, it modifies an unknown byte in the heap. Drop the null-termination line to fix the issue: xstrcpy() automatically null-terminates dest, or fails if the size of src is bigger than the the requested number of bytes to copy. Signed-off-by: Alessio Balsini

Don't need the second variable, and put the comment back.

2019-10-12T22:24:26+00:00

Fix argless find segfault.

2019-10-12T22:23:30+00:00

Inline array becomes garbage outside the if.

hostname: fix error reporting.

2019-10-12T22:15:54+00:00

The functions have their own errno :-(

xargs: various fixes.

2019-10-10T02:24:15+00:00

Don't run the command with no arguments if we run out of input but have already run the command at least once. The implementation of "run the command at least once (unless -r was supplied)" wasn't taking into account whether or not this was our first time round the loop. Fix the exit value, and the -- already documented but not implemented -- behavior if a child exits with status 255. Also extend the tests to cover these cases, plus cases I broke while coming up with the fix. Add more tests to convince myself that we've correctly interpreted how -s is supposed to behave, and fix the corner cases at the bottom end of the range. This fixes some issues we were seeing trying to build the Android SDK for (and more importantly, on) macOS.

ls: fix seqfault on broken locale

2019-10-10T02:18:30+00:00

When user builds toybox CFG_TOYBOX_I18N disabled and tries to list folder contents with multibyte characters other than UTF-8 ls might seqfault since wcrtomb returns -1 while locale set to fi_FI-UTF-8 disable CFG_TOYBOX_I18N touch őőőőaaőő ls

vi: fixes

2019-10-10T02:17:53+00:00

fix: force redraw after :set (no)list fix: force redraw after insert fix: split on zero cursor position fix: yank and push with utf-8 content