odoo / docker

Other
931 stars 1.51k forks source link

Odoo 15: missing geoip2 #449

Closed ikus060 closed 3 months ago

ikus060 commented 1 year ago

It's related to https://github.com/odoo/docker/issues/268

It seams the python requirement for Geoip is not installed in Odoo:15.0 Docker image.

I'm following this documentation: https://www.odoo.com/documentation/15.0/applications/websites/website/configuration/on-premise_geo-ip-installation.html

Would it be possible to install it in the docker image.

ikus060 commented 1 year ago

The following patch should make the trick

diff --git a/14.0/Dockerfile b/14.0/Dockerfile
index df7a31c..119aa2d 100644
--- a/14.0/Dockerfile
+++ b/14.0/Dockerfile
@@ -17,6 +17,7 @@ RUN apt-get update && \
         libssl-dev \
         node-less \
         npm \
+        python3-geoip2 \
         python3-num2words \
         python3-pdfminer \
         python3-pip \
diff --git a/15.0/Dockerfile b/15.0/Dockerfile
index 1952f18..d5d3173 100644
--- a/15.0/Dockerfile
+++ b/15.0/Dockerfile
@@ -17,6 +17,7 @@ RUN apt-get update && \
         libssl-dev \
         node-less \
         npm \
+        python3-geoip2 \
         python3-num2words \
         python3-pdfminer \
         python3-pip \
diff --git a/16.0/Dockerfile b/16.0/Dockerfile
index eaaacd4..b26a496 100644
--- a/16.0/Dockerfile
+++ b/16.0/Dockerfile
@@ -17,6 +17,7 @@ RUN apt-get update && \
         libssl-dev \
         node-less \
         npm \
+        python3-geoip2 \
         python3-num2words \
         python3-pdfminer \
         python3-pip \
lathama commented 3 months ago

@ikus060 can you do a PR for this change?

lathama commented 3 months ago

@ikus060 nevermind it looks to be a dependency now as noted at https://github.com/odoo/odoo/blob/17.0/debian/control

Can you kindly close out this issue in an effort to clean up the queue?