tshark.dev icon indicating copy to clipboard operation
tshark.dev copied to clipboard

--minify breaks svgs

Open pocc opened this issue 6 years ago • 3 comments

This was recognized as an issue in minify and was fixed in v2.5.1.

Now we just need to wait for it to percolate into hugo versions so that in netlify commands, --minify can be specified.

pocc avatar Jul 25 '19 08:07 pocc

This was recognized as an issue in minify and was fixed in v2.5.1.

This was actually fixed 5 days after minify v2.5.1 was released, so the version we want is v2.5.2.

As a Hugo contributor, I came across your issue very accidentally, starting with minify before v2.5.1 not building with Go 1.13 in GOPATH mode (for Debian packaging), and then with a failing Hugo minify SVG test due to the fix for https://github.com/tdewolff/minify/issues/260. If it weren't for these two "accidents", we might have never seen this issue at all!

No, dependencies like minify do not percolate into Hugo automatically, and that is how Go modules work by default. After all, "when it ain't broke, don't fix it". So, next time, please do not hesitate to file an issue over at gohugoio/hugo, or we might never see it. Cheers!

anthonyfok avatar Oct 09 '19 06:10 anthonyfok

Genius! Thanks so much @anthonyfok.

Will close this once hugo has a tagged version that can be pushed to and tested on netlify.

pocc avatar Oct 09 '19 16:10 pocc

Now held up by tdewolff/minify#291

pocc avatar Feb 26 '20 09:02 pocc