Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

automate OL7 updates #487

Open
wants to merge 6 commits into
base: jbr21.631
Choose a base branch
from
Open

Conversation

vprovodin
Copy link
Collaborator

No description provided.

@vprovodin vprovodin requested a review from mkartashev December 26, 2024 09:47
devtoolset-10-binutils-2.35-5.el7.4 \
devtoolset-10-gcc-10.2.1-11.2.0.2.el7 \
devtoolset-10-gcc-c++-10.2.1-11.2.0.2.el7
RUN yum -y install \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's best to have as little RUN commands as possible to reduce the number of layers and get a smaller image.

COPY scl.repo /etc/yum.repos.d/

# Install the necessary build tools
RUN yum -y update; \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's combine operations with && rather than ';. Otherwise it's easy to miss an error generated by an earlier command.

@@ -0,0 +1,57 @@
# NOTE: This Dockerfile is meant to be used from the mkdocker_64.sh script.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mkdocker_64.sh -> mkdocker_x64.sh

BOOT_JDK_REMOTE_FILE=zulu17.30.15-ca-jdk17.0.1-linux_aarch64.tar.gz
BOOT_JDK_SHA=4d9c9116eb0cdd2d7fb220d6d27059f4bf1b7e95cc93d5512bd8ce3791af86c7
BOOT_JDK_LOCAL_FILE=boot_jdk.tar.gz
echo "ULN username:";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no need for ; at the end of these lines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants