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

Feat: add an error message when WTF was failed #109

Merged
merged 1 commit into from
Jun 1, 2018

Conversation

flw-cn
Copy link
Contributor

@flw-cn flw-cn commented Jun 1, 2018

This is just a minor change to avoid a small number of inexperienced
users not be intimidated by a minor mistake, such as the incorrect
setting of the TERM environment variable.

I hope this will be improved a bit of user experience for let WTF
more popular.

This is just a minor change to avoid a small number of inexperienced
users not be intimidated by a minor mistake, such as the incorrect
setting of the TERM environment variable.

I hope this will be improved a bit of user experience for let WTF
more popular.
@senorprogrammer
Copy link
Collaborator

This looks like a great addition. What should I do to trigger it so I can see it execute in action?

@evanfoster
Copy link

Speaking from experience, this causes the problem:

TERM=xterm-termite ./wtf

Glad I saw this! My $TERM var is set to xterm-termite and I was beating my head against the wall trying to get this thing to work. Doing this:

TERM=xterm-256color ./wtf

Made wtf start up. I don't know if this is in the docs (I looked and couldn't find it) but if it isn't you might want to mention acceptable values for $TERM.

@senorprogrammer
Copy link
Collaborator

Great idea. Will add to docs at some point.

@senorprogrammer senorprogrammer merged commit 54a826e into wtfutil:master Jun 1, 2018
@senorprogrammer
Copy link
Collaborator

This commit 7545ead might be of interest to you. You can now specify the terminal type in your config file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants