Skip to content

chore: update github ci tools to ubuntu 22.04#965

Merged
vaeng merged 3 commits intomainfrom
chore--update-github-ci-tools-to-ubuntu22.04
May 6, 2025
Merged

chore: update github ci tools to ubuntu 22.04#965
vaeng merged 3 commits intomainfrom
chore--update-github-ci-tools-to-ubuntu22.04

Conversation

@vaeng
Copy link
Copy Markdown
Contributor

@vaeng vaeng commented May 6, 2025

Ubuntu 20.04 LTS support is going to stop this May.

Pipeline already stopped working.

@vaeng vaeng requested review from ahans and siebenschlaefer May 6, 2025 12:36
Copy link
Copy Markdown
Contributor

@ahans ahans left a comment

Choose a reason for hiding this comment

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

While this will most certainly work, maybe we can drop the extra install steps?

Comment thread .github/workflows/ci.yml Outdated
# Boost must be installed only because the CMake version (3.12) can't
# detect the installed Boost version (1.69)
run: sudo apt-get update && sudo apt-get -y install ninja-build libboost-date-time-dev clang-6.0 g++-10 g++-multilib
run: sudo apt-get update && sudo apt-get -y install ninja-build libboost-date-time-dev clang-15 g++-12 g++-multilib
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.

Have you checked that this is still really necessary? At least clang and gcc are part of the standard image.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wonder if we need multilib

Comment thread .github/workflows/ci.yml
@vaeng vaeng merged commit 0b9f7ee into main May 6, 2025
8 checks passed
@vaeng vaeng deleted the chore--update-github-ci-tools-to-ubuntu22.04 branch May 6, 2025 14:27
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.

2 participants