-
Notifications
You must be signed in to change notification settings - Fork 123
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
rsrc merge failure: corrupt .rsrc section #26
Comments
I used cgo in my program.It's build ok without cgo.
|
Hi @pebble2050, thank you very much for the report and for really good info! ❤️ Please note, I'm afraid that rsrc is now quite far down on my backburner list, so I'm not sure when I'll be able to get to this :( That said, to hopefully help track this in future (fingers crossed), either by me or someone else, could you please also try to add the following small pieces of extra info:
TIA! |
1.rsrc -manifest gtool.exe.manifest -o rsrc.syso
|
2.The source I paste above with below deleted is build and work fine.
and
|
|
|
@pebble2050 did you find any fix for this? I'm getting the same error. |
Don't know if it helps, but I started to get the same error after update GCC to tdm64-gcc-9.2.0. I was able to build using an older version of gcc (tdm64-gcc-5.1.0-2) that I have on my PC. I had to use Not sure if it's a gcc or rsrc issue. |
-ldflags="-linkmode internal" |
Internal linkmode isn't a solution when using certain third-party packages. I'm running into this same issue although it seems like it's just a dump to console, all properties are set fine and the program runs just fine! |
Might hopefully be fixed now (starting with rsrc v0.10.0), thanks to @tc-hib ; please comment if this still occurs to you with rsrc v0.10.0. |
For me with |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@akavel Are you sure this is fixed? I tried this icon: https://github.com/syncthing/syncthing/blob/main/assets/logo.ico
This is what go-winres does for the same icon (it has a different ordering, unfortunately):
|
It is much slower and produces a much bigger file too, compared to rsrc v0.9.0 and go-winres. |
Nope, I'm most certainly not sure... 😕 I'll have to update rsrc's readme to recommend people to use your go-winres once I get my act together... and in case I get my act together even more, I'd try to look at this icon, thanks a lot for linking it! |
Should be fixed very soon anyway, it turned out to be a small oversight. Don't worry too much :) |
I published another release 0.10.2 with a fix attempt suggested by @tc-hib . I still don't expect to be debugging this personally unfortunately, but hopefully this or some other suggestion from another interested contributor will help eventually; I'm especially open to a PR with an automated test case reproducing this bug as part of CI (see .travis.yml file), so that we could easily know when this is successfully closed. |
Hey, unfortunately I can still see this error on v0.10.2.
Using GNU toolchain from Cygwin:
|
Hello, |
@mappu Can you please provide a little more information ? Such as the files you embedded, or the resulting syso file. Thanks. |
@mappu if you could provide the information that @tc-hib is asking about, this could be really helpful in trying to fix your issue! It's showing to be really elusive and difficult to debug, so any help is greatly appreciated - and @tc-hib is an awesome contributor who's proven to be really positively hell-bent at trying to resolve it! |
The text was updated successfully, but these errors were encountered: