Skip to content
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

Merged
merged 1 commit into from
Nov 24, 2024
Merged

CI for ubuntu 22 and 24 #592

merged 1 commit into from
Nov 24, 2024

Conversation

vifactor
Copy link
Collaborator

@vifactor vifactor commented Nov 23, 2024

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

@vifactor vifactor force-pushed the ubuntu-ci branch 4 times, most recently from a92aa9d to 98e4680 Compare November 23, 2024 20:52
@vifactor vifactor changed the title Build app for ubuntu 22 and 24 CI for ubuntu 22 and 24 Nov 23, 2024
@vifactor vifactor requested a review from hannesa2 November 23, 2024 20:58
@vifactor vifactor added merge Build pipeline Build pipeline for CI labels Nov 23, 2024
@vifactor vifactor marked this pull request as ready for review November 23, 2024 21:04
@hannesa2 hannesa2 mentioned this pull request Nov 24, 2024
Copy link
Collaborator

@hannesa2 hannesa2 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution.
I renamed current scripts #593 and please rename the newer script names to similar logic like #593.

@@ -135,6 +135,23 @@ jobs:
name: DLTViewer-linux-x86_64
path: build/dist/DLTViewer*.tgz

buildNewerUbuntu:
Copy link
Collaborator

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.

fetch-depth: 0
submodules: true
- name: install build environment
run: scripts/linux/install_newer.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

- name: install build environment
run: scripts/linux/install_newer.sh
- name: Build project
run: scripts/linux/build_newer.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

and here

@hannesa2 hannesa2 removed the merge label Nov 24, 2024
@hannesa2
Copy link
Collaborator

I guess when qt6 works everywhere #405, we can drop Ubuntu 20 with qt5

@vifactor
Copy link
Collaborator Author

Thank you for your contribution. I renamed current scripts #593 and please rename the newer script names to similar logic like #593.

done

@hannesa2 hannesa2 merged commit a719809 into master Nov 24, 2024
6 checks passed
@hannesa2 hannesa2 deleted the ubuntu-ci branch November 24, 2024 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build pipeline Build pipeline for CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants