<p dir="ltr">Hi</p>
<p dir="ltr">On Jul 29, 2017 6:04 PM, <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>> wrote:<br>
><br>
> Author: wenzeslaus<br>
> Date: 2017-07-29 09:04:18 -0700 (Sat, 29 Jul 2017)<br>
> New Revision: 71318<br>
><br>
> Added:<br>
>    grass/trunk/Dockerfile<br>
> Modified:<br>
>    grass/trunk/README<br>
> Log:<br>
> Dockerfile for the local source code with doc in README<br>
><br>
> Added: grass/trunk/Dockerfile</p>
<p dir="ltr">Great to see docker support!</p>
<p dir="ltr">Here a quick review: I think that some entries can be removed:</p>
<p dir="ltr">...<br>
+        libncurses5-dev \<br>
AFAIK there is no ncurses in GRASS since G7.0.</p>
<p dir="ltr">...<br>
+        libreadline-dev \<br>
This is only used to have readline within r.mapcalc, not sure if needed.</p>
<p dir="ltr">...<br>
+        libxmu-dev \<br>
libxmu-dev? X11 is for long no dependency...<br>
...</p>
<p dir="ltr">+# install GRASS GIS<br>
+RUN ./configure \<br>
...<br>
+    --with-pthread \<br>
 pthread is only used with r.mapcalc parser but internally disabled through workers=0 to avoid errors. I'd take it out.</p>
<p dir="ltr">Best<br>
Markus<br>
</p>