| Summary: | sys-apps/sandbox with CFLAGS="-DNDEBUG" fails to build | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | Guenther Brunthaler <gb_about_gnu> |
| Component: | Sandbox | Assignee: | Sandbox Maintainers <sandbox> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | kensington |
| Priority: | Normal | Keywords: | PATCH |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: | Patch fixing the problem | ||
hmm, guess i forgot to close this. it was (partially) merged shortly after this was filed: https://gitweb.gentoo.org/proj/sandbox.git/commit/?id=87d6537245b6f7cbf028e4c0e187cda7484729f0 |
Created attachment 454096 [details, diff] Patch fixing the problem When using "release"-version CFLAGS containing "-D NDEBUG", sys-apps/sandbox checks for NDEBUG but handles it incorrectly. There are 2 problems: 1. It replaces a function call by a no-op macro, but the macro has a different number of arguments than the function call. 2. It also uses the macro to expand the function name when compiling the function definition, resulting in a missing function name. The patch fixes those problems.