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

Only include bindist-related files in cryptol.msi #1358

Merged
merged 2 commits into from
May 24, 2022
Merged

Only include bindist-related files in cryptol.msi #1358

merged 2 commits into from
May 24, 2022

Conversation

RyanGlScott
Copy link
Contributor

Previously, we would include every file in the current cryptol checkout in cryptol.msi, which would result in horribly bloated installer files. We now only include those files under the dist directory (created by .github/ci.sh bundle_files), which curates only those files we want to include in a binary distribution.

While I was in town, I modified the conventioned used in win32/cryptol.wxs so that we no longer need to reference dist, which does not show up in .tar.gz-based bindists.

Fixes #977.

Previously, we would include every file in the current `cryptol` checkout in
`cryptol.msi`, which would result in horribly bloated installer files. We now
only include those files under the `dist` directory (created by
`.github/ci.sh bundle_files`), which curates only those files we want to
include in a binary distribution.

While I was in town, I modified the conventioned used in `win32/cryptol.wxs`
so that we no longer need to reference `dist`, which does not show up in
`.tar.gz`-based bindists.

Fixes #977.
@RyanGlScott RyanGlScott requested a review from kquick May 23, 2022 10:48
Copy link
Member

@kquick kquick left a comment

Choose a reason for hiding this comment

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

This seems very reasonable. I have no Windows system to verify this with, but I don't anticipate any particular issues.

@RyanGlScott RyanGlScott merged commit 867e91b into master May 24, 2022
@RyanGlScott RyanGlScott deleted the T977 branch March 22, 2024 14:49
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.

Windows installer includes too many files?
2 participants