Skip to content

Conversation

kou
Copy link

@kou kou commented Nov 8, 2023

Because libstdc++ 10.3.0 doesn't support C++23.

This fixes the following build error:

In file included from /home/runner/work/flatbuffers/flatbuffers/src/idl_parser.cpp:17:
In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/algorithm:60:
In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/stl_algobase.h:67:
/usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/stl_iterator.h:2618:35: error: missing 'typename' prior to dependent type name 'iterator_traits<_It>::iterator_category'
      { using iterator_category = iterator_traits<_It>::iterator_category; };
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@github-actions github-actions bot added the CI Continuous Integration label Nov 8, 2023
@kou kou marked this pull request as draft November 8, 2023 21:35
@kou
Copy link
Author

kou commented Nov 8, 2023

(Sorry. This doesn't work well. I'm looking into this...)

Because libstdc++ 10.3.0 doesn't support C++23.

This fixes the following build error:

    In file included from /home/runner/work/flatbuffers/flatbuffers/src/idl_parser.cpp:17:
    In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/algorithm:60:
    In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/stl_algobase.h:67:
    /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/stl_iterator.h:2618:35: error: missing 'typename' prior to dependent type name 'iterator_traits<_It>::iterator_category'
          { using iterator_category = iterator_traits<_It>::iterator_category; };
                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@kou kou changed the title Use g++-11 and clang++-15 for C++23 Use libc++ not libstdc++ for clang++-14 + C++23 Nov 9, 2023
@kou kou marked this pull request as ready for review November 9, 2023 00:13
@kou
Copy link
Author

kou commented Nov 9, 2023

I used different approach: use libc++ instead of libstdc++

@mustiikhalil
Copy link
Collaborator

cc: @dbaileychess

@dbaileychess
Copy link
Collaborator

I just disabled clang and c++23 until its fully supported.

@kou kou deleted the cpp-23 branch November 18, 2023 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants