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

LP_HOSTNAME_ALWAYS=-1 to never show hostname #406

Closed
wants to merge 1 commit into from
Closed

LP_HOSTNAME_ALWAYS=-1 to never show hostname #406

wants to merge 1 commit into from

Conversation

augmentedfourth
Copy link
Contributor

On OS X, I've been having problems determining connection status, and often a local connection shows up as if it were Telnet (color: WARN_RED).

Instead of adding an LP_HOSTNAME_NEVER option, or changing the option name to LP_HOSTNAME and breaking existing configs, just added a value "2" that will prevent showing hostname at all.

I realize this is a hack, because I'd rather find out the terminal status properly, but I wasn't having much luck with that so this option will hopefully suffice for now.

@augmentedfourth
Copy link
Contributor Author

New change looks way more extensive than it is, since I wrapped a section in an "if" wrapper and had to increase the indentation level for everything in the block.

@dolmen dolmen changed the title Add LP_HOSTNAME_ALWAYS option to never show hostname LP_HOSTNAME_ALWAYS=2 to never show hostname Nov 4, 2015
@dolmen
Copy link
Collaborator

dolmen commented Nov 4, 2015

Seems ok, except I would prefer to use the value -1 instead of 2 for this case.
Could you fix that and also the commit message of the first commit (see how I changed the new title for this PR)?

@dolmen dolmen added hostname Related to the hostname lookup or display needs work Needs changes before merge is possible labels Nov 4, 2015
@augmentedfourth
Copy link
Contributor Author

Value changed, commit message updated, and commits squashed.

@augmentedfourth augmentedfourth changed the title LP_HOSTNAME_ALWAYS=2 to never show hostname LP_HOSTNAME_ALWAYS=-1 to never show hostname Nov 4, 2015
@augmentedfourth
Copy link
Contributor Author

Rebased over feature/FQDN merge

@dolmen dolmen added merged! and removed needs work Needs changes before merge is possible labels Nov 5, 2015
dolmen added a commit that referenced this pull request Nov 5, 2015
@dolmen
Copy link
Collaborator

dolmen commented Nov 5, 2015

Thanks! Merged!

@dolmen dolmen closed this Nov 5, 2015
@dolmen dolmen self-assigned this Nov 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hostname Related to the hostname lookup or display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants