Page MenuHomeFreeBSD
Differential D43163

Update exports.5 man page for handling of special characters in directory names
ClosedPublic
Actions

Authored by rmacklem on Dec 22 2023, 12:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 12:50 PM
Unknown Object (File)
Sat, Apr 27, 12:10 PM
Unknown Object (File)
Tue, Apr 9, 4:44 AM
Unknown Object (File)
Mon, Apr 8, 8:50 PM
Unknown Object (File)
Mar 21 2024, 7:03 PM
Unknown Object (File)
Feb 23 2024, 7:20 PM
Unknown Object (File)
Dec 27 2023, 12:34 AM
Subscribers
imp
pauamma_gundo.com

Details

Reviewers
dan.mcgregor_usask.ca
karels
pauamma_gundo.com
Group Reviewers
manpages
Summary

Commit 7c5146da1286 modified mountd so that it uses
strunvis(3) to decode directory names in exports lines.
This allows special characters, such as blanks, to be
encoded in the directory names.

This patch updates the exports.5 man page for this change.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

rmacklem created this revision.Dec 22 2023, 12:04 AM
Herald added a subscriber: imp. · View Herald TranscriptDec 22 2023, 12:04 AM
rmacklem requested review of this revision.Dec 22 2023, 12:04 AM
karels added inline comments.Dec 22 2023, 12:09 AM
usr.sbin/mountd/exports.5
112

I hadn't heard of "imbedded"; apparently "embedded" is much more common.

pauamma_gundo.com accepted this revision.Dec 22 2023, 1:42 AM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.

Approved with s/imbedded/embedded/ throughout.

This revision is now accepted and ready to land.Dec 22 2023, 1:42 AM
rmacklem marked an inline comment as done.Dec 22 2023, 1:45 AM
rmacklem added inline comments.
usr.sbin/mountd/exports.5
112

Yep, I'll change it. google (not surprisingly) gave
multiple responses for "imbed vs embed". Most said
imbed was a less common spelling, but a couple
stated that imbed was a past tense version.

Anyhow, it seems embedded works for everyone,
so I will go with that.

rmacklem updated this revision to Diff 131714.Dec 22 2023, 2:02 AM
rmacklem marked an inline comment as done.

Changed imbedded to embedded as suggested by others.
I also changed a "for" to a "to", which was just a typo.

This revision now requires review to proceed.Dec 22 2023, 2:02 AM
karels accepted this revision.Dec 22 2023, 2:08 AM
This revision is now accepted and ready to land.Dec 22 2023, 2:08 AM
rmacklem closed this revision.Dec 22 2023, 8:40 PM

Committed as 922e78f65fd9.

Revision Contents
Changeset List

PathSize
usr.sbin/
mountd/
exports.5
exports.5.sav
15 lines

Diff 131714

View Options

usr.sbin/mountd/exports.5

Loading...