Skip to content

Commit b83fa26

Browse files
akoclaude
andcommitted
fix: make PAD file linking non-fatal in docker run
docker run failed hard with "linking PAD files: runtime PAD files not found" when the Mendix runtime archive doesn't contain a runtime/pad directory (e.g., Mendix 11.6.3). docker build already handles this gracefully as a warning with a fallback download. Apply the same treatment to docker run. Fixes #37 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9c2e0eb commit b83fa26

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

cmd/mxcli/docker/build.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -281,8 +281,10 @@ func Run(opts RunOptions) error {
281281
// Step 3b: Link PAD runtime files into mxbuild directory
282282
// MxBuild's PAD builder expects template files at mxbuild/{ver}/runtime/pad/,
283283
// but they live in the separately downloaded runtime at runtime/{ver}/runtime/pad/.
284+
// Non-fatal: some Mendix versions don't include PAD files in the runtime archive.
285+
// The docker build step handles this gracefully by downloading the runtime separately.
284286
if err := ensurePADFiles(pv.ProductVersion, w); err != nil {
285-
return fmt.Errorf("linking PAD files: %w", err)
287+
fmt.Fprintf(w, " Warning: %v\n", err)
286288
}
287289

288290
// Step 3c: Ensure demo users exist

0 commit comments

Comments
 (0)