thecodingmachine / docker-images-php

A set of PHP Docker images
MIT License
768 stars 137 forks source link

chore: install mongodb from PECL #367

Closed guoard closed 9 months ago

guoard commented 10 months ago

To access the latest versions of the mongodb extension, we can install it from PECL since it is available there.

mistraloz commented 9 months ago

Thx for the PR. I'm not sure to merge it because of the impact for the build time on ARM. I running the test and i will see.

mistraloz commented 9 months ago

As expected it's longer with PECL :

It's not terminated yet but original one required 26min to build, the new one is not finished after 1 hour. For amd one, it's about the same 7min without PECL, 8min with.

It's a crazy amount of resources with 36 tests jobs (18 in arm) and 18 publish jobs (arm and amd one in one job). So i will not merge it but :

So I'm closing this PR but you can open an issue (feature request) to speaking about that for v5

PS: Maybe another option is to use PECL for amd and apt for arm. I don't like this solution because the same tag will not provide the same versions... but it's acceptable as hotfix. So i'll reopen this PR if it's match for you.