Page MenuHomeFreeBSD

Retire MK_PROFILE infrastructure
ClosedPublic

Authored by emaste on Aug 16 2021, 3:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 12:11 AM
Unknown Object (File)
Dec 6 2024, 4:08 PM
Unknown Object (File)
Nov 21 2024, 10:19 PM
Unknown Object (File)
Nov 19 2024, 8:31 PM
Unknown Object (File)
Nov 19 2024, 6:51 PM
Unknown Object (File)
Nov 19 2024, 6:51 PM
Unknown Object (File)
Nov 19 2024, 6:21 PM
Unknown Object (File)
Nov 14 2024, 8:22 PM

Details

Summary
We announced that the option would be removed before FreeBSD 14; it was
first disabled by default in fe52b7f60ef4.

Relnotes:       Yes
Sponsored by:   The FreeBSD Foundation

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste added subscribers: phk, imp.

TODO: determine if we can/should reuse any of this for code coverage (idea from @phk)

emaste added inline comments.
contrib/bmake/mk/lib.mk
260–262

@sjg, this would have no effect after my change but would you want to leave it to reduce diffs against upstream?

I changed the WITH_PROFILE description in f768ecf247b7acd98fd0ee395522e559769e3e51 as it may not be gone in 14.

This revision is now accepted and ready to land.Nov 12 2024, 2:28 PM
emaste marked an inline comment as not done.Nov 12 2024, 3:12 PM

Updated proposed commit message to

It was disabled by default in fe52b7f60ef4.  We planned to (but did not)
remove the option before FreeBSD 14.  Remove it now, for FreeBSD 15.

I would generally leave contrib alone

contrib/bmake/mk/lib.mk
260–262

Leave it - it isn't used in freebsd and other systems still use profiling

This revision was automatically updated to reflect the committed changes.