summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Support for FreeBSD-style ls ColoringVinX39 hours1-5/+76
| | | | | | | | If LSCOLORS is undefined or partially defined (missing some of the eleven expected value pairs), the patch will automatically fill in those missing values with default colors. However, if the LS_COLORS environment variable is already defined, the application prefer the LS_COLORS approach, maintaining current working flow of the program.
* opt: "Not me" doesn't imply "nobody"Tavian Barnes2025-06-201-7/+12
| | | | | | | | | | | | | We were making `-user <existing>` imply `! -nouser`, which is valid, but we were also makeing `! -user <existing>` imply `-nouser`, which isn't. Fix it by only constraining the `-nouser`/`-nogroup` predicates in the true case. While I'm here, fix a similar latent bug that would have triggered if we ever merged `-readable -and -writable` into one `R_OK | W_OK` test. Fixes: https://github.com/tavianator/bfs/issues/155 Fixes: 305ee902 ("opt: Track data flow information about predicates")
* xspawn: Fix a -Wshadow warning on macOSTavian Barnes2025-06-161-2/+2
|
* xspawn: Work around a macOS posix_spawn() bugTavian Barnes2025-06-151-0/+11
| | | | | The XNU posix_spawn() implementation apparently can't handle fchdir() to a directory opened by the file actions themselves.
* xspawn: Work around a NetBSD posix_spawn() bugTavian Barnes2025-06-151-0/+11
| | | | | | | NetBSD's posix_spawn() surprisingly has the same bug as its posix_spawnp(): the executable is resolved before the file actions. Detect this case and work around it by falling back to fork()/exec() if we need to.
* parse: Allow relative $PATH entries with explicit executable pathsTavian Barnes2025-06-151-21/+48
| | | | | | $ bfs -execdir /bin/exe {} \; is perfectly safe regardless of what's in $PATH.
* xspawn: Don't check xfaccess(..., X_OK) before fchdir()Tavian Barnes2025-06-151-16/+40
| | | | Fixes: 8c130ca ("xspawn: Check X_OK even without $PATH resolution")
* Add support for __attribute__((counted_by(...)))Tavian Barnes2025-05-267-8/+17
|
* tests/posix: Add a test for POSIX bug 1859Tavian Barnes2025-05-261-1/+2
| | | | ... which requires we process root paths strictly in-order.
* bfstd: Add some more integer parsing functionsTavian Barnes2025-04-182-0/+124
|
* bfstd: Add a tcsetwinsize() polyfillTavian Barnes2025-04-152-0/+13
|
* bfstd: Add a tcgetwinsize() polyfillTavian Barnes2025-04-153-34/+40
|
* ctx: Don't leak ctx->kindsTavian Barnes2025-04-031-0/+1
| | | | Fixes: 9c911d7 ("ctx: Track the token kind of each argument")
* parse: Disallow -files0-from with explicit rootsTavian Barnes2025-04-011-0/+18
| | | | This matches the behaviour of GNU find.
* ctx: Track the token kind of each argumentTavian Barnes2025-04-013-0/+22
|
* parse: Only process the last -files0-fromTavian Barnes2025-04-011-54/+68
| | | | | | | | GNU find intentionally makes later -files0-from options override earlier ones, for symmetry with similar features like du --files0-from. Change bfs to match. Link: https://savannah.gnu.org/bugs/?66965
* color: New %p[xX] formats for expressions in diagnosticsTavian Barnes2025-04-014-36/+66
|
* parse: Track expressions, not just strings, for conflict reportingTavian Barnes2025-03-301-59/+85
|
* list: Switch back to the memcpy()/memset() SLIST_REMOVE() implementationTavian Barnes2025-03-291-19/+12
| | | | | | | | | The thread-local scratch variables make it non-reentrant for no good reason. I don't consider the theoretical strict-aliasing violation to be practically relevant. This partially reverts commit 90791fc ("list: Make SLIST_REMOVE() more type-safe").
* bfstd: Fix nproc() on systems without dynamically sized CPU masksTavian Barnes2025-03-101-2/+14
| | | | Notes: Fixes: a36774b ("bfstd: Take sched_getaffinity() into account in nproc()")
* diag: Don't leave unused assertion messages in the binaryTavian Barnes2025-03-041-6/+19
| | | | | Rather than hiding them with %.0s, use a ternary to replace them with an empty string if they would be unused.
* diag: Get rid of struct bfs_locationTavian Barnes2025-03-042-51/+40
| | | | | | | | Just add the standard prefix to the passed format string in the diagnostic macros themselves. This lets us write the whole message with one dprintf() call, minimizing interleaving. It's also a net win for binary size.
* bfstd: Take sched_getaffinity() into account in nproc()Tavian Barnes2025-02-271-5/+48
|
* bfstd: New nproc() functionTavian Barnes2025-02-273-15/+21
|
* Micro-optimize word-at-a-time loop tailsTavian Barnes2025-02-212-10/+14
| | | | | | Compilers apparently aren't smart enough to infer that the odd tail length fixup "loops" run at most once, and could be converted to ifs, so do that manually.
* ioq: Fix typo in commentTavian Barnes2025-02-181-1/+1
|
* trie: Use load8_beu*() for trie_mismatch()Tavian Barnes2025-02-131-29/+23
|
* bfstd: Use load8_leu*() for asciinlen()Tavian Barnes2025-02-131-24/+20
|
* bit: Add the load8_[bl]euN() utilitiesTavian Barnes2025-02-131-1/+53
|
* trie: Make nibble indices big-endianTavian Barnes2025-02-131-7/+7
| | | | Otherwise the order doesn't match lexicographical order on bytes.
* trie: New trie_node_size() helperTavian Barnes2025-02-131-3/+9
|
* trie: Clean up some bounds checkingTavian Barnes2025-02-121-6/+6
|
* trie: Micro-optimize trie_representative()Tavian Barnes2025-02-111-4/+5
| | | | | popcount(map & (bit - 1) & mask) has a longer critical path than popcount(map & (bit - 1)) & mask.
* eval: Don't error out in -ls if the time overflowsTavian Barnes2025-02-071-19/+30
|
* ioq: Use getppid() for the heavy NOP syscallTavian Barnes2025-02-061-1/+1
| | | | | | | | getpid() could hypothetically be cached (and glibc used to do it). getppid() can't be, because the parent could die and reparent the child to init. Link: https://lore.kernel.org/linux-mm/Z58dIOGlJ3grsAge@casper.infradead.org/
* mtab: Use mount IDs instead of dev_t for bfs_fstype()Tavian Barnes2025-02-061-2/+2
| | | | | | | | | This fixes -fstype on btrfs subvolumes, which previously reported "unknown" due to their dev_t being potentially different from the mount point. Link: https://savannah.gnu.org/bugs/?50859 Link: https://lwn.net/Articles/866582/
* stat: Expose mount IDs from statx()Tavian Barnes2025-02-063-2/+40
|
* trie: New trie_{get,set}_{str,mem}() functionsTavian Barnes2025-02-064-23/+93
|
* diag: Try to make diagnostics signal-saferTavian Barnes2025-02-061-3/+16
| | | | Link: https://sourceware.org/bugzilla/show_bug.cgi?id=16060
* ioq: Rewrite the spin loop to avoid a warningTavian Barnes2025-02-031-4/+3
| | | | | | | | | | | | With some GCC versions, --enable-ubsan leads to this warning: src/ioq.c: In function ‘ioq_slot_wait’: src/ioq.c:287:17: warning: ignoring loop annotation 287 | for (int j = 0; j < (1 << i); ++j) { | ^~~ presumably due to UBSan rewriting the shift to check for overflow. Work around this by precomputing the iteration count.
* sighook: Fix sigreset() error handlingTavian Barnes2025-01-201-9/+16
| | | | Notes: Fixes: 62928dd ("sighook: New sigreset() function to reset all handlers")
* sighook: Don't forget to check `initialized` in sigreset()Tavian Barnes2025-01-181-0/+4
|
* sighook: Fix up siginit() error pathsTavian Barnes2025-01-181-8/+14
|
* xspawn: Unblock signals between fork() and exec()Tavian Barnes2025-01-181-2/+15
| | | | | | Otherwise the child process runs with all signals blocked. Fixes: 423cfa4 ("xspawn: Mask signals before fork()")
* sighook: New sigreset() function to reset all handlersTavian Barnes2025-01-182-3/+45
|
* sighook: Convert siglist to a more general rcu_list typeTavian Barnes2025-01-181-52/+66
|
* build: Feature-detect #pragma nounroll supportTavian Barnes2025-01-171-2/+2
|
* list: Don't use leading underscores for globalsTavian Barnes2025-01-161-6/+6
| | | | Notes: Fixes: 90791fc ("list: Make SLIST_REMOVE() more type-safe")
* ioq: Don't unroll the spin loopTavian Barnes2025-01-162-1/+15
|
* ioq: Prefetch slots with a write intentTavian Barnes2025-01-161-1/+1
|
generated by cgit v1.2.3 (git 2.39.1) at 2025-07-05 03:17:56 +0000