-
Notifications
You must be signed in to change notification settings - Fork 249
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
CI for ubuntu 22 and 24 #592
Conversation
a92aa9d
to
98e4680
Compare
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.
.github/workflows/BuildPR.yml
Outdated
@@ -135,6 +135,23 @@ jobs: | |||
name: DLTViewer-linux-x86_64 | |||
path: build/dist/DLTViewer*.tgz | |||
|
|||
buildNewerUbuntu: |
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.
e.g in the year 2028 the name buildNewerUbuntu
is no more sufficient.
.github/workflows/BuildPR.yml
Outdated
fetch-depth: 0 | ||
submodules: true | ||
- name: install build environment | ||
run: scripts/linux/install_newer.sh |
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.
same here
.github/workflows/BuildPR.yml
Outdated
- name: install build environment | ||
run: scripts/linux/install_newer.sh | ||
- name: Build project | ||
run: scripts/linux/build_newer.sh |
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.
and here
I guess when qt6 works everywhere #405, we can drop Ubuntu 20 with qt5 |
Signed-off-by: Viktor Kopp <[email protected]>
Ubuntu 20.04 approaching its EoL. This PR adds modern versions of ubuntu into CI workflow. This makes sure there is a build against Qt6. The added jobs do not make packaging and do not run tests.
P.S. if support of ubuntu 20.04 is still important and since now on ubuntu 22.04 qt 5.15 is used, it is probably worth to simplify install.sh script and build against legacy Qt 5.12, since it is still kinda supported, see #531