legacy/lxc-ls: use posix shell instead of bash

- use case .. in instead of comparison with globs - avoid 'local' While here, also avoid 'find ... -printf' which is not supported on busybox Signed-off-by: Natanael Copa <ncopa@alpinelinux.org> Acked-by: Stéphane Graber <stgraber@ubuntu.com>

legacy/lxc-ls: use posix shell instead of bash
2f0e6908 · Natanael Copa · Stéphane Graber · 11bdaa74 · 2f0e6908
Commit 2f0e6908 authored Dec 25, 2012 by Natanael Copa Committed by Stéphane Graber Dec 25, 2012
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 10 deletions

lxc-ls.in src/lxc/legacy/lxc-ls.in +7 -10

No files found.
--- a/src/lxc/legacy/lxc-ls.in
+++ b/src/lxc/legacy/lxc-ls.in
-#!/bin/bash
+#!/bin/sh
 #
 # lxc: linux Container library
@@ -35,8 +35,6 @@ help() {
 get_parent_cgroup()
 {
-	local hierarchies hierarchy fields subsystems init_cgroup mountpoint
 	parent_cgroup=""
 	# Obtain a list of hierarchies that contain one or more subsystems
@@ -61,11 +59,10 @@ get_parent_cgroup()
 		# Return the absolute path to the containers' parent cgroup
 		# (do not append '/lxc' if the hierarchy contains the 'ns' subsystem)
-		if [[ ",$subsystems," == *,ns,* ]]; then
+		case ",$subsystems," in
-			parent_cgroup="${mountpoint}${init_cgroup%/}"
+			*,ns,*) parent_cgroup="${mountpoint}${init_cgroup%/}";;
-		else
+			*) parent_cgroup="${mountpoint}${init_cgroup%/}/lxc";;
-			parent_cgroup="${mountpoint}${init_cgroup%/}/lxc"
+		esac
-		fi
 		break
 	done
 }
@@ -82,12 +79,12 @@ for i in "$@"; do
 			shift; break;;
 		*)
 			break;;
-        esac
+	esac
 done
 containers=""
 if [ ! -z "$directory" ]; then
-	containers=$(find $directory -mindepth 1 -maxdepth 1 -type d -printf "%f\n" 2>/dev/null)
+	containers=$(find $directory -mindepth 1 -maxdepth 1 -type d 2>/dev/null | sed 's:.*/::')
 fi
 cd "$directory"