aboutsummaryrefslogtreecommitdiffstats
path: root/package
diff options
context:
space:
mode:
authorRafał Miłecki <rafal@milecki.pl>2018-12-25 18:46:44 +0100
committerRafał Miłecki <rafal@milecki.pl>2018-12-28 23:09:38 +0100
commitae622c93b3b4df7a55ef2cdb9986eec2bf42159b (patch)
tree91a41047e9e26ca08fba366c06d82918e7934c2c /package
parent3c899aaf11580353bc9ab1c3d9c3f638db80ba7c (diff)
downloadupstream-ae622c93b3b4df7a55ef2cdb9986eec2bf42159b.tar.gz
upstream-ae622c93b3b4df7a55ef2cdb9986eec2bf42159b.tar.bz2
upstream-ae622c93b3b4df7a55ef2cdb9986eec2bf42159b.zip
Revert "samba36: add hotplug support"
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>
Diffstat (limited to 'package')
-rw-r--r--package/network/services/samba36/Makefile14
-rw-r--r--package/network/services/samba36/files/lib/samba.sh89
-rw-r--r--package/network/services/samba36/files/samba.hotplug11
3 files changed, 0 insertions, 114 deletions
diff --git a/package/network/services/samba36/Makefile b/package/network/services/samba36/Makefile
index d38e942074..b63c2eaad3 100644
--- a/package/network/services/samba36/Makefile
+++ b/package/network/services/samba36/Makefile
@@ -43,12 +43,6 @@ define Package/samba36-server
DEPENDS:=+USE_GLIBC:librt $(ICONV_DEPENDS)
endef
-define Package/samba36-hotplug
- $(call Package/samba/Default)
- TITLE+= hotplug
- DEPENDS:=+block-mount
-endef
-
define Package/samba36-client
$(call Package/samba/Default)
TITLE+= client
@@ -167,13 +161,6 @@ define Package/samba36-server/install
$(LN) samba_multicall $(1)/usr/sbin/smbpasswd
endef
-define Package/samba36-hotplug/install
- $(INSTALL_DIR) $(1)/lib/samba
- $(INSTALL_CONF) ./files/lib/samba.sh $(1)/lib/samba/samba.sh
- $(INSTALL_DIR) $(1)/etc/hotplug.d/block
- $(INSTALL_CONF) ./files/samba.hotplug $(1)/etc/hotplug.d/block/60-samba
-endef
-
define Package/samba36-client/install
$(INSTALL_DIR) $(1)/usr/sbin
$(INSTALL_BIN) $(PKG_BUILD_BIN)/smbclient $(1)/usr/sbin
@@ -187,6 +174,5 @@ endef
$(eval $(call BuildPackage,samba36-client))
$(eval $(call BuildPackage,samba36-server))
-$(eval $(call BuildPackage,samba36-hotplug))
$(eval $(call BuildPackage,samba36-net))
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
-}
diff --git a/package/network/services/samba36/files/samba.hotplug b/package/network/services/samba36/files/samba.hotplug
deleted file mode 100644
index 9c6dbe5e41..0000000000
--- a/package/network/services/samba36/files/samba.hotplug
+++ /dev/null
@@ -1,11 +0,0 @@
-. /lib/samba/samba.sh
-
-samba_dev_filter $DEVNAME
-[ "$FLAG_DEV_TYPE" = "not-mtd" ] || exit
-
-config_load samba
-case $ACTION in
- add) samba_add_section $DEVNAME;;
- remove) samba_delete_section $DEVNAME;;
-esac
-uci commit samba