Allow to enable the asan (default) ,ubsan,tsan and msan.
Details
- Reviewers
bapt arrowd - Group Reviewers
portmgr - Commits
- R11:0592024317ab: features: Add support to enable sanitizers
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
At least meson does have a special flag(s) for enabling sanitizer(s). I suppose they should be used if we detect USES=meson in the works.
Mk/bsd.port.mk | ||
---|---|---|
1019 | The list grows. Should we sort its items? |
Thank you. Now, how about adding support for requesting several sanitizers at the same time? Let SANITIZE_FLAGS accept a list of values like SANITIZE_FLAGS= memory thread and then iterate over it with .for?
Let me know if you don't want to do this. Your change is already pretty useful.
some are incompatible between them, but you can already enable multiples of them with SANITIZE_FLAGS="address,undefined"
Whoa, didn't know that. That's great, can you please expand the comment section then? Something like
# Variables affecting the feature: # SANITIZE_FLAGS Comma-separated list of sanitizers to be enabled during the build # default: memory
Take a look at Mk/Uses/cabal.mk for a more elaborate example.
Mk/Features/sanitize.mk | ||
---|---|---|
7 | Typo: for each bug detected |