RUN git clone instead of COPY to /usr/src/app
#8827
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Avoid the need for a separate /yolov5 git clone directory.
Problem with
COPY . /usr/src/app
is that directory is not a .git directory, so normal ops like git pull etc don't work in Docker currently. This should resolve this issue.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Improvement of Dockerfile configurations for YOLOv5 repository cloning.
📊 Key Changes
COPY . /usr/src/app
command in Dockerfiles across multiple architectures (default, ARM64, CPU).RUN git clone https://github.com/ultralytics/yolov5 /usr/src/app
to directly clone the YOLOv5 repo into the working directory.🎯 Purpose & Impact
.git
directory due to the previous COPY command.