IgnorantGuru / spacefm

SpaceFM File Manager
http://ignorantguru.github.com/spacefm/
GNU General Public License v3.0
489 stars 72 forks source link

spacefm icon: incorrect sRGB profile #347

Open domichel opened 11 years ago

domichel commented 11 years ago

Fvwm-crystal http://fvwm-crystal.sourceforge.net use its own application menu system, the menu entries are generated from the desktop and icons files provided by the applications. After installing pcmanfm, I generated the menu entry for it, and from that time, I get the following warning at each start or restart of fbwm-crystal:

libpng warning: iCCP: known incorrect sRGB profile

I ran the convert command used by fvwm-crystal on the icon files, and get:

for i in *; do convert $i -resize 48x48 i; done convert: iCCP: known incorrect sRGB profile `spacefm.png' @ warning/png.c/MagickPNGWarningHandler 1787.

To run convert spacefm.png -strip spacefm.png convert: iCCP: known incorrect sRGB profile `spacefm.png' @ warning/png.c/MagickPNGWarningHandler 1787.

issue the same warning but fix this issue.

For an explanation, see http://www.imagemagick.org/discourse-server/viewtopic.php?f=1&t=22444

The pcmanfm is the only icon that gave me this warning. That imply it need to be fixed into your repository.

I modified the script that generate the icons in fvwm-crystal to apply the -strip filter, so this will not be a problem any more, but it would be better if you fix it at the source.

BTW. the new desktop manager in fvwm-crystal-3.2.0 do support pcmanfm.

Best!

IgnorantGuru commented 10 years ago

Thanks for the report - I have forwarded this to Goran Simovic (icon creator) in case he wants to make adjustments there. [Unfortunately my email to Goran bounced so I don't currently have a way to reach him.]