Mudlet icon indicating copy to clipboard operation
Mudlet copied to clipboard

Installation of (original) Mingw in `./CI/appveyor.install.ps1` may be bogus

Open SlySven opened this issue 4 years ago • 2 comments

Brief summary of issue / Description of requested feature:

./CI/appveyor.install.ps1 is used in the ./setup-windows-sdk.ps1 script and it includes a function CheckAndInstallMingwGet which is defined in ./CI/appveyor.functions.ps1 and which itself calls an installer function CheckAndInstall with a "mingw-get" argument. This is bogus because that effectively downloads and installs https://osdn.net/frs/redir.php?m=rwthaachen&f=mingw%2F68260%2Fmingw-get-0.6.3-mingw32-pre-20170905-1-bin.zip which is the original Mingw™️ command line setup/installer. It is installed into C\Mingw - however we (or more properly Qt) are not using that AFAICT. Instead we should be specifying Mingw-w64 which is a different product!

Also, as part of the CI process doing this may be pointless (unless it is done as an upgrade ?) as the original Mingw™️ is already installed in the AppVeyor build environment in the Visual Studio 2019 as:

  • in C:\MinGW as "5.3.0" (with GGC 8.2.0 and 9.3.0 compilers)
  • in C:\Qt\5.9.9\mingw53_32 probably the same as above as part of the Qt 5.9.9 installation note that there is also:
  • as part of the Qt 5.14.2 kit there is also:
    • in C:\Qt\5.14.2\mingw73_32 (with GGC 7.3.0 compiler) this must be a Mingw-w64 kit as there is also:
    • in C:\Qt\5.14.2\mingw73_64 (with GGC 7.3.0 compiler) and we all know that the original Mingw:tm: does not do 64-bit code
  • as part of the Qt 5.15.2 kit (the last/latest Qt 5.x) there is also:
    • in C:\Qt\5.15.2\mingw81_32 (with GGC 8.1.0 compiler) this must be a Mingw-w64 kit as there is also:
    • in C:\Qt\5.15.2\mingw81_64 (with GGC 8.1.0 compiler) and again this means this cannot be an original Mingw:tm: thing

SlySven avatar Mar 03 '22 19:03 SlySven

@keneanung ?

vadi2 avatar Mar 04 '22 07:03 vadi2

We do use that indirectly. The mingw-get is used to install the autotools, which are part of the actual MSYS (instead of mingw) and were not part of the Qt distribution when I wrote these scripts. See https://github.com/Mudlet/Mudlet/blob/development/CI/appveyor.functions.ps1#L209

keneanung avatar Mar 07 '22 20:03 keneanung