Skip to content

Commit

Permalink
Apply suggested changes from review comments
Browse files Browse the repository at this point in the history
Signed-off-by: Andreas Heinrich <andreas.heinrich@rwth-aachen.de>
  • Loading branch information
andistorm committed Jul 17, 2024
1 parent b7b5581 commit 2d7d8ee
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 23 deletions.
2 changes: 1 addition & 1 deletion docker/deprecated-images/build-kit/deprecated_wrapper
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ function print_warning {
echo "---------------------------------------------"
echo "WARNING"
echo "This docker image is depreacted."
echo "Please use the debian based build-kit instead 'ghcr.io/everest/everest-ci/build-kit-base' instead."
echo "Please use the debian based build-kit 'ghcr.io/everest/everest-ci/build-kit-base' instead."
echo "Fore more information see https://github.com/EVerest/everest-ci/pull/22"
echo "---------------------------------------------"
echo -e "\033[0m"
Expand Down
10 changes: 3 additions & 7 deletions docker/images/build-env-base/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,7 @@ RUN apt update \
# required by packet sniffer module
pkg-config=1.8.1-1 \
libpcap-dev=1.10.3-1 \
libcap-dev=1:2.66-4 \
# required by RiseV2G
maven=3.8.7-1 \
libxml2-dev=2.9.14+dfsg-1.3~deb12u1 \
libxslt1-dev=1.1.35-1
libcap-dev=1:2.66-4

# Install Python packages
RUN apt update \
Expand All @@ -68,13 +64,13 @@ RUN python3 -m pip install --break-system-packages \
gcovr==7.2

# Install EVerest dependency manager
ARG EDM_VERSION=v0.5.5
ARG EDM_VERSION=v0.6.2
RUN python3 -m pip install --break-system-packages \
git+https://github.com/EVerest/everest-dev-environment@${EDM_VERSION}#subdirectory=dependency_manager

# Install everest-cmake
ARG EVEREST_CMAKE_PATH=/usr/lib/cmake/everest-cmake
ARG EVEREST_CMAKE_VERSION=v0.4.2
ARG EVEREST_CMAKE_VERSION=v0.4.3
RUN git clone https://github.com/EVerest/everest-cmake.git ${EVEREST_CMAKE_PATH} \

Check warning on line 74 in docker/images/build-env-base/Dockerfile

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

docker/images/build-env-base/Dockerfile#L74

Use WORKDIR to switch to a directory
&& cd ${EVEREST_CMAKE_PATH} \
&& git checkout ${EVEREST_CMAKE_VERSION} \
Expand Down
4 changes: 0 additions & 4 deletions docker/images/build-kit-base/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,9 @@ ARG BASE_IMAGE_TAG=latest
FROM ghcr.io/everest/everest-ci/build-env-base:${BASE_IMAGE_TAG}

ENV WORKSPACE_PATH=/workspace
ENV ASSETS_PATH=/assets
ARG EXT_MOUNT=/ext
ENV EXT_MOUNT=$EXT_MOUNT

RUN mkdir $ASSETS_PATH
COPY maven-settings.xml $ASSETS_PATH/

COPY ./entrypoint.sh /

# Disable ownership checks
Expand Down
7 changes: 0 additions & 7 deletions docker/images/build-kit-base/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,6 @@
setup_cache(){
export CPM_SOURCE_CACHE=$EXT_MOUNT/cache/cpm

Check warning on line 4 in docker/images/build-kit-base/entrypoint.sh

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

docker/images/build-kit-base/entrypoint.sh#L4

Double quote to prevent globbing and word splitting.
export CCACHE_DIR=$EXT_MOUNT/cache/ccache

Check warning on line 5 in docker/images/build-kit-base/entrypoint.sh

View check run for this annotation

Codacy Production / Codacy Static Code Analysis

docker/images/build-kit-base/entrypoint.sh#L5

Double quote to prevent globbing and word splitting.

maven_cache_dir=$EXT_MOUNT/cache/m2-repo

# we copy the settings file each time in case the $HOME folder changes
mkdir -p $HOME/.m2 || true
cp $ASSETS_PATH/maven-settings.xml $HOME/.m2/settings.xml
sed -i 's@MAVEN_CACHE_REPOSITORY@'"$maven_cache_dir"'@' $HOME/.m2/settings.xml
}

# if first cmd is exec we're going to exec what follows
Expand Down
4 changes: 0 additions & 4 deletions docker/images/build-kit-base/maven-settings.xml

This file was deleted.

0 comments on commit 2d7d8ee

Please sign in to comment.