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

Issue 399 docker #171

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Issue 399 docker #171

wants to merge 3 commits into from

Conversation

samo38
Copy link
Collaborator

@samo38 samo38 commented Dec 19, 2024

No description provided.

@samo38 samo38 requested a review from ehb54 December 19, 2024 04:24
@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

@samo38
Looks good.
Can you change the RH version to 8.10?
You don't need to make new packages - just while we are editing these files, might as well get more current for next release.

@samo38
Copy link
Collaborator Author

samo38 commented Dec 20, 2024

Yes, for sure! what about the Ubuntu one? It is 18.04 by default. Although I made the 22.04 version.

@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

For Ubuntu, I tested 20 & 22 So they should be fine, passing the right parameters to buildpkg.pl worked for me. We should test 24 at some point, we will need to test ssl and setup a cnf

@demeler
Copy link
Collaborator

demeler commented Dec 20, 2024 via email

@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

@samo38
Also - change qt version to 5.15.16 in both dockerfiles
and adjust the versions in buildpkg.pl (the %supported block) to redhat:8.10

@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

@demeler
You can follow the steps here with some adjustments.
IIRC, we did this one before.

@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

with some adjustments.

Primarily updating version #s, install paths & variable substitution.

@samo38
Copy link
Collaborator Author

samo38 commented Dec 20, 2024

redhat and qt versions updated.

@ehb54
Copy link
Owner

ehb54 commented Dec 20, 2024

@samo38
Thanks.
fyi - I'm testing an Ubuntu 24 build, one change so far, will advise.

@demeler
Copy link
Collaborator

demeler commented Dec 20, 2024 via email

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