Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Infrastructure: Update Windows CI zlib to 1.2.12#6027

Merged
vadi2 merged 1 commit intodevelopmentfrom
update-zlib-1.2.12
Apr 2, 2022
Merged

Infrastructure: Update Windows CI zlib to 1.2.12#6027
vadi2 merged 1 commit intodevelopmentfrom
update-zlib-1.2.12

Conversation

@vadi2
Copy link
Member

@vadi2 vadi2 commented Apr 2, 2022

Brief overview of PR changes/additions

Update Windows CI zlib to 1.2.12

Motivation for adding to Mudlet

So Windows CI builds work

Other info (issues closed, discussion etc)

zlib takes down old releases which is a somewhat hostile move -

image

@add-deployment-links
Copy link

add-deployment-links bot commented Apr 2, 2022

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

@mudlet-machine-account mudlet-machine-account added this to the 4.16.0 milestone Apr 2, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Apr 2, 2022

Messages
✔️

PR type: Infrastructure

Generated by 🚫 dangerJS against b290f15

@vadi2 vadi2 marked this pull request as ready for review April 2, 2022 11:45
@vadi2 vadi2 requested a review from a team April 2, 2022 11:45
@vadi2
Copy link
Member Author

vadi2 commented Apr 2, 2022

The Windows build failed due to:

==== compiling and installing luasql.sqlite3 ====
Installing https://luarocks.org/luasql-sqlite3-2.6.0-1.rockspec

Cloning into 'luasql'...
fatal: remote error: 
  The unauthenticated git protocol on port 9418 is no longer supported.
Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information.

Error: Failed cloning git repository.

I'm guessing we're installing a Luarocks version too old there. I've fixed that for Ubuntu builds, but not for Windows as well - that'll be a separate fix from this.

@vadi2 vadi2 enabled auto-merge (squash) April 2, 2022 16:34
@vadi2 vadi2 disabled auto-merge April 2, 2022 17:45
@vadi2 vadi2 merged commit 43c6e81 into development Apr 2, 2022
@vadi2 vadi2 deleted the update-zlib-1.2.12 branch April 2, 2022 17:46
vadi2 added a commit that referenced this pull request Apr 2, 2022
#### Brief overview of PR changes/additions
Install Luarocks 3.8.0 to compile with Github's security policy.
#### Motivation for adding to Mudlet
CI builds on Windows work again.
#### Other info (issues closed, discussion etc)
#6013 fixed it for Ubuntu builds, this fixes it for Windows builds.

Requires #6027 to be merged first in order to work.
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.

3 participants