Skip to content

Fix Public Dockerfiles#35

Merged
kozlov721 merged 3 commits intomainfrom
fix/docker-small-build
Sep 25, 2024
Merged

Fix Public Dockerfiles#35
kozlov721 merged 3 commits intomainfrom
fix/docker-small-build

Conversation

@kozlov721
Copy link
Copy Markdown
Collaborator

@kozlov721 kozlov721 commented Sep 24, 2024 •
edited
Loading

  • Fixed rvc4/Dockerfile to work with the linked snpe zip file
  • Added -f flag to rm commands ran during docker build so the build doesn't fail if the directories are not present
  • Changed log message if an image cannot be pulled

@kozlov721 kozlov721 added the fix Fixes a bug label Sep 24, 2024
@kozlov721 kozlov721 self-assigned this Sep 24, 2024
@kozlov721 kozlov721 changed the title rm -rf in Dockerfiles Fix Public Dockerfiles Sep 24, 2024
@kozlov721 kozlov721 merged commit 9eabe12 into main Sep 25, 2024
@kozlov721 kozlov721 deleted the fix/docker-small-build branch September 25, 2024 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@klemen1999 klemen1999 Awaiting requested review from klemen1999

@tersekmatija tersekmatija Awaiting requested review from tersekmatija

@conorsim conorsim Awaiting requested review from conorsim

Assignees

@kozlov721 kozlov721

Labels

fix Fixes a bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kozlov721