Skip to content

Only use absl headers if C++14 is available.#7824

Merged
dbaileychess merged 1 commit intogoogle:masterfrom
rouault:FLATBUFFERS_DO_NOT_USE_ABSL
Feb 15, 2023
Merged

Only use absl headers if C++14 is available.#7824
dbaileychess merged 1 commit intogoogle:masterfrom
rouault:FLATBUFFERS_DO_NOT_USE_ABSL

Conversation

@rouault
Copy link
Copy Markdown
Contributor

@rouault rouault commented Feb 11, 2023

If flatbuffers is built in C++11 mode, but there is a recent version of absl which requires C++14, the build will fail.
Cf MapServer/MapServer#6822 for the use case that triggered this.

@dbaileychess
Copy link
Copy Markdown
Collaborator

I would rather add a && (__cplusplus >= 201411) instead of a new macro.

If flatbuffers is built in C++11 mode, but there is a recent version of
absl which requires C++14, the build will fail.
Cf MapServer/MapServer#6822 for the use case
that triggered this.
@rouault rouault force-pushed the FLATBUFFERS_DO_NOT_USE_ABSL branch from cde8c4b to 3d88680 Compare February 15, 2023 18:48
@rouault
Copy link
Copy Markdown
Contributor Author

rouault commented Feb 15, 2023

I would rather add a && (__cplusplus >= 201411) instead of a new macro.

indeed :-) Just amended this PR with this

@rouault rouault changed the title Add a FLATBUFFERS_DO_NOT_USE_ABSL macro to not use absl even if found Only use absl headers if C++14 is available. Feb 15, 2023
@dbaileychess dbaileychess merged commit a56f9ec into google:master Feb 15, 2023
rouault added a commit to rouault/mapserver that referenced this pull request Feb 15, 2023
github-actions bot pushed a commit to MapServer/MapServer that referenced this pull request Feb 16, 2023
jmckenna pushed a commit to MapServer/MapServer that referenced this pull request Feb 16, 2023
Port of upstream fix in google/flatbuffers#7824

Co-authored-by: Even Rouault <even.rouault@spatialys.com>
dbaileychess pushed a commit that referenced this pull request Mar 15, 2023
If flatbuffers is built in C++11 mode, but there is a recent version of
absl which requires C++14, the build will fail.
Cf MapServer/MapServer#6822 for the use case
that triggered this.
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
If flatbuffers is built in C++11 mode, but there is a recent version of
absl which requires C++14, the build will fail.
Cf MapServer/MapServer#6822 for the use case
that triggered this.
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
If flatbuffers is built in C++11 mode, but there is a recent version of
absl which requires C++14, the build will fail.
Cf MapServer/MapServer#6822 for the use case
that triggered this.
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.

2 participants