Page MenuHomeFreeBSD
Differential D53948

bsd.lib.mk: document INTERNALLIB and PRIVATELIB
ClosedPublic
Actions

Authored by kib on Nov 27 2025, 4:44 PM.
Tags
None
Referenced Files
F154150041: D53948.id167204.diff
Sun, Apr 26, 2:52 PM
Unknown Object (File)
Sat, Apr 25, 1:44 AM
Unknown Object (File)
Mon, Apr 20, 7:09 PM
Unknown Object (File)
Mon, Apr 20, 2:01 PM
Unknown Object (File)
Fri, Apr 17, 3:40 AM
Unknown Object (File)
Wed, Apr 15, 11:28 AM
Unknown Object (File)
Wed, Apr 15, 11:25 AM
Unknown Object (File)
Sat, Apr 11, 3:31 PM
View All 47 Files
Subscribers
bdrewery
imp

Details

Reviewers
des
emaste
dim
Commits
rGe60861eb41ca: bsd.lib.mk: document INTERNALLIB and PRIVATELIB

Diff Detail

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

Event Timeline

kib created this revision.Nov 27 2025, 4:44 PM
Herald added subscribers: imp, bdrewery. · View Herald TranscriptNov 27 2025, 4:44 PM
kib requested review of this revision.Nov 27 2025, 4:44 PM
emaste accepted this revision.Nov 27 2025, 5:02 PM

All the variables in here would benefit from documentation but we can start with these

This revision is now accepted and ready to land.Nov 27 2025, 5:02 PM
dim accepted this revision.Nov 27 2025, 5:57 PM

LGTM with one nit.

share/mk/bsd.lib.mk
5

For consistency, either add "we" before "build", or remove it in the INTERNALLIB entry.

des requested changes to this revision.Nov 27 2025, 6:06 PM
des added inline comments.
share/mk/bsd.lib.mk
2
8

This is incorrect, as I told you on IRC. Headers for private libraries are installed in /usr/include/private.

This revision now requires changes to proceed.Nov 27 2025, 6:06 PM
kib updated this revision to Diff 167203.Nov 27 2025, 6:42 PM
kib marked 3 inline comments as done.

INTERNALLIB: s/pic/pie/; only state that the result is suitable for linking into binaries. This sounds like a bug to me: we should build pic version instead of pie.
PRIVATELIB: headers go into include/private/<name>

des accepted this revision.Nov 27 2025, 6:47 PM
This revision is now accepted and ready to land.Nov 27 2025, 6:47 PM
kib updated this revision to Diff 167204.Nov 27 2025, 6:48 PM

Slightly more clarity for normal libs.

This revision now requires review to proceed.Nov 27 2025, 6:48 PM
emaste accepted this revision.Nov 27 2025, 6:51 PM
This revision is now accepted and ready to land.Nov 27 2025, 6:51 PM
Closed by commit rGe60861eb41ca: bsd.lib.mk: document INTERNALLIB and PRIVATELIB (authored by kib). · Explain WhyNov 27 2025, 7:53 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rGe60861eb41ca: bsd.lib.mk: document INTERNALLIB and PRIVATELIB.

Revision Contents
Changeset List

PathSize
share/
mk/
15 lines

Diff 167209

View Options

share/mk/bsd.lib.mk

Loading...