Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | tests: Improve test coverage a bit | Tavian Barnes | 2020-11-10 | 13 | -0/+350 | |
| | ||||||
* | tests: Add missing expectations from dd3bbb9 | Tavian Barnes | 2020-11-09 | 2 | -0/+38 | |
| | ||||||
* | opt: Predicates aren't true when they're false | Tavian Barnes | 2020-11-09 | 2 | -2/+14 | |
| | | | | | | | | | | | | This unfortunate typo was mostly harmless; since the predicates were always assumed to be true, they wouldn't conflict. The exception is -user/-group, which set -nouser/-nogroup to false for users/groups that exist. Even -O0 wasn't enough to suppress the bug, due to a missing optlevel check fixed in the previous commit. Fixes: 305ee902874b49351f4916e303c293523f11570b | |||||
* | opt: Check optlevel before removing unreachable expressions | Tavian Barnes | 2020-11-09 | 1 | -2/+4 | |
| | ||||||
* | Use two newlines for all pre-eval warnings | Tavian Barnes | 2020-11-09 | 2 | -2/+2 | |
| | ||||||
* | eval: Fix the status bar to only print the parent again | Tavian Barnes | 2020-11-04 | 1 | -5/+7 | |
| | ||||||
* | eval: Make a crude attempt to handle double-width chars in the status bar | Tavian Barnes | 2020-11-04 | 1 | -11/+49 | |
| | ||||||
* | Enable -Wsign-compare to catch bugs like 726d7801 | Tavian Barnes | 2020-11-04 | 6 | -11/+11 | |
| | ||||||
* | spawn: Fix error pipe write failure handling | Tavian Barnes | 2020-11-04 | 1 | -1/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A short history lesson: this code (from exec.c) used to just be write(...); In 649d85b, an empty if block was added to placate _FORTIFY_SOURCE's warn_unused_result: if (write(...) != sizeof(error)) { // ... } This is fine since the parent doesn't rely on receiving those writes (e.g. the child could die before writing), and the small write to a pipe won't fail anyway. But when bfs_spawn() was created, the implementation changed to this: while (write(...) < sizeof(error)); The while loop was presumably added to mirror musl's posix_spawn() implementation, which handles EINTR. But musl's is actually correct: while (write(...) < 0); whereas mine has a subtle bug: since sizeof(error) is unsigned, the bfs code did an unsigned comparison, meaning -1 from write() would *not* restart the loop. Fix it by comparing with 0 like musl, since short writes are impossible here. Perhaps it's time for -Wsign-compare, despite the other 18 instances being false positives. | |||||
* | Makefile: Fail early on sanitizer errors | Tavian Barnes | 2020-11-03 | 1 | -0/+8 | |
| | ||||||
* | New -status option to display a status bar | Tavian Barnes | 2020-11-03 | 5 | -14/+122 | |
| | ||||||
* | bar: Implement terminal status bars | Tavian Barnes | 2020-11-03 | 3 | -0/+320 | |
| | ||||||
* | dstring: New dstrvprintf() function | Tavian Barnes | 2020-11-03 | 2 | -9/+28 | |
| | ||||||
* | Update the project URL | Tavian Barnes | 2020-10-14 | 2 | -2/+2 | |
| | ||||||
* | Release 2.02.0 | Tavian Barnes | 2020-10-14 | 3 | -3/+7 | |
| | ||||||
* | util: New BFS_FLEX_SIZEOF() macro for more precise flexible array allocations | Tavian Barnes | 2020-10-13 | 4 | -4/+28 | |
| | | | | | See http://www.open-std.org/jtc1/sc22/wg14/www/docs/dr_282.htm for all the fun behind this. | |||||
* | pwcache: Adjust some naming conventions | Tavian Barnes | 2020-10-06 | 3 | -14/+14 | |
| | ||||||
* | mtab: Adjust some naming conventions | Tavian Barnes | 2020-10-06 | 3 | -7/+7 | |
| | ||||||
* | expr, eval: Clean up header dependencies | Tavian Barnes | 2020-10-06 | 3 | -19/+23 | |
| | ||||||
* | exec: Adjust some calling conventions | Tavian Barnes | 2020-10-06 | 3 | -11/+11 | |
| | ||||||
* | printf: Adjust some calling conventions | Tavian Barnes | 2020-10-06 | 3 | -22/+22 | |
| | ||||||
* | ctx: Don't include color.h | Tavian Barnes | 2020-10-06 | 1 | -4/+4 | |
| | ||||||
* | diag: New bfs_perror() function | Tavian Barnes | 2020-10-05 | 6 | -37/+53 | |
| | ||||||
* | parse: Fail if -color is passed and the colors couldn't be parsed | Tavian Barnes | 2020-10-04 | 3 | -0/+13 | |
| | ||||||
* | parse: More accurate error reporting for cfdup() | Tavian Barnes | 2020-10-04 | 1 | -1/+7 | |
| | ||||||
* | parse: Report errors when failing to add a root | Tavian Barnes | 2020-10-04 | 1 | -1/+5 | |
| | ||||||
* | diag: Factor debug_flag string conversion into its own function | Tavian Barnes | 2020-10-04 | 1 | -25/+21 | |
| | ||||||
* | time: Don't call tzset() on every x{local,gm}time() | Tavian Barnes | 2020-10-01 | 1 | -2/+11 | |
| | | | | It turns out tzset() checks /etc/localtime every time you call it. | |||||
* | dstring: Try to avoid calling vsnprintf() twice in dstrprintf() | Tavian Barnes | 2020-10-01 | 1 | -10/+26 | |
| | ||||||
* | main: Preserve errno over close() in redirect() | Tavian Barnes | 2020-10-01 | 1 | -6/+7 | |
| | ||||||
* | util: Move redirect() and isopen() to main.c | Tavian Barnes | 2020-10-01 | 3 | -48/+22 | |
| | ||||||
* | util: Don't rely on bftw | Tavian Barnes | 2020-09-30 | 4 | -40/+39 | |
| | | | | And rename format_mode() to xstrmode() while I'm at it. | |||||
* | ctx: Perserve errno better in bfs_ctx_open() | Tavian Barnes | 2020-09-28 | 1 | -0/+7 | |
| | ||||||
* | Rename struct cmdline to bfs_ctx | Tavian Barnes | 2020-09-27 | 18 | -562/+786 | |
| | | | | | The API remains similar, with some added accessor functions for lazy initialization of the pwcache and mtab. | |||||
* | exec: Output a human-readable description of terminating signals | Tavian Barnes | 2020-09-23 | 1 | -1/+5 | |
| | ||||||
* | bftw: Fix bftw_cached_stat() with BFS_STAT_TRYFOLLOW | Tavian Barnes | 2020-09-20 | 1 | -1/+5 | |
| | ||||||
* | tests: Add missing ground truth | Tavian Barnes | 2020-09-20 | 1 | -0/+12 | |
| | ||||||
* | printf: Format the empty string for %l of non-links | Tavian Barnes | 2020-09-20 | 2 | -9/+15 | |
| | | | | It makes a difference if the format specifier has a width. | |||||
* | stat: Rename bfs_stat_flag to _flags | Tavian Barnes | 2020-09-18 | 7 | -23/+23 | |
| | | | | Flags enums should be plural. | |||||
* | Don't call stat() just to determine symbolic lengths | Tavian Barnes | 2020-09-18 | 5 | -11/+30 | |
| | | | | | | | The new bftw_cached_stat() helper gets us stat info if we already have it, but doesn't call stat() if we don't. In that case we just take a guess for the initial length to readlinkat(). This lets us avoid stat() entirely in many cases for -lname and -printf %l. | |||||
* | util: Make the initial allocation bigger for xreadlinkat() | Tavian Barnes | 2020-09-18 | 1 | -1/+6 | |
| | | | | | | | | | Most symlinks are more than 1 character long, so rather than wasting time with 1, 2, 4, ... when we don't have a good guess for the length, start with a bigger one. Credit to https://twitter.com/RichFelker/status/1306321019108556806 for reminding me of this. | |||||
* | pwcache: Fix an invalid free if strdup()ing the group name fails | Tavian Barnes | 2020-09-02 | 1 | -3/+4 | |
| | ||||||
* | pwcache: Work around gr_mem being misaligned on macOS | Tavian Barnes | 2020-08-13 | 1 | -3/+14 | |
| | | | | C.f. https://github.com/php/php-src/commit/d80f0ff6c0a557d8c993a9d2bd006fb488f6d564 | |||||
* | Implement -xattrname | Tavian Barnes | 2020-08-13 | 12 | -20/+159 | |
| | | | | From macOS find. | |||||
* | bftw: Make some flag names more explicit | Tavian Barnes | 2020-07-29 | 4 | -36/+36 | |
| | ||||||
* | bftw: Rename bftw_typeflag to bftw_type, and make it not a bitmask | Tavian Barnes | 2020-07-29 | 9 | -107/+106 | |
| | ||||||
* | Update default branch to 'main' | Tavian Barnes | 2020-07-10 | 1 | -2/+2 | |
| | ||||||
* | eval: Fix an assertion failure with -D search -s | Tavian Barnes | 2020-06-16 | 1 | -0/+1 | |
| | ||||||
* | eval: Fix a segfault with -D search -S eds | Tavian Barnes | 2020-06-16 | 1 | -0/+1 | |
| | ||||||
* | Implement exponential deepening search | Tavian Barnes | 2020-06-16 | 6 | -31/+114 | |
| |