Strip away copyrighted ICC profile #313

Merged
merged 1 commit into from Jul 19, 2014

2 participants

@DavidPrevot

While searching for embedded non-free ICC profile:

find . -regextype posix-extended -iregex '.*\.(jpg|png|tif|ico)' \
    -exec sh -c 'identify -verbose "$0" \
    | grep -i copyright && echo "$0"' {} \;

docs/_static/logo.png showed up. The proposed file just removes the embedded non-free ICC profile:

exiftool -icc_profile"-<=" docs/_static/logo.png

It makes the file a bit smaller, without changing the image.

https://lists.debian.org/debian-devel/2014/05/msg00339.html and the whole thread provide more information.

David Prévot Strip away copyrighted ICC profile
afbc4e3
@jeremeamia

Thanks @DavidPrevot. I'm going to run this by our open source approvers before we merge.

@jeremeamia jeremeamia added the docs label Jun 26, 2014
@jeremeamia jeremeamia merged commit a70f846 into aws:master Jul 19, 2014

2 checks passed

Details continuous-integration/travis-ci The Travis CI build passed
Details default Scrutinizer: No new issues — Travis: Passed
@jeremeamia

Took a little longer than expected, but this has been cleared. Merged.

@DavidPrevot DavidPrevot deleted the DavidPrevot:ICC-free branch Oct 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment