From 8a43568bcea5744a217274bc899ab9ee5d1fa403 Mon Sep 17 00:00:00 2001
From: Matthew Peveler <matt.peveler@gmail.com>
Date: Tue, 14 Jul 2020 12:52:05 -0400
Subject: [PATCH] Define libcdio version as environment variables in docker

Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
---
 Dockerfile | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 4b0fbad..ef46bd4 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -32,22 +32,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