-
Notifications
You must be signed in to change notification settings - Fork 71
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
I keep ending up with a 0 byte executable #81
Comments
hey @Nickhoyer , could you try using some variation of the code in the docs here Line 92 in cf0a017
|
Same, I keep getting 0 bytes. Even when specifying a new location that's not the same location of the executable it's still 0 bytes. Not sure how to fix it, and I'm on Linux(Ubuntu Server) Looking at the photo Nickhoyer posted it's not a download link for a executable, but a page with JSON on it. If that makes sense? For an example here's mine Going to the "new exe download url" shows So maybe this is the problem? I have no idea lol |
The github api returns json to basic get requests. You have to set
|
It'd be helpful if you could modify your code to use the example code linked to in #81 (comment), modify the final |
That's... Interesting. And if you replace the contents of that update function with a call to the GitHub updater, then it spits out an empty file again? |
Ah, yeah that'll do it! Glad you were able to get to the bottom of it. Maybe it was the same thing affecting @Nickhoyer |
Is there a way to configure self_update to not extract? |
I tried changing my main function from async to sync, I tried running as admin, running in a different folder, closing the app after updating, but every time, it looks to be correctly downloading a 13.2MB file from github, but then without warnings, it replaces my local exe with a 0 byte file that will never run.
Is there anything I can do to avoid this? I didn't see it mentioned in other issues. I'm on windows, and so is my ci runner.
The text was updated successfully, but these errors were encountered: