[feat] Add Github Actions

hgruniaux commented

2024-03-04 17:40:23 +01:00

(Migrated from github.com)

Actually the action only configure and build the project. Not running or testing is done.

hgruniaux commented

2024-03-04 17:42:30 +01:00

(Migrated from github.com)

Actually we have the same errors as on the ENS computers. The C++20 class std::source_location is not supported by GCC 11 (the version used by the ENS computers and Github)...

Actually we have the same errors as on the ENS computers. The C++20 class `std::source_location` is not supported by GCC 11 (the version used by the ENS computers and Github)...

hgruniaux commented

2024-03-04 22:33:15 +01:00

(Migrated from github.com)

I hate baremetal developpment...

hgruniaux commented

2024-03-04 22:35:54 +01:00

(Migrated from github.com)

For now, we can disable clang-tidy, or use custom targets that call clang-tidy with custom arguments. The problem seems to be CMake that is not able to provide enough information to clang.

What you think ?

For now, we can disable clang-tidy, or use custom targets that call clang-tidy with custom arguments. The problem seems to be CMake that is not able to provide enough information to clang. What you think ?

hgruniaux commented

2024-03-04 22:38:36 +01:00

(Migrated from github.com)

Maybe add

if(CMAKE_EXPORT_COMPILE_COMMANDS)
    set(CMAKE_CXX_STANDARD_INCLUDE_DIRECTORIES ${CMAKE_CXX_IMPLICIT_INCLUDE_DIRECTORIES})
endif()

See https://stackoverflow.com/questions/20587228/clang-error-stddef-file-not-found

Maybe add ```cmake if(CMAKE_EXPORT_COMPILE_COMMANDS) set(CMAKE_CXX_STANDARD_INCLUDE_DIRECTORIES ${CMAKE_CXX_IMPLICIT_INCLUDE_DIRECTORIES}) endif() ``` See https://stackoverflow.com/questions/20587228/clang-error-stddef-file-not-found

desfreng commented

2024-03-04 22:43:22 +01:00

(Migrated from github.com)

It works on my machine. I was able to reproduce clang-tidy error about directories includes. And I have find a workaround. I think my workaround does not works because of the version of clang-tidy...

It works on my machine. I was able to reproduce `clang-tidy` error about directories includes. And I have find a workaround. I think my workaround does not works because of the version of clang-tidy...

hgruniaux commented

2024-03-04 22:44:20 +01:00

(Migrated from github.com)

And what is exactly your workaround ? Do you have tested the above solution ?

desfreng commented

2024-03-04 22:45:35 +01:00

(Migrated from github.com)

To manually add include dirs in a environment variables. The bug is in clang-tidy, cmake works as expected.

hgruniaux commented

2024-03-04 23:30:45 +01:00

(Migrated from github.com)

IT WORKS (at least in Debug mode).
Still remains some cleanup and fixing release mode