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

build.yml Potential workaround for Chocolatey-CLI regression #34

Closed
wants to merge 0 commits into from

Conversation

anchorlightforge
Copy link
Contributor

@anchorlightforge anchorlightforge commented Jul 6, 2023

See issue #33

After performing this change to build.yml, I was able to complete my first workflow run since the choco update initially broke new builds:
https://github.com/anchorlightforge/dxvk-remix/actions/runs/5480126437

This commit does force a specific Vulkan version but this is easy to revert.

@anchorlightforge
Copy link
Contributor Author

This fix might not be as bulletproof as I hoped. Apparently with --noop the command just ignores any operation that would have happened, meaning if the build environment doesn't have the Vulkan SDK already installed you'll have the build fail.

As for why this still managed to fix the repo, I'll have to do some further investigation. I won't have a lot of time on my hands for the next week but I'll continue to dig into the issue in hopes that I can find something useful, and in the meantime I'll keep updating my own fork for as long as it works.

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.

None yet

1 participant