fix: enforce LF newlines for sm_train.sh on Windows (Fixes #5897)

wasim-builds · wasim-builds · commit fa7b7aa9bc65 · 2026-07-25T14:30:28.000+05:30
diff --git a/sagemaker-train/src/sagemaker/train/model_trainer.py b/sagemaker-train/src/sagemaker/train/model_trainer.py
@@ -1132,7 +1132,7 @@ def _prepare_train_script(
             execute_driver=execute_driver,
         )
 
-        with open(os.path.join(tmp_dir.name, TRAIN_SCRIPT), "w") as f:
+        with open(os.path.join(tmp_dir.name, TRAIN_SCRIPT), "w", newline="\n") as f:
             f.write(train_script)
 
     @classmethod

Original file line number	Diff line number	Diff line change
`@@ -1132,7 +1132,7 @@ def _prepare_train_script(`
`1132`	`1132`	`execute_driver=execute_driver,`
`1133`	`1133`	`)`
`1134`	`1134`
`1135`		`- with open(os.path.join(tmp_dir.name, TRAIN_SCRIPT), "w") as f:`
	`1135`	`+ with open(os.path.join(tmp_dir.name, TRAIN_SCRIPT), "w", newline="\n") as f:`
`1136`	`1136`	`f.write(train_script)`
`1137`	`1137`
`1138`	`1138`	`@classmethod`