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

remove terminal escapes from configure script for non-tty users #127

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

Conversation

dsheets
Copy link

@dsheets dsheets commented Jun 19, 2013

This patch removes the terminal formatting escape sequences output from the configure script. This is helpful for non-tty users like automated test systems. See OCamlPro/opam-repository#812.

Alternately, if you'd like to keep the vt100 escapes, I would like to humbly ask that you check the return code of "tty -s" and conditionally suppress the escape sequences.

Thank you.

@hbbio
Copy link
Member

hbbio commented Jun 23, 2013

What you suggest is indeed the best solution, i.e. checking tty -s and using vt100 escapes when available. Could you please submit it?

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