Page MenuHomeFreeBSD

Retire WITHOUT_DEBUG flag, due to poor support in many ports and not being really necessary
ClosedPublic

Authored by madpilot on Sep 29 2017, 10:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 7:24 PM
Unknown Object (File)
Sun, Nov 24, 12:15 AM
Unknown Object (File)
Sat, Nov 23, 3:29 PM
Unknown Object (File)
Thu, Nov 21, 8:54 PM
Unknown Object (File)
Nov 19 2024, 8:13 PM
Unknown Object (File)
Nov 19 2024, 5:13 PM
Unknown Object (File)
Nov 19 2024, 1:51 AM
Unknown Object (File)
Nov 15 2024, 3:08 PM
Subscribers

Details

Reviewers
mat
rene
Group Reviewers
portmgr
Commits
rP452147: Retire WITHOUT_DEBUG flag.
Summary

While building ports in a poudriere jail defining WITH_DEBUG I had the need to disable debug for specific ports failing to build due to requiring too much RAM for debugging build (llvm, for example).

I discovered that most ports who check for WITH_DEBUG are ignoring WITHOUT_DEBUG.

Also, the same intended effect of the WITHOUT_DEBUG flag can be obtained by using ".undef WITH_DEBUG" without requiring port Makefiles to have any special support.

Test Plan

If required an exp-run could be useful to make sure I did not break anything.

The patch does not modify the behavior of many ports having a DEBUG OPTION, bug not respecting WITH_DEBUG. A more complex patch is needed to fix that, but removing WITHOUT_DEBUG is a first small step in the right direction to fix that too.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I think the WITHOUT_DEBUG knob should be removed instead.

madpilot retitled this revision from Ports honouring WITH_DEBUG are ignoring WITHOUT_DEBUG in many cases to Retire WITHOUT_DEBUG flag, due to poor support in many ports and not being really necessary.Oct 9 2017, 4:26 PM
madpilot edited the summary of this revision. (Show Details)
madpilot edited the summary of this revision. (Show Details)
madpilot edited the test plan for this revision. (Show Details)

Added a simpler patch which removes all references to WITHOUT_DEBUG and adds a note to UPDATING to inform users of the change and suggests the supported way.

This looks ok to me.

This revision is now accepted and ready to land.Oct 15 2017, 10:42 AM
This revision was automatically updated to reflect the committed changes.