Th3Ya0vi / vdrift-ogre

Automatically exported from code.google.com/p/vdrift-ogre
0 stars 0 forks source link

Icon=stuntrally.png #44

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In the desktop files there are lines "Icon=stuntrally.png". This is incorrect, 
there should not be the file extension. Such .desktop files won't pass the 
OpenSUSE build service post-build-checks.

Original issue reported on code.google.com by neptu...@mail.ru on 6 Apr 2011 at 5:26

GoogleCodeExporter commented 8 years ago
Fixed in last commit.

Original comment by scrawl...@gmail.com on 6 Apr 2011 at 6:16

GoogleCodeExporter commented 8 years ago
For the record, the standard isn't very clear on this and the latest released 
.desktop specification version (1.0) in fact has the extension in an example 
file. Still, most .desktop files on Ubuntu seem to omit the extension, so I 
guess it is the right way to go.

Original comment by tapiovie...@gmail.com on 6 Apr 2011 at 6:23

GoogleCodeExporter commented 8 years ago
openSUSE simply won't allow .desktop files with extension. The build will fail.

Original comment by neptu...@mail.ru on 6 Apr 2011 at 6:29

GoogleCodeExporter commented 8 years ago
openSUSE has stricter post-build checks than Ubuntu: it won't allow a .desktop 
file without Icon field, it won't allow .desktop file which refers to an icon 
that is not installed, etc.

Original comment by neptu...@mail.ru on 6 Apr 2011 at 6:33

GoogleCodeExporter commented 8 years ago
That's why your icion file also should be better installed properly with "make 
install" rather than manually.

Original comment by neptu...@mail.ru on 6 Apr 2011 at 6:34

GoogleCodeExporter commented 8 years ago
The icon is installed, look at dist/CMakeLists.txt

Original comment by tapiovie...@gmail.com on 7 Apr 2011 at 7:59