Skip to content

Commit

Permalink
Fix UndefinedVar warnings during Docker build
Browse files Browse the repository at this point in the history
  • Loading branch information
tueda committed Aug 15, 2024
1 parent d20bb0c commit 5d8b145
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@ ENV MG_FOLDER_PATH="${SOFTWARE_FOLDER}/${MG_FOLDER}"
ENV MG_BINARY_PATH="${SOFTWARE_FOLDER}/${MG_FOLDER}/bin/mg5_aMC"

#### CERN ROOT environment variables
ENV PATH=$PATH:$ROOTSYS/bin
ENV LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$ROOTSYS/lib
ENV PATH=${PATH:+$PATH:}$ROOTSYS/bin
ENV LD_LIBRARY_PATH=${LD_LIBRARY_PATH:+$LD_LIBRARY_PATH:}$ROOTSYS/lib

Check warning on line 49 in Dockerfile

View workflow job for this annotation

GitHub Actions / build

Variables should be defined before their use

UndefinedVar: Usage of undefined variable '$LD_LIBRARY_PATH' More info: https://docs.docker.com/go/dockerfile/rule/undefined-var/


### Install MadGraph 5 Python dependencies
Expand All @@ -69,7 +69,7 @@ RUN echo "set auto_convert_model T" | python3 ${MG_BINARY_PATH}
RUN echo "import model EWdim6-full" | python3 ${MG_BINARY_PATH}

# Delphes environment variables
ENV ROOT_INCLUDE_PATH="${ROOT_INCLUDE_PATH}:${MG_FOLDER_PATH}/Delphes/external"
ENV ROOT_INCLUDE_PATH="${ROOT_INCLUDE_PATH:+$ROOT_INCLUDE_PATH:}${MG_FOLDER_PATH}/Delphes/external"

Check warning on line 72 in Dockerfile

View workflow job for this annotation

GitHub Actions / build

Variables should be defined before their use

UndefinedVar: Usage of undefined variable '$ROOT_INCLUDE_PATH' More info: https://docs.docker.com/go/dockerfile/rule/undefined-var/


#### Copy files
Expand Down

0 comments on commit 5d8b145

Please sign in to comment.