search

inexorgame / vulkan-renderer

A new 3D game engine for Linux and Windows using C++20 and Vulkan API 1.3, in very early but ongoing development
https://inexor.org
MIT License
756 stars 33 forks source link

CI is broken #515

Closed IAmNotHanni closed 1 year ago

IAmNotHanni commented 1 year ago

Describe The Bug

CI is broken but local build works fine for me

Steps To Reproduce

Building locally works fine for me: CC=clang-14 CXX=clang++-14 cmake -GNinja -Bbuild -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_FLAGS="-fsanitize=address,undefined,leak" -DINEXOR_BUILD_WITH_CHECK=ON

then cmake --build build

Expected Behavior

CI should pass, give green icon, maybe bake some sweet cookies for me idk

Affected Code

The entire code base?

Operating System

The CI's operating system

Application Version

48d87e259e32a03fc4e7d1eab50b74d6ca36555b

Additional Context

none

Relevant Log Output

none

Related: #513

IAmNotHanni commented 1 year ago

But we already have git config --global --add safe.directory, right?

IceflowRE commented 1 year ago

But we already have git config --global --add safe.directory, right?

This is automatically done by checkout/v3 at the top.

IAmNotHanni commented 1 year ago

So what is the problem? An update in a conan package?

IceflowRE commented 1 year ago

Probably, no clue currently. if you delete your cmake build folder you should be able to reproduce it locally.

IAmNotHanni commented 1 year ago

This is from the GCC build:

 ERROR: Invalid setting '12.2' is not a valid 'settings.compiler.version' value.
Possible values are ['4.1', '4.4', '4.5', '4.6', '4.7', '4.8', '4.9', '5', '5.1', '5.2', '5.3', '5.4', '5.5', '6', '6.1', '6.2', '6.3', '6.4', '6.5', '7', '7.1', 
'7.2', '7.3', '7.4', '7.5', '8', '8.1', '8.2', '8.3', '8.4', '9', '9.1', '9.2', '9.3', '9.4', '10', '10.1', '10.2', '10.3', '11', '11.1', '11.2', '11.3', '12']
IceflowRE commented 1 year ago

We have to version fix our compiler and cannot use the latest.

IceflowRE commented 1 year ago

This issue is basically resolved when #513 and #512 is done.

IceflowRE commented 1 year ago

Closed by #522. - we will use a fixed compiler version and the stable new way of Conan 2.0.