Merge pull request #498 from MasterOdin/patch-1

Define libcdio version as environment variables in docker

Dockerfile

@@ -33,22 +33,24 @@ RUN apt-get update && apt-get install --no-install-recommends -y \
 
 # libcdio-paranoia / libcdio-utils are wrongfully packaged in Debian, thus built manually
 # see https://github.com/whipper-team/whipper/pull/237#issuecomment-367985625
-RUN curl -o - 'https://ftp.gnu.org/gnu/libcdio/libcdio-2.1.0.tar.bz2' | tar jxf - \
-    && cd libcdio-2.1.0 \
+ENV LIBCDIO_VERSION 2.1.0
+RUN curl -o - "https://ftp.gnu.org/gnu/libcdio/libcdio-${LIBCDIO_VERSION}.tar.bz2" | tar jxf - \
+    && cd libcdio-${LIBCDIO_VERSION} \
     && autoreconf -fi \
     && ./configure --disable-dependency-tracking --disable-cxx --disable-example-progs --disable-static \
     && make install \
     && cd .. \
-    && rm -rf libcdio-2.1.0
+    && rm -rf libcdio-${LIBCDIO_VERSION}
 
 # Install cd-paranoia from tarball
-RUN curl -o - 'https://ftp.gnu.org/gnu/libcdio/libcdio-paranoia-10.2+2.0.1.tar.bz2' | tar jxf - \
-    && cd libcdio-paranoia-10.2+2.0.1 \
+ENV LIBCDIO_PARANOIA_VERSION 10.2+2.0.1
+RUN curl -o - "https://ftp.gnu.org/gnu/libcdio/libcdio-paranoia-${LIBCDIO_PARANOIA_VERSION}.tar.bz2" | tar jxf - \
+    && cd libcdio-paranoia-${LIBCDIO_PARANOIA_VERSION} \
     && autoreconf -fi \
     && ./configure --disable-dependency-tracking --disable-example-progs --disable-static \
     && make install \
     && cd .. \
-    && rm -rf libcdio-paranoia-10.2+2.0.1
+    && rm -rf libcdio-paranoia-${LIBCDIO_PARANOIA_VERSION}
 
 RUN ldconfig