Skip to content

Build geos with tpl PR #243 - #2700

Closed
Algiane wants to merge 3 commits into
developfrom
feature/algiane/NDEBUG-flag-in-Debug-build
Closed

Algiane wants to merge 3 commits into
developfrom
feature/algiane/NDEBUG-flag-in-Debug-build

Conversation

@Algiane

@Algiane Algiane commented Sep 22, 2023 •

Copy link
Copy Markdown
Contributor

This PR avoids the spurious detection of the NDEBUG compilation flag by CMake when importing the hdf5 package.
See TPL PR #243.

Note that ci tests in Debug configuration should fail due to an erroneous assertion (see issue #2631) and should be fixed by PR #2699).

@Algiane Algiane self-assigned this Sep 22, 2023
@Algiane Algiane added the type: bug Something isn't working label Sep 22, 2023
@Algiane Algiane linked an issue Sep 22, 2023 that may be closed by this pull request
@tbeltzun tbeltzun added the flag: TPL(s) build check A PR accompanying a TPL PR to check against GEOS's CI label Sep 28, 2023
@tbeltzun

tbeltzun commented Oct 16, 2023 •

Copy link
Copy Markdown
Contributor

Included in #2721 (with a test added), or #2699.

@tbeltzun tbeltzun closed this Oct 16, 2023
@Algiane
Algiane deleted the feature/algiane/NDEBUG-flag-in-Debug-build branch November 30, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flag: TPL(s) build check A PR accompanying a TPL PR to check against GEOS's CI type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failing ci tests in Debug mode

2 participants