From 13add8b7c6ee276e206418f29fe4aa1ead741165 Mon Sep 17 00:00:00 2001 From: Dennis Webb Date: Wed, 24 Dec 2025 20:03:33 -0600 Subject: [PATCH] MAESTRO: Fix Issue #86 - Replace echo with printf in CLI installation instructions - Changed macOS installation command to use printf instead of echo - Changed Linux installation command to use printf instead of echo - Added trailing newline to ensure proper script formatting - Tested both commands to verify they generate valid executable scripts The echo command creates literal \n characters instead of actual newlines, making the generated wrapper scripts invalid. Using printf correctly interprets escape sequences and generates valid executable scripts. Fixes #86 --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 4ee3b843..f9fb09f8 100644 --- a/README.md +++ b/README.md @@ -628,10 +628,10 @@ The CLI is bundled with Maestro as a JavaScript file. Create a shell wrapper to ```bash # macOS (after installing Maestro.app) -echo '#!/bin/bash\nnode "/Applications/Maestro.app/Contents/Resources/maestro-cli.js" "$@"' | sudo tee /usr/local/bin/maestro-cli && sudo chmod +x /usr/local/bin/maestro-cli +printf '#!/bin/bash\nnode "/Applications/Maestro.app/Contents/Resources/maestro-cli.js" "$@"\n' | sudo tee /usr/local/bin/maestro-cli && sudo chmod +x /usr/local/bin/maestro-cli # Linux (deb/rpm installs to /opt) -echo '#!/bin/bash\nnode "/opt/Maestro/resources/maestro-cli.js" "$@"' | sudo tee /usr/local/bin/maestro-cli && sudo chmod +x /usr/local/bin/maestro-cli +printf '#!/bin/bash\nnode "/opt/Maestro/resources/maestro-cli.js" "$@"\n' | sudo tee /usr/local/bin/maestro-cli && sudo chmod +x /usr/local/bin/maestro-cli # Windows (PowerShell as Administrator) - create a batch file @"