Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build Failed: Missing include in ftxui code #64

Open
ClausKlein opened this issue Aug 28, 2023 · 0 comments · Fixed by ArthurSonzogni/FTXUI#738
Open

Build Failed: Missing include in ftxui code #64

ClausKlein opened this issue Aug 28, 2023 · 0 comments · Fixed by ArthurSonzogni/FTXUI#738

Comments

@ClausKlein
Copy link
Contributor

ClausKlein commented Aug 28, 2023

FAILED: _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/string.cpp.o 
/usr/bin/g++-13  -I/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src -I/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/include -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-protector-strong -fcf-protection -fstack-clash-protection  -D_GLIBCXX_ASSERTIONS -g -std=c++20 -flto=auto -fno-fat-lto-objects -Wall -Werror -Wextra -Wcast-align -Wdeprecated -Wmissing-declarations -Wnon-virtual-dtor -Wnull-dereference -Woverloaded-virtual -Wpedantic -Wshadow -Wunused -MD -MT _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/string.cpp.o -MF _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/string.cpp.o.d -o _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/string.cpp.o -c /builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp
In file included from /builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp:8:
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/include/ftxui/screen/string.hpp:51:48: error: 'uint32_t' was not declared in this scope
   51 | WordBreakProperty CodepointToWordBreakProperty(uint32_t codepoint);
      |                                                ^~~~~~~~
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/include/ftxui/screen/string.hpp:7:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
    6 | #include <vector>    // for vector
  +++ |+#include <cstdint>
    7 | 
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp: In function 'bool {anonymous}::IsCombining(uint32_t)':
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp:1415:45: error: 'ftxui::CodepointToWordBreakProperty' cannot be used as a function
 1415 |   return ftxui::CodepointToWordBreakProperty(ucs) == WBP::Extend;
      |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp: At global scope:
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/src/ftxui/screen/string.cpp:1570:66: error: 'ftxui::WordBreakProperty ftxui::CodepointToWordBreakProperty(uint32_t)' redeclared as different kind of entity
 1570 | WordBreakProperty CodepointToWordBreakProperty(uint32_t codepoint) {
      |                                                                  ^
/builds/ClausKlein/cmake_template/build/_deps/ftxui-src/include/ftxui/screen/string.hpp:51:19: note: previous declaration 'ftxui::WordBreakProperty ftxui::CodepointToWordBreakProperty'
   51 | WordBreakProperty CodepointToWordBreakProperty(uint32_t codepoint);
      |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
[121/196] Building CXX object _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/terminal.cpp.o
[122/196] Building CXX object _deps/ftxui-build/CMakeFiles/screen.dir/src/ftxui/screen/screen.cpp.o
ninja: build stopped: subcommand failed.

Cleaning up project directory and file based variables
00:01
ERROR: Job failed: exit code 1

see gitlab MR 1

ArthurSonzogni added a commit to ArthurSonzogni/FTXUI that referenced this issue Aug 28, 2023
Add missing include guard.
This fix cpp-best-practices/cmake_template#64

Co-authored-by: ArthurSonzogni <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant