Skip to content

Conversation

@sol
Copy link
Contributor

@sol sol commented Dec 7, 2025

This reduces the build time by 2 minutes from over 5m down to 1m 26s.

Please wait with merging this until the related changes are upstream.

@sol sol changed the title Use "nightly" binaries MicroHs CI Use "nightly" binaries for MicroHs CI Dec 7, 2025
@sol sol marked this pull request as ready for review December 8, 2025 10:53
@sol
Copy link
Contributor Author

sol commented Dec 8, 2025

From what I understand @augustss (please correct me if I got this wrong) isn't eager to provide platform dependent binaries from upstream and instead wants to go for a cross platform distribution approach. I think this makes a lot of sense for end users.

For CI specifically, I still can't justify not to shave off several minutes of build time when it's both easily possible and when there are no apparent downsides to this approach.

So for the time being I'm going to provide nightly binaries specifically for CI use, at least until we converge on a better option.

@MaximilianAlgehed if you want to dog feed this, then this is ready to merge, but please squash the commits.

run: |
PATH="$HOME/.mcabal/bin:$PATH"
mcabal -r build
run: mcabal -r build
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think technically the -r wouldn't be necessary anymore, as dependencies are provided by the nightly.

@augustss
Copy link
Contributor

augustss commented Dec 8, 2025

I intend to provide a fastinstall target in the Makefile that installs in about 10s, but does not use any platform specific binaries.

@MaximilianAlgehed MaximilianAlgehed merged commit b659f39 into nick8325:master Dec 9, 2025
18 checks passed
@sol sol deleted the setup-mhs branch December 9, 2025 12:28
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