Skip to content

Fix syntax error in install.sh (if closed with } instead of fi)#186

Merged
calreynolds merged 1 commit intodatabricks-solutions:mainfrom
malcolndandaro:bugfix/install/error
Feb 25, 2026
Merged

Fix syntax error in install.sh (if closed with } instead of fi)#186
calreynolds merged 1 commit intodatabricks-solutions:mainfrom
malcolndandaro:bugfix/install/error

Conversation

@malcolndandaro
Copy link
Collaborator

Summary

  • Fixes a syntax error on line 862 of install.sh that breaks the curl-based installer:
    bash <(curl -sL https://raw.githubusercontent.com/databricks-solutions/ai-dev-kit/main/install.sh)
    /dev/fd/11: line 862: syntax error near unexpected token `}'
    
  • The bug was introduced in commit 31f18dd (PR Fix silent and global install failures #169) which expanded a one-liner [ "$SCOPE" = "project" ] && { ... } into a multi-line if/then block but closed it with } instead of fi.

Fix

-    if [ "$SCOPE" = "project" ]; then 
+    if [ "$SCOPE" = "project" ]; then
         mkdir -p ".ai-dev-kit"
         echo "$ver" > ".ai-dev-kit/version"
-    }
+    fi

Test plan

  • Run bash -n install.sh — should pass with no syntax errors
  • Run bash <(curl -sL <raw-url>/install.sh) — installer should complete without syntax error

🤖 Generated with Claude Code

Commit 31f18dd (PR databricks-solutions#169) expanded a one-liner into a multi-line
if/then block but closed it with `}` instead of `fi`, causing a
syntax error on line 862 when running the installer via curl.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@calreynolds calreynolds merged commit 4383c9a into databricks-solutions:main Feb 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants