Page MenuHomeFreeBSD

include: put includes into -dev packages
ClosedPublic

Authored by kevans on Nov 3 2022, 7:29 PM.
Tags
None
Referenced Files
F110645762: D37256.id.diff
Fri, Feb 21, 9:41 AM
F110641077: D37256.id.diff
Fri, Feb 21, 8:06 AM
Unknown Object (File)
Thu, Feb 20, 10:59 AM
Unknown Object (File)
Sat, Feb 15, 2:20 PM
Unknown Object (File)
Thu, Feb 13, 9:33 AM
Unknown Object (File)
Thu, Feb 13, 2:59 AM
Unknown Object (File)
Thu, Jan 30, 9:26 AM
Unknown Object (File)
Jan 6 2025, 7:39 PM
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 Not Applicable
Unit
Tests Not Applicable

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.