Page MenuHomeFreeBSD
Differential D13959

Convert ls(1) to not use libxo(3).
ClosedPublic
Actions

Authored by cem on Jan 17 2018, 8:52 PM.
Tags
None
Referenced Files
F147257704: D13959.diff
Mon, Mar 9, 12:16 PM
Unknown Object (File)
Tue, Mar 3, 4:13 AM
Unknown Object (File)
Mon, Mar 2, 6:57 AM
Unknown Object (File)
Mon, Mar 2, 5:40 AM
Unknown Object (File)
Sun, Mar 1, 12:25 PM
Unknown Object (File)
Tue, Feb 17, 6:53 AM
Unknown Object (File)
Dec 14 2025, 8:36 AM
Unknown Object (File)
Dec 14 2025, 8:36 AM
View All Files
Subscribers
None

Details

Reviewers
manu
imp
kmacy
Group Reviewers
manpages
Commits
rS328100: Convert ls(1) to not use libxo(3)
Summary

libxo imposes a large burden on system utilities. In the case of ls, that
burden is difficult to justify -- any language that can interact with json
output can use readdir(3) and stat(2).

Logically revert r291607, r285857, r285803, r285734, r285425, r284494, r284489,
r284252, and r284198.

Test Plan
$ id
uid=0(root) gid=0(wheel) groups=0(wheel),5(operator)
$ kyua test
...
35/35 passed (0 failed)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14447
Build 14594: arc lint + arc unit

Event Timeline

cem created this revision.Jan 17 2018, 8:52 PM
Herald added a reviewer: manpages. · View Herald TranscriptJan 17 2018, 8:52 PM
Harbormaster completed remote builds in B14447: Diff 38109.Jan 17 2018, 8:52 PM
cem added a reviewer: kmacy.Jan 17 2018, 9:12 PM
imp accepted this revision.Jan 17 2018, 10:34 PM

My only regret is that phab won't let me click accept twice.

This revision is now accepted and ready to land.Jan 17 2018, 10:34 PM
Closed by commit rS328100: Convert ls(1) to not use libxo(3) (authored by cem). · Explain WhyJan 17 2018, 10:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
bin/
ls/
2 lines
1 line
11 lines
3 lines
12 lines
57 lines
205 lines
52 lines

Diff 38109

View Options

bin/ls/Makefile

Loading...
View Options

bin/ls/Makefile.depend

Loading...
View Options

bin/ls/extern.h

Loading...
View Options

bin/ls/ls.h

Loading...
View Options

bin/ls/ls.1

Loading...
View Options

bin/ls/ls.c

Loading...
View Options

bin/ls/print.c

Loading...
View Options

bin/ls/util.c

Loading...