From 3649168f158767fe8a61b21ba2a3dd4db1487e92 Mon Sep 17 00:00:00 2001
From: Jose Diaz-Gonzalez <email@josediazgonzalez.com>
Date: Sun, 22 Sep 2024 20:16:19 -0400
Subject: [PATCH] feat: error out when the Dockerfile is missing during
 builder-dockerfile execution

Users may inadvertently set the builder to 'dockerfile' and have build errors when there is no Dockerfile. This change makes it clear that the dockerfile _must_ exist in order for the build to succeed.

Closes #7181
---
 plugins/builder-dockerfile/builder-build | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/plugins/builder-dockerfile/builder-build b/plugins/builder-dockerfile/builder-build
index 6b72cb12f..08a0d5d10 100755
--- a/plugins/builder-dockerfile/builder-build
+++ b/plugins/builder-dockerfile/builder-build
@@ -23,6 +23,11 @@ trigger-builder-dockerfile-builder-build() {
 
   pushd "$SOURCECODE_WORK_DIR" &>/dev/null
 
+  if [[ ! -f "$SOURCECODE_WORK_DIR/Dockerfile" ]]; then
+    dokku_log_fail "No Dockerfile found in source code directory"
+    return 1
+  fi
+
   if fn-plugn-trigger-exists "pre-build-dockerfile"; then
     dokku_log_warn "Deprecated: please upgrade plugin to use 'pre-build' plugin trigger instead of pre-build-dockerfile"
     plugn trigger pre-build-dockerfile "$APP"