-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update GHA linux runners to Ubuntu 20 #11627
Conversation
a26a837
to
dd9b161
Compare
0ad4e57
to
4d8d1df
Compare
4d8d1df
to
0de62a5
Compare
@OhadMeir ready for review. |
sudo apt-get install libglfw3-dev libglfw3; | ||
sudo apt-get install gcc-7 g++-7; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO Add comment on all places we force the GCC version with the reason
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -258,8 +258,8 @@ jobs: | |||
|
|||
|
|||
#-------------------------------------------------------------------------------- | |||
U18_SH_RSUSB_LiveTest: # Ubuntu 2018, Shared, Legacy live-tests | |||
runs-on: ubuntu-18.04 | |||
U20_SH_RSUSB_LiveTest: # Ubuntu 2020, Shared, Legacy live-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Log file names should be updated. See example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch - Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
GHA deprecate Ubuntu 18 runners,
We update the CI to use Ubuntu 20 runners
See https://github.blog/changelog/2022-08-09-github-actions-the-ubuntu-18-04-actions-runner-image-is-being-deprecated-and-will-be-removed-by-12-1-22/
When I used the default compiler (GCC 9.5) it used lto (link time optimization) and it failed on an internal error.
So I forced it to use GCC 7 which compile.