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

Switch to MSYS2 for installing Emacs on MS-Windows in CI (fix for #55) #56

Merged
merged 1 commit into from
Oct 26, 2021

Conversation

ikappaki
Copy link
Contributor

Hi,

could you please consider fix for #55 whereby the Emacs MS-Windows precompiled binaries installed with purcell/setup-emacs@master GitHub action used in Eldev CI, ships with a GnuTLS version which is susceptible to a bug with expired issuer certificates.

The solution proposed here is to install Emacs as an msys2 package. msys2 itself is already preinstalled on the GitHub Windows 2019 servers, we just utilize it to install the latest Emacs package and set up the global PATH to point to it with an additional GitHub action.

Once a new binary is available in purcell/setup-emacs, we can switch back to that installer. The benefit of purcell/setup-emacs over msys2 is that you can specify which version to install, while with msys2 you can only download the latest available precompiled version (which can change without notice, i.e. going from 27.2 to 28.5 once that becomes available), i.e. no fine grain control which exact version/binary is being used.

All test pass now on CI.

Thanks

Fixes emacs-eldev#55. Should be reverted once new prebuilt Emacs MS-Windows
binaries become available with a newer GnuTLS version.
@doublep
Copy link
Collaborator

doublep commented Apr 14, 2022

@ikappaki: Hello. Now that Emacs 28.1 is out, can we revert to using purcell/setup-emacs also for Windows? Or are there other problems?

@ikappaki
Copy link
Contributor Author

@ikappaki: Hello. Now that Emacs 28.1 is out, can we revert to using purcell/setup-emacs also for Windows? Or are there other problems?

Hi @doublep, yes we can revert, submitted #69 for it. (it is cs090218/setup-emacs-windows we revert to, not purcell/setup-emacs, I had made a mistake in the comments). thanks

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

2 participants