Tarsnap / tarsnap

Command-line client code for Tarsnap.
https://tarsnap.com
Other
864 stars 60 forks source link

Add Dockerfile #613

Closed ferryman0608 closed 5 months ago

ferryman0608 commented 5 months ago

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 . ; Run the following command to run the container: docker run -it 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.

gperciva commented 5 months ago

We're not interested in including a Dockerfile in the main repository. There are a few tarsnap dockerfiles already on github.

Your LLM has produced a file which is somewhat hard to read, installs an out-of-date version of tarsnap, and attempts to check its signature with an obsolete keyfile. Either your script does not actually check the signature, or you did not attempt to run the dockerfile.

Given that you've submitted similar PRs to multiple other projects over the past 12 hours, I suspect that you're not examining the LLM output with sufficient care.