Page MenuHomeFreeBSD

Rebuild sysent when capabilities.conf is updated.
ClosedPublic

Authored by markj on Jul 6 2020, 3:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 13, 6:24 AM
Unknown Object (File)
Thu, Oct 9, 6:54 AM
Unknown Object (File)
Wed, Oct 1, 6:24 PM
Unknown Object (File)
Sun, Sep 28, 9:30 PM
Unknown Object (File)
Sat, Sep 20, 3:23 PM
Unknown Object (File)
Sep 12 2025, 2:07 AM
Unknown Object (File)
Aug 31 2025, 2:06 PM
Unknown Object (File)
Aug 15 2025, 7:29 AM
Subscribers
None

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Jul 6 2020, 3:47 PM
markj created this revision.

Huh, I thought we already did this. Did I inadvertently drop it when I consolidated?

Huh, I thought we already did this. Did I inadvertently drop it when I consolidated?

No idea. I just noticed the problem when updating capabilities.conf, I can't remember if this worked in the past.

sys/conf/sysent.mk
28 ↗(On Diff #74118)

This will work, but does cause some harmless rebuilds (e.g. things like linux combat that don't use capabilities.conf). I wonder if this should be added in the individual Makefiles.

sys/conf/sysent.mk
28 ↗(On Diff #74118)

I thought about it and went with this approach since:

  • the likelihood of a spurious rebuild is low, and it's harmless anyway
  • the makefile would be duplicating metadata from syscalls.conf

I'm happy to change it if you prefer, though.

brooks added inline comments.
sys/conf/sysent.mk
28 ↗(On Diff #74118)

It might be worth commenting that it's a known (unimportant) issue.

This revision is now accepted and ready to land.Jul 6 2020, 5:20 PM
markj marked an inline comment as done.

Add a comment about spurious rebuilds.

This revision now requires review to proceed.Jul 6 2020, 6:35 PM
This revision is now accepted and ready to land.Jul 6 2020, 6:48 PM
This revision was automatically updated to reflect the committed changes.