search

rogeriopradoj / vagrant-ip-show

A simple vagrant plugin for listing all ips of VM networks
https://rogeriopradoj.github.io/vagrant-ip-show/
MIT License
21 stars 6 forks source link

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

Closed r1w1s1 closed 8 years ago

r1w1s1 commented 8 years ago

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

rogeriopradoj commented 8 years ago

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 commented 8 years ago

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

rogeriopradoj commented 8 years ago

Valeu!!! (Thanks a lot)