From 249f29cef689cbd8ae95bc3e91e0526d4c70ab64 Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Tue, 3 Oct 2017 18:35:49 +0530 Subject: [PATCH] networks: Numeric sort of interfaces instead of lexical sort This does not change the order of any of the current devices. Only device supported currently that has multiple interfaces is APU. Signed-off-by: Sunil Mohan Adapa Signed-off-by: Joseph Nuthalapati --- actions/networks | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/actions/networks b/actions/networks index aeaa2b235..d0c6d7346 100755 --- a/actions/networks +++ b/actions/networks @@ -7,11 +7,10 @@ set -e # Creates network-manager connections. function get-interfaces { - # XXX: Sorting of interfaces is non-numeric - WIRED_IFACES=$(nmcli --terse --fields type,device device | grep "^ethernet:" | cut -d: -f2 | sort) + WIRED_IFACES=$(nmcli --terse --fields type,device device | grep "^ethernet:" | cut -d: -f2 | sort -V) NO_OF_WIRED_IFACES=$(echo $WIRED_IFACES | wc -w) - WIRELESS_IFACES=$(nmcli --terse --fields type,device device | grep "^wifi:" | cut -d: -f2 | sort) + WIRELESS_IFACES=$(nmcli --terse --fields type,device device | grep "^wifi:" | cut -d: -f2 | sort -V) NO_OF_WIRELESS_IFACES=$(echo $WIRELESS_IFACES | wc -w) }