-
Notifications
You must be signed in to change notification settings - Fork 394
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
Make CI package the linux build as tar.gz right away #4126
Comments
GitHub CI currently just uses upload-artifact, which just takes a folder and creates a zip. The best way I see would be to make a CI action for release artifacts specifically that can only be manually ran. That would then take care of creating the proper file type, set the |
As I said on Discord, the point of The current dev build Workflow creates an archive without a top-level dir, a.k.a. a tarbomb. |
WSL2 exists |
It's for release builds (which is why I linked docs on how to release).
Can we tell CI to create tar.gz and then ship that in a zip if it really wants? Someone releasing from Windows would just unzip it and it'd be perfectly ready to go.
Does this solve CI's inability to package them into tar.gz?
Is it mentioned in the docs that it needs to be inside a folder?
If a problem is lack of Linux environment, getting a Linux environment clearly does solve it. I have it in a VM. My Win10 is too old for WSL2 and too finetuned to simply update it. I'm talking about a potential dev without a linux environment at hand. And as mentioned above, one needs to understand how to handle permissions properly when a Linux environment already exists. All of this could have been covered in the docs, for example a set of commands that do it all properly. |
The release checklist was written around GitLab CI, which would cover most of that. |
And is that not possible with zip? From a quick google search it does seem to be fully supported. |
It has r/w/x, but only for files, and not user/group. That's probably sufficient, but I still think we should use |
.zips do have permissions, but the way to do that isn't something entirely standardized per se. As such, GitHub Actions' automatic zipping will not preserve these permissions, although if you manually .zip within the action it would preserve these permissions (although of course that'll be a double .zip, and at that point you could just put a .tar.gz in that .zip. |
https://github.com/TASEmulators/BizHawk/wiki/Release-checklist
This requirement makes no sense because it means someone without a Linux environment can't make a proper release, since linux file perms are not a thing on windows, and maybe other problems. GitHub Actions should be able to do it automatically. (And GitLab CI should give a
.tar.gz
instead of a.tar.zip
.)The text was updated successfully, but these errors were encountered: