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

Upload and retain opam.exe artifact #5762

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jonahbeckford
Copy link
Contributor

Makes it easy for code reviewers (so they don't have to rebuild the source code). Makes it easy to compare-and-constrast old version.

Helps #5718

Makes it easy for code reviewers (so they don't have to rebuild the source code).
Makes it easy to compare-and-constrast old version.

Helps ocaml#5718
@rjbou
Copy link
Collaborator

rjbou commented Dec 13, 2023

Thanks a lot for the idea!
I'll update the PR with a generation for unix & macos too, triggered via a label.

@rjbou
Copy link
Collaborator

rjbou commented Dec 14, 2023

Updated the PR with:

  • uploading artefact for linux, mac & windows binaries
  • uploading artefact if the pR have label CI:BINARIES
  • artefact available for 15 days

@rjbou
Copy link
Collaborator

rjbou commented Dec 14, 2023

oups, there is no gh cli on windows jobs.

@kit-ty-kate
Copy link
Member

Windows artifacts are already uploaded since 9ccb2d1 but i'm not sure macOS needs it too given how fast opam takes to compile on there.

I'd be incline to close this if everyone agrees.

@jonahbeckford
Copy link
Contributor Author

The artifacts were generated because of how difficult it was to reproduce the exact environment (C compiler, Unix shell, 3rd party downloads) that generated the opam.exe binary of a PR. Much more difficult on Windows than other OS-es. The compile speed was unimportant.

If no one has complained on other OS-es, then okay to drop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants