Skip to content

Remove ICC profiles #540

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

Merged
merged 1 commit into from
Dec 13, 2014
Merged

Conversation

DavidPrevot
Copy link

While checking for copyrighted ICC profiles, under non free license,
the following file has been spotted, and the ICC profile has been
removed:

  exiftool -icc_profile"-<=" src/skin/menu-other-origins.png

The image has been checked identical to its source, and in addition,
the file is about 3 kB smaller without these useless binary data.

The binary ICC profile itself is “Copyright (c) 1998 Hewlett-Packard
Company” as one can see using iccdump once extracted with, e.g.:

  exiftool -icc_profile -b -w icc src/skin/menu-other-origins.png

P.-S.: previously submitted as RequestPolicy/requestpolicy#431 and forwarded by a third party as #431 (I didn’t notice it until now).

@myrdd
Copy link
Member

myrdd commented Dec 13, 2014

Hi @DavidPrevot! Do I understand correctly that the ICC profile can be removed because the image is displayed on monitors only and not printed? (http://docs.gimp.org/en/gimp-imaging-color-management.html)

@DavidPrevot
Copy link
Author

Hi,

Le 13/12/2014 09:32, Martin Kimmerle a écrit :

Do I understand correctly that this can be removed because the image is displayed on monitors only and not printed?

Well, it’s an old but standard version of sRGB, so even printing won’t
actually change the image. The root issue is the (sourceless) binary ICC
profile is copyrighted, by HP and not compliant with the license used to
distribute requestpolicy (GPL-3+).

Some more background on the issue:
https://lists.debian.org/debian-devel/2014/05/msg00339.html

Regards

David

@myrdd
Copy link
Member

myrdd commented Dec 13, 2014

Alright @DavidPrevot thanks for the info – I'll merge this.

myrdd added a commit that referenced this pull request Dec 13, 2014
@myrdd myrdd merged commit d27363e into RequestPolicyContinued:dev-1.0 Dec 13, 2014
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

Successfully merging this pull request may close these issues.

2 participants