Split perl and non-perl images? about docker-nginx-brotli HOT 5 CLOSED

good point, will try to look into this in the next couple days

from docker-nginx-brotli.

Thanks @fholzer!

50% image size reduction seems nice.

Wanted to share a completely unscientific data point of the runtime memory difference as reported by docker stats.
I'm running just a SPA app -- local dev only, nothing production. So just a small/trivial data point, with that said, I was curious to see MEM USAGE
1.16.1

1.19.1

Could be completely unrelated to the Perl removal, not sure, but maybe it's possible there could be a small reduction in memory usage too.

Either way, nice work and danke!

from docker-nginx-brotli.

This appears to be the relevant commit that introduced the Perl module.
nginxinc/docker-nginx@89f7997

Curious, do you happen to know why Perl was introduced upstream? Maybe the nginx Docker maintainers are targeting specific customer(s) that use Perl?
Not having used Perl on the web since the Movable Type days it seems a bit unusual to ship it in the 'generic' nginx image. But perhaps there's a good reason; would be interested to learn why.

from docker-nginx-brotli.

@fholzer Not sure if you needed Perl for your purposes -- what are your thoughts? I ask b/c both @btmorex and an nginx Docker maintainer, @thresheek seem to avoid it:

the default "latest" or similar images do not have it.

nginxinc/docker-nginx@89f7997

Would you be open to a PR? What's your take on it?

from docker-nginx-brotli.

As perl is rarely used I'm more than OK with removing it from my image, though I won't be maintaining a separate image for it. I hope no one counts on it being supported.

Latest table stable and mainline version images are updated and no longer include perl:

• latest (master/stable)
• v1.18.0
• mainline
• v1.19.1 (still building)

from docker-nginx-brotli.