-
Notifications
You must be signed in to change notification settings - Fork 88
Update medal.yml #406
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
base: main
Are you sure you want to change the base?
Update medal.yml #406
Conversation
Fix Medal Download link.
That is a dynamic URL though. https://dnschecker.org/server-headers-check.php shows that it currently redirects to Right now, most manifests check the MD5 of the downloaded installer. The medal manifest doesn't check that, so a dynamic URL could be okay. Alternatively we could directly link to a specific installer exe like the one I just mentioned, and add the hash verification, but I am not sure if that's a benefit for a service that almost nobody uses or maintains the manifest for. Remove the useless |
I see, It would download the exe but still fails. Unfortunately it looks like there is more wrong with the installer config that I haven't had the time to study the docs to fix. |
@DJMalachite I recommend running Bottles with |
Thanks I will give that a go when I get home from work 👍 |
@Arcitec Ive done some testing! First and foremost you were right Now there is still a few issues
Changing to Sys-wine fixes that issue but then introduces another The However it can never start as it runs into a Vulkan error
At this point I think it would be better for me to close this merge request and open a issue instead. |
When I checked If someone tests this and confirms it works, I would merge this. You can just remove the question mark |
@jntesteves I did notice that aswell. Ill see if i can get a local build going and test that out. |
You don't need to build anything to test this, just set an environment variable. There are instructions following the link in the first message in this PR, in the big bold title link. In sum, anyone can test this PR by just launching Bottles from a terminal like so:
|
@jntesteves Ill be honest i read up to cloning and then assumed i needed to build my own version of the app.... Anyway I have tested with the command you linked above and now Medal runs with the Soda runner without complaining about admin. However not out fo the woods just yet and it looks like there is a SquirrelCrash when it gets to the updater stage I |
@DJMalachite I'll try to help with what I remember:
|
upped the Dependencies to dotnet481 to fix the mscoree.dll was not found issue
Needs to be added to fix mscoree.dll
Fix Medal Download link.
potentially fix the medal installer as the installer fails with "installer is not well configured" probably because the file is never downloaded
Maybe Fixes #395
Type of change
Was This Tested Using a Local Repository?