Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable --tty for subprocess when parent process is non-tty #1306

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Commits on Sep 14, 2023

  1. Disable --tty for subprocess when parent process is non-tty

    First, stdin not need --tty option, it only dependent --interactive option.
    Secendly, when running in subprocess mode, disable --tty option if
    parent process does not have a tty. This prevents the
    subprocess incorrectly detecting tty and enabling interactive
    features like `ansible-config init`.
    For example, if you use `ansible-navigator config init -m stdout > config`, the subprocess will incorrectly detect tty and waiting 'q' to exit less and cause errors of config. This change avoids that.
    xz-dev committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    b415279 View commit details
    Browse the repository at this point in the history