From: nbd <nbd@3c298f89-4303-0410-b956-a3cf2f4a3e73>
Date: Fri, 7 Aug 2009 00:08:01 +0000 (+0000)
Subject: base-files: clean up init.d/fstab and port it to the new config_get
X-Git-Url: https://git.rohieb.name/openwrt.git/commitdiff_plain/e4895b7178ed1f4de4d98e94a2f7eb11a1b58fe2

base-files: clean up init.d/fstab and port it to the new config_get

... also remove the unused noauto variable in swapon.

Signed-off-by: Malte S. Stretz <mss@apache.org>

git-svn-id: svn://svn.openwrt.org/openwrt/trunk@17159 3c298f89-4303-0410-b956-a3cf2f4a3e73
---

diff --git a/package/base-files/files/etc/init.d/fstab b/package/base-files/files/etc/init.d/fstab
index 840350ce1..7d1dffa0b 100755
--- a/package/base-files/files/etc/init.d/fstab
+++ b/package/base-files/files/etc/init.d/fstab
@@ -5,15 +5,14 @@ START=20
 
 do_mount() {
 	local cfg="$1"
-	config_get fstype "$cfg" fstype
-	fstype="${fstype:-auto}"
-	config_get options "$cfg" options
-	options="${options:-rw}"
+
 	config_get device "$cfg" device
-	[ -n "$device" ] || return 0
 	config_get target "$cfg" target
-	[ -n "$target" ] || return 0
+	[ -n "$device" -a -n "$target" ] || return 0
+
 	mkdir -p $target
+	config_get fstype "$cfg" fstype 'auto'
+	config_get options "$cfg" options '-rw'
 	config_get_bool enabled "$cfg" "enabled" '1'
 	[ "$enabled" -eq 0 ] && options="noauto,$options"
 	echo "$device	$target	$fstype	$options	0	0" >> /tmp/fstab
@@ -21,32 +20,29 @@ do_mount() {
 
 do_swapon() {
 	local cfg="$1"
+
 	config_get device "$cfg" device
-	[ -n "$device" ] || return 0
 	config_get_bool enabled "$cfg" "enabled" '1'
-	[ "$enabled" -gt 0 ] && {
-		echo "$device	none	swap	${noauto}sw	0	0" >> /tmp/fstab
-	}
+	[ -n "$device" -a "$enabled" -gt 0 ] || return 0
+	echo "$device	none	swap	sw	0	0" >> /tmp/fstab
 }
 
 do_unmount() {
 	local cfg="$1"
+
 	config_get target "$cfg" target
-	[ -n "$target" ] || return 0
 	config_get_bool enabled "$cfg" "enabled" '1'
-	[ "$enabled" -gt 0 ] && {
-		umount $target
-	}
+	[ -n "$target" -a "$enabled" -gt 0 ] || return 0
+	umount $target
 }
 	
 do_swapoff() {
 	local cfg="$1"
+
 	config_get device "$cfg" device
-	[ -n "$device" ] || return 0
 	config_get_bool enabled "$cfg" "enabled" '1'
-	[ "$enabled" -gt 0 ] && type swapoff >/dev/null && {
-		swapoff $device
-	}
+	[ -n "$device" -a  "$enabled" -gt 0 ] && type swapoff >/dev/null || return 0
+	swapoff $device
 }
 
 start() {