Page MenuHomeFreeBSD
Differential D49952

Search for mntopts.h globally, not locally
ClosedPublic
Actions

Authored by brooks on Apr 21 2025, 10:18 PM.
Tags
None
Referenced Files
F132476910: D49952.id154057.diff
Fri, Oct 17, 6:15 AM
Unknown Object (File)
Fri, Oct 3, 12:00 PM
Unknown Object (File)
Tue, Sep 23, 8:30 PM
Unknown Object (File)
Sat, Sep 20, 4:34 AM
Unknown Object (File)
Sep 17 2025, 1:28 PM
Unknown Object (File)
Sep 16 2025, 11:56 AM
Unknown Object (File)
Aug 30 2025, 8:13 AM
Unknown Object (File)
Aug 22 2025, 6:19 PM
View All 55 Files
Subscribers
asomers
dim
guest-patmaddox

Details

Reviewers
mckusick
jrtc27
imp
olce
emaste
dim
Commits
rGa123502ef781: Search for mntopts.h globally, not locally
Summary

Change the include directives to use <mntopts.h> instead of "mntopts.h"
now that it's installed by libutil (the latter option was dubious
regardless since a -I${SRCTOP}/sbin/mount was required anyway).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

brooks created this revision.Apr 21 2025, 10:18 PM
Herald added a reviewer: olce. · View Herald TranscriptApr 21 2025, 10:18 PM
Herald added a subscriber: asomers. · View Herald Transcript
brooks requested review of this revision.Apr 21 2025, 10:18 PM
Harbormaster completed remote builds in B63621: Diff 154057.Apr 21 2025, 10:18 PM
brooks added a parent revision: D49951: Move mntopts(3) suppport into libutil.Apr 21 2025, 10:18 PM
imp accepted this revision.Apr 21 2025, 10:30 PM
This revision is now accepted and ready to land.Apr 21 2025, 10:30 PM
brooks mentioned this in D49958: Move mount related functions into libmount.Apr 22 2025, 3:47 PM
emaste accepted this revision.Apr 22 2025, 4:42 PM
dim accepted this revision.Apr 22 2025, 5:28 PM
dim added a subscriber: dim.

LGTM, as long as there are include guards in mntopts.h :)

guest-patmaddox added a subscriber: guest-patmaddox.Apr 22 2025, 6:05 PM
olce accepted this revision.Apr 22 2025, 7:29 PM
Closed by commit rGa123502ef781: Search for mntopts.h globally, not locally (authored by brooks). · Explain WhyApr 22 2025, 9:52 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rGa123502ef781: Search for mntopts.h globally, not locally.

Revision Contents
Changeset List

PathSize
contrib/
smbfs/
mount_smbfs/
3 lines
lib/
libutil/
3 lines
sbin/
init/
2 lines
mount/
2 lines
2 lines
mount_cd9660/
3 lines
mount_fusefs/
3 lines
mount_msdosfs/
3 lines
mount_nfs/
2 lines
mount_nullfs/
3 lines
mount_udf/
3 lines
mount_unionfs/
3 lines
tests/
sys/
fs/
fusefs/
3 lines
3 lines
2 lines
usr.sbin/
autofs/
2 lines
mountd/
2 lines

Diff 154125

View Options

contrib/smbfs/mount_smbfs/mount_smbfs.c

Loading...
View Options

lib/libutil/mntopts.c

Loading...
View Options

sbin/init/init.c

Loading...
View Options

sbin/mount/mount.c

Loading...
View Options

sbin/mount/mount_fs.c

Loading...
View Options

sbin/mount_cd9660/mount_cd9660.c

Loading...
View Options

sbin/mount_fusefs/mount_fusefs.c

Loading...
View Options

sbin/mount_msdosfs/mount_msdosfs.c

Loading...
View Options

sbin/mount_nfs/mount_nfs.c

Loading...
View Options

sbin/mount_nullfs/mount_nullfs.c

Loading...
View Options

sbin/mount_udf/mount_udf.c

Loading...
View Options

sbin/mount_unionfs/mount_unionfs.c

Loading...
View Options

tests/sys/fs/fusefs/fallocate.cc

Loading...
View Options

tests/sys/fs/fusefs/mockfs.cc

Loading...
View Options

tests/sys/fs/fusefs/mount.cc

Loading...
View Options

usr.sbin/autofs/automount.c

Loading...
View Options

usr.sbin/mountd/mountd.c

Loading...