Skip to content

Treat bundled Boost headers as system includes - #3

Merged
Knogle merged 2 commits into
open.mpfrom
codex/fix-build-errors-in-plugin-89l2za
Jan 30, 2026
Merged

Knogle merged 2 commits into
open.mpfrom
codex/fix-build-errors-in-plugin-89l2za

Conversation

@Knogle

@Knogle Knogle commented Jan 30, 2026

Copy link
Copy Markdown
Owner

Motivation

  • GCC 15 treats certain Boost concept-check patterns as -Werror=nonnull errors which breaks the build, so warnings from bundled Boost headers must be suppressed without touching global compiler flags.
  • The previous approach using a compiler flag to disable nonnull as an error is removed in favor of marking the bundled Boost headers as system includes so the compiler will not treat Boost warnings as project errors.

Description

  • Replace include_directories(${PROJECT_SOURCE_DIR}/lib/boost) with include_directories(SYSTEM ${PROJECT_SOURCE_DIR}/lib/boost) to mark the bundled Boost headers as system includes.
  • Remove the prior GCC-only -Wno-error=nonnull compile option so no ad-hoc warning-suppression flags are kept in the build system.

Testing

  • No automated tests or builds were run for this change.

Codex Task

@Knogle
Knogle merged commit 3cbc8dc into open.mp Jan 30, 2026
4 checks passed
Knogle added a commit that referenced this pull request Mar 13, 2026
Treat bundled Boost headers as system includes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant