docker-library / buildpack-deps

MIT License
445 stars 113 forks source link

Add zip package #145

Closed loganmzz closed 9 months ago

loganmzz commented 1 year ago

unzip command is available, but zip isn't

tianon commented 1 year ago

See "What's included?" in https://hub.docker.com/_/buildpack-deps (https://github.com/docker-library/docs/tree/master/buildpack-deps#whats-included). :eyes:

Can you describe which category you think zip would belong in, and maybe provide some example modules/packages in an ecosystem or two that rely on it? :sweat_smile: :bow:

loganmzz commented 1 year ago
  1. Anywhere unzip command is available. That means all: https://github.com/docker-library/buildpack-deps/blob/27aee70cbf3f37d471d327a6993d9c882bf4b222/Dockerfile.template#L43
  2. I don't think things have changed since this commit: https://github.com/docker-library/buildpack-deps/commit/3fd6bc9602ab42ec1f11dd5680e528d2266f3325
tianon commented 1 year ago

Right, let me try to rephrase. The goal of our set of packages is installing modules, which does commonly require un-zipping, but not commonly zipping (that's more of a packaging creation concern, which is not the intent of this image). :sweat_smile:

tianon commented 9 months ago

Going to close for now, but happy to reconsider if we can find example public modules which fail to install because they're missing zip :eyes: