-
Notifications
You must be signed in to change notification settings - Fork 47
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
disabled password echoing #60
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@yejingchen 我仿佛记得以前哪个 openwrt 的版本上会没有 |
@Senorsen 森森你 release 一下吧 |
Done. |
Also updated https://dl.zjuqsc.com/linux/ repos. |
Maybe someone else should take over this work. =) |
这个好坑的,Ubuntu 16.10 也没有。前几天在室友的电脑上装了 Ubuntu 之后想拨号结果还手动改了代码…… |
http://stackoverflow.com/a/3980904 |
在 wsl 上测试了一下 bash 的 read -s 是可以正常工作的,而 dash 不能。鉴于稳定版 win10 wsl 还是 Ubuntu
14.04,这个问题应该是由于脚本还在使用 /bin/sh 而 Ubuntu 上 sh 默认软链到 dash 导致的。我建议把脚本全部切换到
bash,并明确指出我们依赖 bash(但是这样我们就不 POSIX了),或者我们可以在读取密码之前 stty(不推荐
2017年3月23日 11:09,"wfwf1997" <notifications@github.com>写道:
… 这个好坑的,Ubuntu 16.10 也没有。前几天在室友的电脑上装了 Ubuntu 之后想拨号结果还手动改了代码……
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#60 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AE4e8GHy8-zm7osBzXf6ivUkF-MGJXRyks5roeJZgaJpZM4MfOLU>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Password echoing is an embarrassing thing...
Added
read ...
-s
password
.