Skip to content

Conversation

@jreineckearm
Copy link
Contributor

@jreineckearm jreineckearm commented Oct 20, 2025

This PR gets CI going again after a GitHub windows-2022 runner image update that upgrade GNU tools, including an update from GDB 11.2 to 16.2. This resurfaced issues from #446 which had been suppressed for the time being by pinning runner to windows-2022.

It introduces the following changes:

  • Move runner spec back to windows-latest. Pinning no longer helps with the workaround.
  • Pin mingw installation step to fixed version which contains an older GDB. Unfortunately, that version (10.2) is even older than what was running on windows-2022 before (11.2).
  • Update Windows build jobs to add the chocolatey install to GITHUB_PATH. Which ensures it's present on PATH for following steps of that job.

Downgrading to GDB 10.2 for Windows testing isn't great. But quickest way to get CI going again.
Picking up #446 next as it's root causes become more and more a problem for our CI on Windows.

GITHUB_PATH updated so PATH points to that version in
following steps.

Another workaround for problems with GCC 14/GDB 16
after GitHub windows-2020 runner image was updated
to use them. This only buys more time and doesn't
fix the underlying problem.

Signed-off-by: Jens Reinecke <jens.reinecke@arm.com>
@jreineckearm jreineckearm force-pushed the get-tests-going-again branch from 03aa88c to d96904e Compare October 20, 2025 18:15
mingw 12.2.0 (GDB 12.1.0 / GCC 12.2.0)
to
mingw 11.2.0.07112021 (TO BE CONFIRMED)

GDB 12.1.0 already shows problems with unicode tests:
"can't get string table" error

Signed-off-by: Jens Reinecke <jens.reinecke@arm.com>
@jreineckearm jreineckearm force-pushed the get-tests-going-again branch from 8fa2134 to 3663573 Compare October 20, 2025 18:40
doesn't help anymore.

Signed-off-by: Jens Reinecke <jens.reinecke@arm.com>
@jreineckearm jreineckearm changed the title Investigate solution/workaround for #446 Workaround for #446 after windows-2022 runner image updates Oct 21, 2025
@jreineckearm
Copy link
Contributor Author

Added @jonahgraham and @cwalther for awareness. @omarArm , can you please review?

@jreineckearm jreineckearm marked this pull request as ready for review October 21, 2025 09:41
Copy link
Contributor

@omarArm omarArm left a comment

Choose a reason for hiding this comment

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

LGTM

@jreineckearm jreineckearm merged commit a61275b into eclipse-cdt-cloud:main Oct 21, 2025
4 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