Skip to content

fix(ci): replace unavailable gcc-13/g++-13 with gcc/g++ in CodeQL workflow - #207

Merged
clauspruefer merged 3 commits into
mainfrom
copilot/fix-analyze-c-cpp-job-again
Aug 6, 2026
Merged

fix(ci): replace unavailable gcc-13/g++-13 with gcc/g++ in CodeQL workflow#207
clauspruefer merged 3 commits into
mainfrom
copilot/fix-analyze-c-cpp-job-again

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

gcc-13/g++-13 are no longer locatable as packages on the ubuntu-22.04 GitHub Actions runner image, causing the "Analyze (c-cpp)" CodeQL job to fail at the dependency install step.

Changes

  • .github/workflows/codeql.yml
    • Replace gcc-13 g++-13 with gcc g++ in apt-get install
    • Update CC/CXX env vars and -DCMAKE_C_COMPILER/-DCMAKE_CXX_COMPILER flags accordingly
-  gcc-13 g++-13 \
+  gcc g++ \
...
-  CC: gcc-13
-  CXX: g++-13
-  cmake -B _build -DCMAKE_C_COMPILER=gcc-13 -DCMAKE_CXX_COMPILER=g++-13
+  CC: gcc
+  CXX: g++
+  cmake -B _build -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++

Copilot AI self-assigned this Aug 6, 2026
Copilot AI lite review requested due to automatic review settings August 6, 2026 13:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: clauspruefer <17313789+clauspruefer@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 6, 2026 13:34
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Analyze (c-cpp) fix(ci): replace unavailable gcc-13/g++-13 with gcc/g++ in CodeQL workflow Aug 6, 2026
Copilot AI requested a review from clauspruefer August 6, 2026 13:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

.github/workflows/codeql.yml:50

  • The build step hard-codes gcc/g++. If the workflow installs a newer compiler (needed for the project’s C++23 std::generator usage), this step should also point CMake at the same toolchain to avoid accidentally building with the older default compiler.
        env:
          CC: gcc
          CXX: g++
        run: |
          cmake -B _build -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++
          cmake --build _build --parallel $(nproc)

Comment thread .github/workflows/codeql.yml

@clauspruefer clauspruefer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot this will not fix the problem! We need a gcc >= 13 and g++ >= 13 which is not the case anymore with changing back to gcc and g++ from gcc-13 and g++-13 or did you update the base Ubuntu VM image used to a newer one? If not, check how to install the gcc-14 and g++14 packages.

Co-authored-by: clauspruefer <17313789+clauspruefer@users.noreply.github.com>

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

@copilot this will not fix the problem! We need a gcc >= 13 and g++ >= 13 which is not the case anymore with changing back to gcc and g++ from gcc-13 and g++-13 or did you update the base ...

Updated in commit 62e128b: the CodeQL workflow now adds ppa:ubuntu-toolchain-r/test, installs gcc-14/g++-14, and uses them via CC/CXX and CMake compiler flags.

@clauspruefer
clauspruefer marked this pull request as ready for review August 6, 2026 15:16
@clauspruefer
clauspruefer merged commit 5e2f567 into main Aug 6, 2026
5 of 7 checks passed
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 this pull request may close these issues.

3 participants