Skip to content

Fix interactive prompts silently failing when installed via bash <(curl ...)#34

Merged
jrock2004 merged 2 commits into
mainfrom
copilot/fix-installation-os-selection
Feb 19, 2026
Merged

Fix interactive prompts silently failing when installed via bash <(curl ...)#34
jrock2004 merged 2 commits into
mainfrom
copilot/fix-installation-os-selection

Conversation

Copilot AI commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

When running bash <(curl -L .../curl-install.sh), stdin can end up bound to the process substitution pipe rather than the terminal (notably on macOS with bash 3.2). Any read or select call immediately receives EOF, so the OS selection prompt appears but user input is ignored — empty input falls to the * default case ("Exit") and installation cancels.

Changes

  • scripts/curl-install.sh — before exec-ing install.sh, redirect stdin from /dev/tty if it is not already a TTY:

    if [ -t 0 ]; then
        exec ./install.sh
    else
        exec ./install.sh < /dev/tty
    fi
  • scripts/install.sh — defensive early guard that reconnects stdin to /dev/tty for any invocation path where stdin is not a terminal:

    if [ ! -t 0 ] && [ -c /dev/tty ]; then
        exec < /dev/tty
    fi

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@jrock2004 jrock2004 marked this pull request as ready for review February 19, 2026 21:22
Copilot AI changed the title [WIP] Fix installation issue with OS selection in curl-install script Fix interactive prompts silently failing when installed via bash <(curl ...) Feb 19, 2026
Copilot AI requested a review from jrock2004 February 19, 2026 21:22
@jrock2004 jrock2004 merged commit 168c778 into main Feb 19, 2026
2 of 6 checks passed
@jrock2004 jrock2004 deleted the copilot/fix-installation-os-selection branch February 19, 2026 21:23
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