Page MenuHomeFreeBSD

include: put includes into -dev packages
ClosedPublic

Authored by kevans on Nov 3 2022, 7:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 31, 4:57 AM
Unknown Object (File)
Sun, Mar 29, 9:21 AM
Unknown Object (File)
Sun, Mar 29, 12:54 AM
Unknown Object (File)
Sat, Mar 28, 12:18 PM
Unknown Object (File)
Tue, Mar 24, 6:20 AM
Unknown Object (File)
Tue, Mar 24, 4:40 AM
Unknown Object (File)
Sun, Mar 22, 4:52 AM
Unknown Object (File)
Sun, Mar 22, 4:38 AM
Subscribers

Details

Summary

The includes build is kind of funky, as we support either copying or
symlinking files into /usr/include. For copies, we were supplying
the include/ ${TAG_ARGS}, which puts packages into FreeBSD-runtime,
without any consideration to the fact that we're installing headers.

Let's copy the approach that the symlinks target uses for now, and
add ",dev" to the TAG_ARGS so that headers at least end up in
FreeBSD-runtime-dev, which is more appropriate. Some of these includes
are actually technically supposed to be in *other* packages and their
INCSGROUP's PACKAGE setting is actually correct, but this is less
trivial to solve. This is a bandaid to fix the immediate problem of
some headers ending up in two different packages.

PR: 267526

Diff Detail

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

Event Timeline

kevans requested review of this revision.Nov 3 2022, 7:29 PM

I think that we need a FreeBSD-kernel-headers and put everyone of those includes in this package tbh.
But if it's easier to fix the if_wg problem for now I'm fine with this as a bandaid solution (haven't had time to update my tree past wg yet).

This revision is now accepted and ready to land.Nov 3 2022, 7:56 PM
dfr added a subscriber: dfr.

Looks good to me.

This revision was automatically updated to reflect the committed changes.