-
Notifications
You must be signed in to change notification settings - Fork 8
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
Icon replacement in Windows causes exe fail to excutation #4
Comments
There is an error in the doc: the key/value separator of command line arguments is So the command should be:
Also if your path(s) contains spaces your should enclose it with double quotes. |
I will update the documentation |
I have the same error, but when I do:
|
This error is very generic, so I can do nothing for you. You'll have to investigate on your own.
|
When I put my app and my ico in the bin folder I run this command, but it still does that error: FYI, the app did work before. EDIT: I am trying a different icon now. |
I tried with a different ico, and it still shows that error. |
I also tried by opening the Anoles Resourcer and change the icon with that, but it still does that error. (I used a fresh working copy of the app) My guess it that my version of Windows (Windows 10) doesn't allow it. |
does app.exe is NWJS or another binary ? Then, this project is only a wrapper around winresourcer, which is a bit old now (2008). It's possible it don't work on recent OS. |
You may try with another project (rcedit or resourcehacker) |
Yes, I am using NWJS |
I checked and the package does have "IDR_MAINFRAME". Okay, I will try those other projects |
The icon does update but the app does not run. Same with Resource Hacker - in which case it throws a different error. |
Are you using 32 or 64? |
I was using 64 |
I think there is a problem with 64, 32 did work for me. |
I use this to replace a icon of an nwjs Windows application.
The result exe failed to excutation with an error:.
I aslo found the exe after icon replacement, exe file size reduce 20M.
The text was updated successfully, but these errors were encountered: