-
Notifications
You must be signed in to change notification settings - Fork 25
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
The plugin now no longer works on x86 #14
Comments
It's broken for x64 as well. Seems the latest MinGW (13.1.0) has removed this file. I'm about to try
which should tell the Action to no longer attempt to remove this file. Even if this works, the Action should be fixed. |
Also broken for me; tracking this thread. |
Version 12.2.0 works. Version 13.1.0 was released on 17th of September and broke the builds: No idea what changed in that release, because the MinGW team stopped tagging releases and updating the Changelog two major versions ago... (https://www.mingw-w64.org/changelog/) In any case, pinning to 12.2.0 is a workaround for now. |
The error is different though, no longer about Line |
18 | $exe_path = (Get-Command $Exe).Path
| ~~~~~~~~~~~~~~~~
| The term 'i686-w64-mingw32-gcc' is not recognized as a name of a cmdlet, function, script file, or executable
| program. Check the spelling of the name, or if a path was included, verify that the path is correct and try
| again. |
MinGW 13.1.0 broke stuff in the setup-mingw step. Pinning to 12.2.0 is a temporary workaround. See egor-tensin/setup-mingw#14 for details.
MinGW 13.1.0 broke stuff in the setup-mingw step. Pinning to 12.2.0 is a temporary workaround. See egor-tensin/setup-mingw#14 for details.
Confirming 12.2.0 works for me, thanks @V0ldek |
Works around egor-tensin/setup-mingw#14. (cherry picked from commit 0369f6f)
Works around egor-tensin/setup-mingw#14. (cherry picked from commit 0369f6f)
Works around egor-tensin/setup-mingw#14. (cherry picked from commit 0369f6f)
Works around egor-tensin/setup-mingw#14. (cherry picked from commit 0369f6f)
Works around egor-tensin/setup-mingw#14. (cherry picked from commit 0369f6f)
As recommended in: egor-tensin/setup-mingw#14
# [2.8.0](v2.7.8...v2.8.0) (2023-10-24) ### Add * Jensen-Shannon Divergence-based distance ([799284c](799284c)) * metadata retrieval from C ([4e98cfb](4e98cfb)), closes [#198](#198) ### Docs * Compatibility with JavaDoc ([d28dc1a](d28dc1a)) ### Fix * Missing header for `size_t` ([e7bbb2c](e7bbb2c)) ### Make * Update C build pipelines for C# deps ([89f53e7](89f53e7)) * Use older MinGW ([9b91f5d](9b91f5d)), closes [egor-tensin/setup-mingw#14](egor-tensin/setup-mingw#14) * Use recent compiler for NPM uploads ([90042b1](90042b1))
According to the maintainer of the Chocolatey listing for Mingw, the Chocolatey install method was changed for 13.1.0 due to a few reasons: Unfortunately this involved a change in the location of the install directory, leading to the issue with |
egor-tensin/setup-mingw#14 is still not fixed, pinning to 12.2.0 to avoid issues
This adds a workaround for: - egor-tensin/setup-mingw#14
The plugin now no longer works on x86 and throws this exception:
Here the workflow
The text was updated successfully, but these errors were encountered: