set Chocolatey version to 1.4.0 to avoid .NET Framework 4.8 dependency; add error handling for Docker build failures in docker.ps1

Author: youngbash88

Dockerfile.windows.x86_64

@@ -8,6 +8,8 @@ FROM mcr.microsoft.com/windows/servercore:$WIN_VER
 
 ENV chocolateyUseWindowsCompression "true"
 ENV RUST_TOOLCHAIN="1.46.0"
+# Set Chocolatey version to 1.4.0 to avoid .NET Framework 4.8 dependency
+ENV chocolateyVersion="1.4.0"
 
 ADD https://aka.ms/vs/16/release/vs_buildtools.exe C:\TEMP\vs_buildtools.exe
 ADD https://win.rustup.rs/x86_64 C:\TEMP\rustup-init.exe

docker.ps1

@@ -34,11 +34,17 @@ function Build-Tag {
 }
 
 function Build-Container {
+    # Build the container and check for failures
+    # Docker build returns 0 on success, non-zero on failure
     $tag = Build-Tag
     docker build -t $tag `
-    --build-arg GIT_BRANCH=$GIT_BRANCH `
-    --build-arg GIT_COMMIT=$GIT_COMMIT `
-    -f Dockerfile.windows.x86_64 .
+        --build-arg GIT_BRANCH=$GIT_BRANCH `
+        --build-arg GIT_COMMIT=$GIT_COMMIT `
+        -f Dockerfile.windows.x86_64 .
+    if ($LASTEXITCODE -ne 0) {
+        Write-Error "Build failed with exit code $LASTEXITCODE"
+        exit $LASTEXITCODE
+    }
     Write-Host "Build completed for $tag"
 }