search

Monogramm / docker-dolibarr

:whale: :elephant: Docker image for Dolibarr.
https://github.com/Dolibarr/dolibarr
GNU Affero General Public License v3.0
71 stars 23 forks source link

:green_heart: Add Dolibarr 11 #53

Closed kosssi closed 4 years ago

kosssi commented 4 years ago

It's an upgrade for new Dolibarr v11 https://wiki.dolibarr.org/index.php/Roadmap_and_Release_11.0.0

kosssi commented 4 years ago

Le build ne passe pas si tu as une idée, je ne comprends pas vraiment le problème. https://travis-ci.org/Monogramm/docker-dolibarr/jobs/646014364#L765

madmath03 commented 4 years ago

Bonjour @kosssi,

Le build ne passe pas si tu as une idée, je ne comprends pas vraiment le problème. https://travis-ci.org/Monogramm/docker-dolibarr/jobs/646014364#L765

Le build ne passe pas car les options --with-freetype-dir, --with-png-dir, --with-jpeg-dir ne sont plus disponibles dans l'image docker PHP 7.4.

Au vu des modifications de cette PR, j'imagine que tu as rajouté ces modifications "à la main". Je dis ça car ce projet utilise la même pratique que la plupart des images docker officielles: il y à la racine du projet un script update.sh qui va générer les images Docker automatiquement à partir de templates et des releases Dolibarr sur GitHub. Ce script va générer des images pour PHP 7.2 par défaut.

Y'a t-il une raison de vouloir pousser PHP 7.4 sur la dernière version de Dolibarr ? Je n'ai pas vu d'info indiquant que Dolibarr 11 ne fonctionnait que sous PHP 7.4 et je ne souhaite pas embarquer une version très (trop) récente de PHP pour des containers utilisés en production 😉

kosssi commented 4 years ago

Au vu des modifications de cette PR, j'imagine que tu as rajouté ces modifications "à la main". Je dis ça car ce projet utilise la même pratique que la plupart des images docker officielles: il y à la racine du projet un script update.sh qui va générer les images Docker automatiquement à partir de templates et des releases Dolibarr sur GitHub. Ce script va générer des images pour PHP 7.2 par défaut.

Génial, je viens de refaire la PR du coup. L'upgrade de PHP 7.4 doit être fait dans une autre PR de tout façon et ce n'est pas prioritaire :D

kosssi commented 4 years ago

@madmath03 tout à l'air bon ;)