Re: [Tails-dev] review and merge feature/6999-compress-png

Supprimer ce message

Répondre à ce message
Auteur: intrigeri
Date:  
À: The Tails public development discussion list
Sujet: Re: [Tails-dev] review and merge feature/6999-compress-png
Hi,

sajolida wrote (01 Mar 2015 18:00:02 GMT) :
> Thanks to the investigation of naar on PNG compression, I'm
> submitting a branch that compresses the PNG images we ship from the
> website in our ISO


Yay!

> [...] and adapt the release process accordingly.


That part seems quite problematic to me, as "The compression seems to
be deterministic. Running it several times produces the same binaries"
(found in the commit message that introduces this find command) is
apparently not entirely correct: I've just run that find command, and
it changed no less that 120 image files. If I commit the result as
documented, I bet our .git will grow by ~3% (~2M), just like fetching
your branch did here.

So, it may lead to regular, substantial increases in the size of our
.git directory. Let's please avoid that :)

It might be that this non-determinism happens only because I'm running
different versions of optipng and advdef (and then, will happen "only"
when we switch RMs back and forth, or when the RM upgrade their
system). It might be that this would also happen with the very same
version of the tools. It could be arch-dependent, and a RM running an
amd64 system might get different files than someone running (i386)
Tails.

=> If we want to include this command, this requires quite some more
testing to evaluate what its potential painful impact it. Or, let's
just revert commit 488af40 and merge the branch?

Another option would be to have ikiwiki recompress these images when
building the wiki. This would be a bit costly, so likely not suitable
for local builds, but just fine on our production website I guess.
Is there any plugin that does something like that? If not, it should
be relatively trivial to write.

Cheers!