egor-tensin / setup-mingw

GitHub action to set up MinGW-w64
MIT License
74 stars 25 forks source link

The plugin now no longer works on x86 #14

Open Roberto-Gentili opened 9 months ago

Roberto-Gentili commented 9 months ago

The plugin now no longer works on x86 and throws this exception:

`Chocolatey` upgraded 1/1 packages. 
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Remove-Item: D:\a\_temp\d5bbce78-0bb4-467d-86c8-58d03d7142e8.ps1:140
Line |
 140 |          Remove-Item (Join-Path $mingw_lib 'libpthread.dll.a')
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path
     | 'C:\ProgramData\chocolatey\lib\mingw\tools\install\mingw32\i686-w64-mingw32\lib\libpthread.dll.a' because it
     | does not exist.

Here the workflow

MarkCallow commented 9 months ago

It's broken for x64 as well. Seems the latest MinGW (13.1.0) has removed this file. I'm about to try

with:
  static: 0

which should tell the Action to no longer attempt to remove this file.

Even if this works, the Action should be fixed.

Roberto-Gentili commented 9 months ago

It's broken for x64 as well. Seems the latest MinGW (13.1.0) has removed this file. I'm about to try

Even with this: image

... It does not work:

image

jesse-lemurian commented 9 months ago

Also broken for me; tracking this thread.

Roberto-Gentili commented 9 months ago

I solved by using an old compiler version:

image

... We should do some experiments to understand which version is the latest working one

V0ldek commented 9 months ago

Version 12.2.0 works.

Version 13.1.0 was released on 17th of September and broke the builds:

image

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.

V0ldek commented 9 months ago

It's broken for x64 as well. Seems the latest MinGW (13.1.0) has removed this file. I'm about to try

Even with this: image

... It does not work

The error is different though, no longer about pthreads.

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.
jesse-lemurian commented 9 months ago

Confirming 12.2.0 works for me, thanks @V0ldek

shueja commented 5 months ago

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: http://disq.us/p/2w5c5tj

Unfortunately this involved a change in the location of the install directory, leading to the issue with libpthread being missing.