Skip to content
This repository was archived by the owner on Apr 16, 2026. It is now read-only.

Improve functionality for newer lean versions#39

Open
MithicSpirit wants to merge 7 commits into
mpedramfar:masterfrom
MithicSpirit:download_lean
Open

Improve functionality for newer lean versions#39
MithicSpirit wants to merge 7 commits into
mpedramfar:masterfrom
MithicSpirit:download_lean

Conversation

@MithicSpirit

Copy link
Copy Markdown

This removes the bundled lean servers in favor of downloading newer versions directly from github. Alternatively, the user may also explicitly specify a folder from which to get the lean server (if they wish to use a custom build for example).

Also, this removes compression from the library zipfile when building in devmode. This is done because newer versions of mathlib are quite large and compression takes a long time (on my machine this reduced build times from around 4 minutes to just 45 seconds). Since devmode will usually be for local development using the loopback device for networking, the bandwidth will be quite large and the larger file size shouldn't cause problems (you'll save more time from reduced build times than you'll waste waiting for the network).

Allows user to specify where to get the lean server rather than using
the one bundled with lean-game-maker. Also automatically downlods
lean-game-maker as necessary.
These are no longer necessary since the last commit.
Compression takes very long on newer version of mathlib which are very
large. Since devmode is presumably meant for local development, (i.e.,
on the loopback device, which has very high bandwidth), the increased
file size should not be a problem.
@MithicSpirit MithicSpirit mentioned this pull request Jun 16, 2023
This is necessary for newer versions of lean
@MithicSpirit

Copy link
Copy Markdown
Author

Additional commit is to properly support newer versions of lean, which have a problem with old versions of BrowserFS. This is ported from #38.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant