src/bin/ls, branch mainFreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=main2026-02-11T19:56:37Zls: check fts_children() for errors that may not surface otherwise2026-02-11T19:56:37ZKyle Evanskevans@FreeBSD.org2026-02-11T19:55:55Zurn:sha1:7bf81e39d83087dc7f984077b5eed5a48df794d4
In particular, if one simply does a non-recursive `ls` on a directory
that is not accessible, there are some classes of errors that may cause
it to fail that wouldn't be surfaced unless we do an fts_read() that
will recurse into the inaccessible directory. Catch those kinds of
errors here since we cannot expect to an FTS_ERR/FTS_DNR entry to follow
up on them.
PR: 287451
Reviewed by: kib
Discusssed with: des
Differential Revision: https://reviews.freebsd.org/D51056
ls: --group-directories requires stat information2025-01-22T07:29:29ZPiotr Pawel Stefaniakpstef@FreeBSD.org2025-01-21T22:47:30Zurn:sha1:6ba2f08cea8cd5c801b78eb8842f7f96bf2d255c
Fixes: 8b92977857f8
ls.1: bump date after 82fa7f83b53b and 8b92977857f82025-01-16T16:05:43ZPiotr Pawel Stefaniakpstef@FreeBSD.org2025-01-16T16:05:43Zurn:sha1:4ed2448a4f2b32d0c4055c282e4cc2b691d621edls: implement --group-directories-first for compatibility with GNU ls2025-01-16T16:00:17ZPiotr Pawel Stefaniakpstef@FreeBSD.org2025-01-06T21:26:14Zurn:sha1:8b92977857f8acaa539c21800b024edea9eee8b4
Also implement --group-directories which takes a parameter.
"first" is equivalent to --group-directories-first, "last" gives
reversed sorting.
Changes in sorting between elements of the same type (files,
directories) are not intended.
Differential Revision: https://reviews.freebsd.org/D48347
ls -lh: humanize the total2025-01-16T16:00:17ZPiotr Pawel Stefaniakpstef@FreeBSD.org2025-01-06T17:59:03Zurn:sha1:82fa7f83b53b3c5b7f3a5c5560edc7caeeaa1029
Before this change, the total printed on the first line was always in
blocks.
Now the long format with -h will print the "humanized number" instead.
`ls -sh` never provided sizes in anything other than blocks, total or
individual, and this commit doesn't change that.
The total number of blocks is a sum of fts_statp->st_blocks, so it's
multiplied by 512 and not by blocksize.
$ ls -lh /usr/bin | head -n 2
total 442 MB
-r-xr-xr-x 1 root wheel 70B Dec 20 21:06 CC
Differential Revision: https://reviews.freebsd.org/D48329
ls: Release resources before returning from traverse()2025-01-14T14:20:26ZMark Johnstonmarkj@FreeBSD.org2025-01-14T14:20:26Zurn:sha1:e6c9c463384d11a44af3e5f2cc947fb69f3a1968
PR: 278476
MFC after: 2 weeks
Reported by: valgrind
ls: Make -, apply to -s as well as -l.2024-07-24T20:07:06ZDag-Erling Smørgravdes@FreeBSD.org2024-07-24T20:06:39Zurn:sha1:647d4a8cafd2c9b291cab388191bc7fcfe12a66b
While here, remove a bogus comment about a gcc bug. The bug was in ls,
which used an incorrect format string, and in libc, which accepted it.
MFC after: 1 week
Reviewed by: brooks
Differential Revision: https://reviews.freebsd.org/D46067
Remove residual blank line at start of Makefile2024-07-15T22:43:39ZWarner Loshimp@FreeBSD.org2024-07-15T04:46:32Zurn:sha1:e9ac41698b2f322d55ccf9da50a3596edb2c1800
This is a residual of the $FreeBSD$ removal.
MFC After: 3 days (though I'll just run the command on the branches)
Sponsored by: Netflix
ls.1: Mention CLICOLOR environment variable2024-02-21T10:54:58ZGordon Berglinggbe@FreeBSD.org2024-02-21T10:54:58Zurn:sha1:ffea732c5192070a6877409428b0a2cc1728aa16
Mention CLICOLOR environment variable in the manual page.
PR: 276556
Submitted by: bsdcode at disroot dot org
Reviewed by: imp
MFC after: 3 days
Differential Revision: https://reviews.freebsd.org/D43970
ls: Fix -v and associated test2024-02-19T16:50:04ZWarner Loshimp@FreeBSD.org2024-02-19T16:48:05Zurn:sha1:ef75877fc2d93199aab2b509089136c433af1f20
The cleanup of d854370fa86b7 had a cut and paste error (so f_verssort
was set to 1 and then to 0 rather thame f_timesort being set to 0).
Fixes: d854370fa86b7
Sponsored by: Netflix