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

changes deprecat "ifconfig" to new "ip" command #5

Merged
merged 2 commits into from
May 7, 2016
Merged

changes deprecat "ifconfig" to new "ip" command #5

merged 2 commits into from
May 7, 2016

Conversation

r1w1s1
Copy link
Contributor

@r1w1s1 r1w1s1 commented May 7, 2016

changes deprecat "ifconfig" to new "ip" command and make more reliable in other Linux distros.

changes deprecat "ifconfig" to new "ip" command and make more reliable in other Linux distros.
@rogeriopradoj
Copy link
Owner

Hi, @ricardson .

Firstly, thanks for contributing! I didn't know about the ifconfig x ip commands!

Please, change the line with this:

ip a | grep 'inet ' | awk '{ print $2}'  | grep -v 127.0.0.1 | cut -d/ -f1

You proposed way put break line between the ip. I prefer it maintains the same behavior, it the line above make it.

Could you update your pull request?

Thanks in advance!

@r1w1s1
Copy link
Contributor Author

r1w1s1 commented May 7, 2016

Update. I already tested with CentOS7, Fedora23 and Debian Jessie all images are official.

@rogeriopradoj rogeriopradoj merged commit 729abd4 into rogeriopradoj:master May 7, 2016
@rogeriopradoj
Copy link
Owner

Valeu!!! (Thanks a lot)

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