Skip to content

Conversation

@staticfloat
Copy link
Member

We need to serve not only .tar.gz files, but .zip files as well, as many windows users do not have an easy way to extract tarballs.

@staticfloat
Copy link
Member Author

This should solve issues like the one reported here: https://discourse.julialang.org/t/julia-nightly-is-27-days-old/90054

Copy link
Member

@DilumAluthge DilumAluthge left a comment

Choose a reason for hiding this comment

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

Is there a reason that we use PowerShell to create the .zip file, instead of using 7zip or something else that we already have lying around?

But LGTM regardless.

@staticfloat
Copy link
Member Author

Because if we stop shipping p7zip, this will keep on working.

@DilumAluthge DilumAluthge added the automerge Kodiak will auto-merge this PR once all CI is green. (This label was previously called "merge me".) label Nov 14, 2022
@DilumAluthge
Copy link
Member

In hindsight we should have gotten this in before 1.8.3.

@DilumAluthge
Copy link
Member

At least it will be in 1.8.4 (once I backport this PR to the release-julia-1.8 branch in this repo).

We need to serve not only `.tar.gz` files, but `.zip` files as well, as
many windows users do not have an easy way to extract tarballs.
@DilumAluthge DilumAluthge force-pushed the sf/windows_upload_zip_files branch from 97725f0 to 2a4c5dd Compare November 14, 2022 22:28
@staticfloat staticfloat merged commit 1134c2c into main Nov 15, 2022
@staticfloat staticfloat deleted the sf/windows_upload_zip_files branch November 15, 2022 03:44
@DilumAluthge DilumAluthge mentioned this pull request Nov 28, 2022
2 tasks
@DilumAluthge DilumAluthge removed the automerge Kodiak will auto-merge this PR once all CI is green. (This label was previously called "merge me".) label Dec 18, 2022
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