Skip to content

Run pngquant + optipng - #424

Merged
ben merged 1 commit into
progit:masterfrom
vmarkovtsev:master
Sep 27, 2015
Merged

Run pngquant + optipng#424
ben merged 1 commit into
progit:masterfrom
vmarkovtsev:master

Conversation

@vmarkovtsev

Copy link
Copy Markdown
Contributor

Optimize the size of PNG images by running the following two commands:

find . -name *.png -exec pngquant -f --skip-if-larger {} -o {} \;
find . -name *.png -exec optipng -o5 {} \;

These two commands save about 8.9M (64%). Evaluate the size with

find . -name *.png -print0 | xargs -0 du -csh

References:

Optimize the size of PNG images by running the following two commands:

```
find . -name *.png -exec pngquant -f --skip-if-larger {} -o {} \;
find . -name *.png -exec optipng -o5 {} \;
```

These two commands save about 8.9M (64%). Evaluate the size with

```
find . -name *.png -print0 | xargs -0 du -csh
```

References:

* pngquant: https://pngquant.org
* optipng: http://optipng.sourceforge.net
@ben

ben commented Sep 27, 2015

Copy link
Copy Markdown
Member

Nice. Thanks!

ben added a commit that referenced this pull request Sep 27, 2015
@ben
ben merged commit d6ba858 into progit:master Sep 27, 2015
@jnavila

jnavila commented Sep 28, 2015

Copy link
Copy Markdown
Member

If I recall, the PNGs from the repo aren't deployed on the web site. So, this only helps for ebooks and pdf.

peff added a commit to git/git-scm.com that referenced this pull request Jun 3, 2016
This is basically just a mass-import of every png in
progit2's book/ directory, which appears to be what was done
originally by @schacon when the book was initially imported.

The main goal is to pick up any fixes (such as the one
in progit/progit2#178), but it's
hard to tell which images have changed content. In
progit/progit2#424, all of the
images were png-optimized, so just looking at the tree-diff
doesn't say much. Literally every file image was touched
there.

Still, these are the images going into the current builds of
the ebooks, so one assumes they are what we should be
showing.
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.

3 participants