Skip to content
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

PNG warning: iCCP: known incorrect sRGB profile #23

Closed
SnarkBoojum opened this issue Oct 3, 2017 · 5 comments
Closed

PNG warning: iCCP: known incorrect sRGB profile #23

SnarkBoojum opened this issue Oct 3, 2017 · 5 comments

Comments

@SnarkBoojum
Copy link

It looks like some of the png files in the textures directory trigger warnings with recent versions of libpng. To fix them, using "mogrify *.png" is enough to fix things.

@kilbith
Copy link
Collaborator

kilbith commented Oct 3, 2017

Upstream issue.

@kilbith kilbith closed this as completed Oct 3, 2017
@SnarkBoojum
Copy link
Author

You didn't take into account the fact that mogrifying the files in craftguide fixes things : that proves the png files are the problem.

Searching around on the net, it looks like more recent versions of libpng will print warnings where the previous ones didn't : you'll get more reports as those versions get in distributions. And at some point, you'll see them too...

Could you please reopen the issue?

@ghost
Copy link

ghost commented Oct 5, 2017

The images are likely part of minetest_game, or some other mods, and the issue lies there with them. This mod only comes with one or two images, and surely you're not suggesting they're in error?

If so, perhaps you can provide the corrected files?

@SnarkBoojum
Copy link
Author

I'm not suggesting they are : I affirm they are, otherwise mogrifying them wouldn't fix the issue.

Here are the files after mogrification:
craftguide_arrow
craftguide_bg
craftguide_book
craftguide_fire
craftguide_furnace
craftguide_shapeless
craftguide_sign
craftguide_sign_inv

@SnarkBoojum
Copy link
Author

It's annoying, because the issue is closed while the problem is still open!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants