Skip to contentSkip to conversation

package/minidlna/S60minidlnad: add force-reload to rebuild DB

PATCH New Benoît Thébaudeau 2 participants 2 messages Aug 21, 2015 – Nov 28, 2015

lore
Share & tools
lore

Benoît Thébaudeau

author
#0 lore raw

PATCH 1/1 package/minidlna/S60minidlnad: add force-reload to rebuild DB

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 <[email protected]>
---
 package/minidlna/S60minidlnad | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
package/minidlna/S60minidlnad +6 -3
index f99999a..b4b62ba 100644
@@ -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
Signature · 1 line
2.1.4
Applied
#1 lore raw

PATCH 1/1 package/minidlna/S60minidlnad: add force-reload to rebuild DB

Dear Beno?t Th?baudeau,

Quoted 9 lines · 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 <[email protected]>
---
 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

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