From ae622c93b3b4df7a55ef2cdb9986eec2bf42159b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
Date: Tue, 25 Dec 2018 18:46:44 +0100
Subject: Revert "samba36: add hotplug support"
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit fd569e5e9d0a46ea957cb253e97a4b3ea8c2c540.

After an extra review & discussion few concerns were raised regarding
that feature:
1) It reacts to hotplug.d "block" events instead of more accurate (but
   currently unavailable) "mount" events.
2) It requires *something* to mount block device before samba hotplug.d
   gets fired. Otherwise samba_add_section() will just return.
3) It doesn't reload Samba which some users may expect
4) It operates on /etc/ which is not a right place for autogenerated
   ephemeral config.
5) It doesn't include any cleanup for non-existing shares.

Cc: Rosy Song <rosysong@rosinson.com>
Cc: Jo-Philipp Wich <jo@mein.io>
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 .../network/services/samba36/files/lib/samba.sh    | 89 ----------------------
 1 file changed, 89 deletions(-)
 delete mode 100644 package/network/services/samba36/files/lib/samba.sh

(limited to 'package/network/services/samba36/files/lib/samba.sh')

diff --git a/package/network/services/samba36/files/lib/samba.sh b/package/network/services/samba36/files/lib/samba.sh
deleted file mode 100644
index a72a2a1933..0000000000
--- a/package/network/services/samba36/files/lib/samba.sh
+++ /dev/null
@@ -1,89 +0,0 @@
-#!/bin/sh
-#
-# Copyright (C) 2018 OpenWrt.org
-# Copyright (C) 2018 rosysong@rosinson.com
-#
-
-. /lib/functions.sh
-
-FLAG_DEV_TYPE=
-FLAG_DEV_MOPT=
-FLAG_HAS_SECT=
-
-samba_dev_filter() { # <devname> <[path,/dev/]>
-	case $1 in
-	${2}mtdblock*|\
-	${2}ubi*)
-		FLAG_DEV_TYPE="mtd"
-		;;
-	${2}loop*|\
-	${2}mmcblk*|\
-	${2}sd*|\
-	${2}hd*|\
-	${2}md*|\
-	${2}nvme*|\
-	${2}vd*|\
-	${2}xvd*)
-		FLAG_DEV_TYPE="not-mtd"
-		;;
-	*)
-		[ -b ${2}${1} ] && FLAG_DEV_TYPE="not-mtd"
-		[ -b /dev/mapper/$1 ] && FLAG_DEV_TYPE="not-mtd"
-		;;
-	esac
-}
-
-samba_cfg_lookup() { # <section> <name>
-	config_get name $1 name
-	[ "$name" = "$2" ] || return
-	FLAG_HAS_SECT=y
-}
-
-samba_cfg_delete() { # <section> <name>
-	config_get name $1 name
-	[ "$name" = "$2" ] || return
-	uci -q delete samba.$1
-}
-
-samba_find_mount_point() { # <devname>
-	# search mount point in /proc/mounts
-	while read l; do
-		local d=$(echo $l | awk '/^\/dev/ {print $1}')
-		[ "$d" = "/dev/$1" ] || continue
-
-		FLAG_DEV_MOPT=$(echo $l | awk '/^\/dev/ {print $2}')
-		break
-	done < /proc/mounts
-}
-
-_samba_add_section() { # <devname> <mount point>
-	uci -q batch <<-EOF
-		add samba sambashare
-		set samba.@sambashare[-1].browseable='yes'
-		set samba.@sambashare[-1].name='$1'
-		set samba.@sambashare[-1].path='$2'
-		set samba.@sambashare[-1].users='root'
-		set samba.@sambashare[-1].read_only='no'
-		set samba.@sambashare[-1].guest_ok='yes'
-		set samba.@sambashare[-1].create_mask='0755'
-		set samba.@sambashare[-1].dir_mask='0755'
-	EOF
-}
-
-samba_add_section() { # <devname> [<mount point>]
-	FLAG_HAS_SECT=
-	FLAG_DEV_MOPT=
-
-	config_foreach samba_cfg_lookup sambashare $1
-	[ -z "$FLAG_HAS_SECT" ] || return
-
-	samba_find_mount_point $1
-	[ -n "$FLAG_DEV_MOPT" ] || return
-
-	[ -n "$2" -a "$2" = "$FLAG_DEV_MOPT" ] || \
-		_samba_add_section $1 $FLAG_DEV_MOPT
-}
-
-samba_delete_section() { # <devname>
-	config_foreach samba_cfg_delete sambashare $1
-}
-- 
cgit v1.2.3