Skip to content

Commit

Permalink
Attempt to fix more linter issues
Browse files Browse the repository at this point in the history
  • Loading branch information
shauneccles committed Mar 6, 2021
1 parent 6ee7d41 commit f2a7c1f
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 21 deletions.
9 changes: 2 additions & 7 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,14 @@ ENV PATH="/ledfx/venv/bin:$PATH"

# Install dependencies and ledfx, remove uneeded packages
#
RUN apt-get update
RUN apt-get install -y --no-install-recommends \
RUN apt-get update && apt-get install -y --no-install-recommends \
gcc \
# alsa-utils \
libatlas3-base \
portaudio19-dev \
# pulseaudio \
python3-dev
python3-dev && apt-get purge -y gcc python3-dev && apt-get clean -y && apt-get autoremove -y && rm -rf /var/lib/apt/lists/*
RUN pip install ledfx-dev
RUN apt-get purge -y gcc python3-dev
RUN apt-get clean -y
RUN apt-get autoremove -y
RUN rm -rf /var/lib/apt/lists/*

# Add user `ledfx` and create home folder
RUN useradd --create-home ledfx
Expand Down
7 changes: 3 additions & 4 deletions Dockerfile-4stage
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
alsa-utils \
libatlas3-base \
portaudio19-dev \
pulseaudio
pulseaudio && rm -rf /var/lib/apt/lists/*
RUN apt-get clean -y
RUN apt-get autoremove -y
RUN rm -rf /var/lib/apt/lists/*


### Create docker image from venv-image as compile-image to speed up builds
#
Expand All @@ -25,8 +25,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
libc-dev \
nodejs \
npm \
python3-dev
RUN rm -rf /var/lib/apt/lists/*
python3-dev && rm -rf /var/lib/apt/lists/*

### Create docker image from compile-image to pull and build from github
#
Expand Down
8 changes: 4 additions & 4 deletions Dockerfile-armv7
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
libatlas3-base \
# llvm-9 \
portaudio19-dev \
pulseaudio \
&& apt-get clean -y \
&& apt-get autoremove -y
pulseaudio && rm -rf /var/lib/apt/lists/*
RUN apt-get clean -y \
RUN apt-get autoremove -y
# && ln -s /usr/bin/llvm-config-9 /usr/bin/llvm-config

############### COMPILE IMAGE ###############
Expand All @@ -30,7 +30,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
libc-dev \
# nodejs \
# npm \
python3-dev
python3-dev && rm -rf /var/lib/apt/lists/*

########## BUILD IMAGE ##########
FROM compile-image AS build-image
Expand Down
3 changes: 1 addition & 2 deletions Dockerfile.compile
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,5 @@ ENV PATH="/ledfx/venv/bin:$PATH"
WORKDIR /ledfx
RUN apt-get update && apt-get install -y --no-install-recommends \
gcc \
libc-dev
libc-dev && apt-get clean -y && apt-get autoremove -y && rm -rf /var/lib/apt/lists/*
RUN pip install ledfx-dev
RUN rm -rf /var/lib/apt/lists/*
5 changes: 1 addition & 4 deletions Dockerfile.venv
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,4 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
# libasound2 \
# libasound2-plugins \
portaudio19-dev \
pulseaudio
RUN apt-get clean -y
RUN apt-get autoremove -y
RUN rm -rf /var/lib/apt/lists/*
pulseaudio && apt-get clean -y && apt-get autoremove -y && rm -rf /var/lib/apt/lists/*

0 comments on commit f2a7c1f

Please sign in to comment.