What about icon for windows build? Is it works?
We can’t add icon to the bundle =(
So sorry, it seems I forgot about this issue, never added it to our issue tracker. Added an issue for it now: DEF-3126
If it is urgent, please use the method @Pkeod explained above until we have resolved the issue.
Unfortunately this method helps with console window, but main game window still without icon =(
Managed to bundle a windows app with custom icon, both for binary and window;
Used this website (there are probably better ways to do it): http://icoconvert.com/
With these settings:
already made a pull request into DefOS with set_window_icon feature
* facepalm *
Still useful!
Might have something to do with native extension built exe? I just tested with a bundled version of DefOS and old method I post didn’t seem to work, but the method you added does.
Bingo! @Pkeod
Bundled icon works only without any native extensions and without AppManifest (if build makes without build server) in the project!
(@sven can you move posts about icon into other forum topic?)
I’m not Sven but I did it anyway
thank you!
This is still an issue. Changing the exe icon manually is a pain, and console window still uses GLFW_ICON instead of the main game window. Also the console window still shows up by default on release builds.
This sucks because I want to release a game on Steam but without these obvious things it will lose a part of professional quality.
Yes, not good. Let me see what can be done. Shouldn’t be that big of a thing to fix.
I can’t really say since it works when native extensions aren’t used, @Mathias_Westerdahl might have a more direct answer?
Not sure actually, since I believe it’s actually Bob that merges the icon into the exe.
But pehaps it’s related to the NE build not being built with /subsystem:windows. This is a flag I’ll add to the SDK when I’m back at my desk next week (not sure what release that’ll be though)
So there’s two problems:
- Changing/setting the icon is a pain
- The console window shows up on release builds.
#1 is a build pipeline issue
#2 is related to native extensions
@Mathias_Westerdahl which one did you refer to with the build flag?
I’ll be looking into #2 next week, and at the same time take a look at #1, to see if it’s related (as I mentioned, it should be bob that does the icon embedding)
I’m convinced they are related.
I added the /subsystem:windows some time ago (pre-NE) and one of the reasons was to get rid of the console window. It doesn’t seem to be applied for NE builds which explains #2.
Regarding #1, I don’t think you can set the console application icon, so not using the /subsystem:windows would mean it’s ignored. Setting the flag will likely correct this.
Solved in 1.2.135