Page MenuHomeFreeBSD

sys/mount.h: Add missing headers so it's standalone for _WANT_MOUNT
ClosedPublic

Authored by jrtc27 on Fri, Jun 6, 2:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 1, 2:50 PM
Unknown Object (File)
Tue, Jul 1, 2:50 PM
Unknown Object (File)
Tue, Jul 1, 2:50 PM
Unknown Object (File)
Tue, Jul 1, 2:50 PM
Unknown Object (File)
Mon, Jun 30, 6:26 PM
Unknown Object (File)
Sun, Jun 29, 1:33 PM
Unknown Object (File)
Sun, Jun 29, 12:19 PM
Unknown Object (File)
Sun, Jun 29, 10:36 AM
Subscribers
None

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 64673
Build 61557: arc lint + arc unit

Event Timeline

jrtc27 requested review of this revision.Fri, Jun 6, 2:12 PM
jrtc27 created this revision.
This revision is now accepted and ready to land.Fri, Jun 6, 2:46 PM

You need at least sys/types.h there as well.

In D50716#1158047, @kib wrote:

You need at least sys/types.h there as well.

Hm, isn't it more that there's an existing issue of sys/types.h not being explicitly included for normal userspace includes? All kinds of userspace-visible types are using uintN_t for example here. It just works because sys/ucred.h includes sys/types.h (and perhaps via other include chains too). I can definitely fix that, but I feel that should be a separate (precursor?) commit.

In D50716#1158047, @kib wrote:

You need at least sys/types.h there as well.

Hm, isn't it more that there's an existing issue of sys/types.h not being explicitly included for normal userspace includes? All kinds of userspace-visible types are using uintN_t for example here. It just works because sys/ucred.h includes sys/types.h (and perhaps via other include chains too). I can definitely fix that, but I feel that should be a separate (precursor?) commit.

I have no objections about sys/types.h being a preparational commit.