[Buildroot] [PATCH] package/minidlna/S60minidlnad: add force-reload to rebuild DB

[Buildroot] [PATCH] package/minidlna/S60minidlnad: add force-reload to rebuild DB

[Benoît Thébaudeau]

Add a force-reload operation that restarts minidlnad and makes it
rebuild its database. This is what Debian does, and this is useful when
media_dir is changed in the configuration file or when inotify can not
detect changes inside the media directories (e.g. in case of a mount).

Signed-off-by: Beno?t Th?baudeau <benoit@wsystem.com>
---
 package/minidlna/S60minidlnad | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/package/minidlna/S60minidlnad b/package/minidlna/S60minidlnad
index f99999a..b4b62ba 100644
--- a/package/minidlna/S60minidlnad
+++ b/package/minidlna/S60minidlnad
@@ -7,7 +7,7 @@ start() {
 	echo -n "Starting minidlna: "
 	umask 077
 	start-stop-daemon -S -q -p /var/run/minidlna/minidlna.pid \
-		--exec /usr/sbin/minidlnad
+		--exec /usr/sbin/minidlnad -- "$@"
 	[ $? = 0 ] && echo "OK" || echo "FAIL"
 }
 stop() {
@@ -19,7 +19,7 @@ restart() {
 	stop
 	# Sleep needed for minidlna to restart properly
 	sleep 1
-	start
+	start "$@"
 }
 
 case "$1" in
@@ -32,8 +32,11 @@ case "$1" in
   restart|reload)
 	restart
 	;;
+  force-reload)
+	restart -R
+	;;
   *)
-	echo "Usage: $0 {start|stop|restart}"
+	echo "Usage: $0 {start|stop|restart|force-reload}"
 	exit 1
 esac
 
-- 
2.1.4

[Buildroot] [PATCH] package/minidlna/S60minidlnad: add force-reload to rebuild DB

[Thomas Petazzoni]

Dear Beno?t Th?baudeau,

On Fri, 21 Aug 2015 19:55:18 +0200, Beno?t Th?baudeau wrote:
> Add a force-reload operation that restarts minidlnad and makes it
> rebuild its database. This is what Debian does, and this is useful when
> media_dir is changed in the configuration file or when inotify can not
> detect changes inside the media directories (e.g. in case of a mount).
> 
> Signed-off-by: Beno?t Th?baudeau <benoit@wsystem.com>
> ---
>  package/minidlna/S60minidlnad | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)

Applied, thanks. I've mistakenly applied it to master, my intention was
to apply it to next. But now that it is pushed, it is too late. Anyway,
not a big deal, since your patch can hardly introduce a regression.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com