Skip to content

Add Dockerfile#43

Open
ferryman0608 wants to merge 1 commit intosamhocevar:mainfrom
ferryman0608:patch-1
Open

Add Dockerfile#43
ferryman0608 wants to merge 1 commit intosamhocevar:mainfrom
ferryman0608:patch-1

Conversation

@ferryman0608
Copy link

Type of Change: Feature
Explanation: In an effort to support the project's reach and user experience, I've developed a Dockerfile with the aim of streamlining the setup process and ensuring consistent environments across different platforms. This initiative is part of a larger project I'm working on, which leverages large language models to automatically generate Dockerfiles tailored for specific projects.

Usage:
Place the Dockerfile in the root directory of the project;
Run the following command in the root directory of the project to build the image: docker build -t <image_name> . ;
Run the following command to run the container: docker run -it <image_name> bash;

Attached to this PR is the Dockerfile I've proposed. I’m fully open to discussing any aspects of it and am willing to make necessary adjustments to better align with your project's goals and practices.

Thank you very much for considering this contribution.

Comment on lines +2 to +3
LABEL maintainer="EasyPi Software Foundation"
ENV DEBIAN_FRONTEND=noninteractive

Choose a reason for hiding this comment

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

Suggested change
LABEL maintainer="EasyPi Software Foundation"
ENV DEBIAN_FRONTEND=noninteractive
LABEL org.opencontainers.image.authors="EasyPi Software Foundation"
ARG DEBIAN_FRONTEND=noninteractive

@reneleonhardt
Copy link

Hello @ferryman0608 thank you very much for your work!

You're explicitly not using an init entrypoint like catatonit or tini, did you test if all signals like Ctrl+C are being propagated correctly to rinetd to terminate gracefully?

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