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

fixed third time bash 3.x errors in 31_network_devices.sh #767

Merged
merged 1 commit into from Feb 11, 2016

Conversation

jsmeix
Copy link
Member

@jsmeix jsmeix commented Feb 11, 2016

fixed third time all remaining bash 3.x errors in 31_network_devices.sh
see #758 plus heavy syntax cleanup that I needed to understand that code at least to the extent that (hopefully) enables me to fix the bash 3.x errors correctly - i.e. without breaking the functionality of the code.

I would very much appreciate it if someone with proper networking knowledge can verify if the functionality of the code is still o.k.

…syntax cleanup that I needed to understand that code at least to the extent that (hopefully) enables me to fix the bash 3.x errors correctly - i.e. without breaking the functionality of the code.
@jsmeix jsmeix added bug The code does not do what it is meant to do waiting for info labels Feb 11, 2016
@jsmeix jsmeix self-assigned this Feb 11, 2016
@gdha gdha added this to the Rear v1.18 milestone Feb 11, 2016
@gdha
Copy link
Member

gdha commented Feb 11, 2016

Looks fine to me - thank you

gdha added a commit that referenced this pull request Feb 11, 2016
fixed third time bash 3.x errors in 31_network_devices.sh
@gdha gdha merged commit 3686763 into rear:master Feb 11, 2016
@jsmeix jsmeix deleted the fix_third_bash_3_error branch February 11, 2016 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The code does not do what it is meant to do fixed / solved / done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants