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

MDEV-505: feature request: add \H option for mysql client prompt #163

Closed
wants to merge 1 commit into from
Closed

Conversation

udan11
Copy link

@udan11 udan11 commented Mar 8, 2016

MDEV-505: feature request: add \H option for mysql client prompt

Introduce \H option which behaves mostly like \h. The only exception is
when the client connects to the server hosted on localhost. In this case, the
hostname will be used instead.

I already sent my MCA so the developers of MariaDB may use this patch however they consider best.

Best regards,
Dan Ungureanu

Introduce `\H` option which behaves mostly like `\h`. The only exception is
when the client connects to the server hosted on localhost. In this case, the
hostname will be used instead.
@svoj svoj assigned svoj Mar 9, 2016
@svoj
Copy link

svoj commented Mar 9, 2016

Hi Dan,

This task was added to 10.2.0 backlog, which hasn't been scheduled yet. I can guess it will be around May 2016.

Thanks,
Sergey

break;
const char *prompt;
prompt= connected ? mysql_get_host_info(&mysql) : "not_connected";
if (strstr(prompt, "Localhost"))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Original patch had strstr("localhost") as well, it was supposed for "localhost" connections via TCP/IP. Not sure if it makes sense though.

@svoj
Copy link

svoj commented May 10, 2016

Pushed eca0607

@svoj svoj closed this May 10, 2016
vuvova added a commit that referenced this pull request Jun 22, 2017
vuvova added a commit that referenced this pull request Dec 4, 2017
This reverts commit 1cfdff5.

Fix it differently.
Cleanup, test results didn't change.
dr-m pushed a commit that referenced this pull request Dec 8, 2017
This reverts commit 1cfdff5.

Fix it differently.
Cleanup, test results didn't change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants