Closed badabing2005 closed 1 year ago
It was indeed the PyInstaller version. Setting it to 5.9.0 worked.
Feel free to close this ticket, I'm leaving it open in case you want to make it compatible with the latest PyInstaller. I'm not sure this would be an issue with cdrx/pyinstaller-windows or this action.
Thanks for your work.
I'd say it's very likely an error to do with the Docker image, and is out of my bandwidth at the moment, so PR's are most definitely welcome for this :)
Thanks,
Seeing that you're just using the cdrx/pyinstaller-windows
docker image, the PR would have to be for that project, which seems to be 4 years old.
You probably just have to reference one of the newer forks, perhaps this or this.
I'm low on bandwidth as well, but I do have to revisit this whole build aspect, to make sure my project builds fine for all platforms, which is currently broken for Linux and Macs, and I'm currently manually build those. One of these days (months ...lol), when I find some time, I will rework it all. Thanks again.
I've started a new branch here where I'll try to push an image up that this action can reference python3-10-pyinstaller-5-3
It would be fantastic if you could test this when I create a stable version of the branch before merging into main
Sure, I'd be happy to. Thanks
Can you try your workflow please
Example:
- name: PyInstaller Windows
uses: JackMcKew/pyinstaller-action-windows@python3-10-pyinstaller-5-3
with:
path: src
Cool, it worked. https://github.com/badabing2005/PixelFlasher/actions/runs/4847280857/jobs/8637394909 Many thanks for the prompt fix.
I'd say it's very likely an error to do with the Docker image, and is out of my bandwidth at the moment, so PR's are most definitely welcome for this :)
I'll try helping with this still. Didn't mean to leave the last issue that related to this hang so long with no progress / inputs / PRs.😴
Keep in mind - I've been trying to take a mental break last few weeks & trying to take in the moment of advancements we are experiencing in humanity.
That said - I'll take the feet down shortly and put this on my to-do list.
Other than that, I hope you're doing well @JackMcKew ✌️
https://github.com/JackMcKew/pyinstaller-action-windows/issues/34#issuecomment-1529196967 fixed this issue for me as well.
Much thanks!
I've been using this module for over a year now, and today it started failing.
I can see that in previous successful builds, it pulled
PyInstaller 5.9.0
whereas now it pulledPyInstaller 5.10.1
I'm not sure if it is due to that.I appreciate if you have any insights. Thanks