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

Update Dockerfile to fix D1GraphicsTool install step #7748

Merged
merged 2 commits into from
Feb 12, 2025

Conversation

wkdgmr
Copy link
Contributor

@wkdgmr wkdgmr commented Feb 12, 2025

D1GraphicsTool-Linux-Qt5.deb no longer exists and causes the container setup to fail

This PR fixes the issue by updating the URL called in the curl and subsequent commands to the correct/actual latest version

(Sorry, didn't mean to close the previous one)

@AJenbo AJenbo merged commit 3970aae into diasurgical:master Feb 12, 2025
23 checks passed
@StephenCWills
Copy link
Member

Is it really better to target 1.1.0 specifically instead of latest? We'll have to explicitly update the Dockerfile whenever d1gt is updated, and there will be no errors like this one to tip us off.

Of course, we didn't actually notice this error for 2 months so maybe it's inconsequential.

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