Page MenuHomeWildfire Games

Fix the window icon when running the mod mod standalone.
ClosedPublic

Authored by Itms on Apr 2 2018, 11:48 AM.

Details

Summary

When running pyrogenesis -mod=mod, I get

WARNING: Window icon not found.

This is fixed by moving the icon to the mod mod.

Test Plan

Reproduce the warning and check it is fixed. Take a look at the window icon as well.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Itms created this revision.Apr 2 2018, 11:48 AM
Owners added a subscriber: Restricted Owners Package.Apr 2 2018, 11:48 AM
Stan added a subscriber: Stan.Apr 2 2018, 12:29 PM

I experienced it too. I'm wondering though is that Icon is pyrogenesis or 0 A.D. Like if one day we make a full engine split will it make sense to have it still in mod mod ?

elexis added a subscriber: elexis.Apr 2 2018, 12:44 PM
In D1427#58327, @Stan wrote:

I experienced it too. I'm wondering though is that Icon is pyrogenesis or 0 A.D. Like if one day we make a full engine split will it make sense to have it still in mod mod ?

If we do the engine split (which I might want to do next release) and someone comes up with a logo for pyrogenesis, we can have an icon in the mod mod and a 0 A.D. icon in the public mod.

Stan added a comment.Apr 2 2018, 2:27 PM

In this case we might want to duplicate the icon instead of moving it. I believe SVN has a copy option :)

elexis accepted this revision.Apr 4 2018, 1:20 PM

I cant test it with regards to rendering since my window manager doesn't display these icons.
But I could confirm that the file is read with your patch, as I get the warning if the filename were wrong or the thing not mounted.
I can also confirm that the described problem exists and that it's solved with this patch.

I'm ok with both moving and copying as proposed by Stan.

This revision is now accepted and ready to land.Apr 4 2018, 1:20 PM
This revision was automatically updated to reflect the committed changes.