[GitHub] [james-project] rouazana commented on a change in pull request #396: JAMES-3261 ZIP packaging for all Guice servers

[GitBox <gi...@apache.org>]

rouazana commented on a change in pull request #396:
URL: https://github.com/apache/james-project/pull/396#discussion_r617354470



##########
File path: dockerfiles/run/guice/memory/Dockerfile
##########
@@ -22,7 +22,10 @@ RUN apt-get update && apt-get install -y unzip
 
 # Get data we need to run James : build results and configuration
 ADD destination/james-server-memory-guice.zip /root
-RUN unzip james-server-memory-guice.zip && cp -r /root/james-server-memory-guice/* /root
+RUN unzip james-server-memory-guice.zip && \
+  cp -r /root/james-server-memory-guice/* /root && \
+  rm james-server-memory-guice.zip && \
+  rm -r james-server-memory-guice

Review comment:
       unzip -d allows to extract to an arbitrary directory




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org