[CI] Remove redundant godot-cpp build stage.

[bruvzg]

With a new build script introduced in godotengine/godot-cpp#636 godot-cpp test project automatically build the library, and separate library build step is redundant.

[akien-mga]

I noticed that too. I think it might be a bug that the test project rebuilds godot-cpp if it was already built, ideally it would reuse the existing build and only rebuild if flags changed.

But if the current behavior is intended, then yes we can remove the first build from CI.

[bruvzg]

I think it might be a bug that the test project rebuilds godot-cpp if it was already built, ideally it would reuse the existing build and only rebuild if flags changed.

I'm not sure if it's possible, test project will has its own .sconsign.dblite database in the project folder and use library as a subproject. Scons seems to have an option to set db path, but it's probably won't work if used with multiple projects.

[akien-mga]

Thanks!

[akien-mga]

It seems to be possible to build test after godot-cpp without rebuilding the library with build_library=no, it's used on CI for godot-cpp: https://github.com/godotengine/godot-cpp/blob/60037decc36e223fbcce78586a26e7dd9468033d/.github/workflows/ci.yml#L76-L83

It's equivalent to building both at once so it's probably fine as is, but if ever start adding more test projects, we could use that.