Re: [oe] [meta-oe][PATCH] minidlna: Fix package and bump to version 1.1.4

2015-03-05 Thread Andreas Oberritter
Hello Baptiste,

On 04.03.2015 13:56, Baptiste DURAND wrote:
 Signed-off-by: Baptiste DURAND baptiste.dur...@gmail.com
 ---
  meta-oe/recipes-multimedia/minidlna/minidlna.inc   | 39 
  .../minidlna/minidlna/minidlna-daemon.init.d   | 54 
 ++
  .../minidlna/minidlna/minidlna-daemon.service  | 10 
  .../recipes-multimedia/minidlna/minidlna_1.1.4.bb  |  6 +++
  .../minidlna/minidlna/fix-makefile.patch   | 41 
  .../search-for-headers-in-staging-dir.patch| 48 ---
  .../nonworking/minidlna/minidlna_1.0.25.bb | 19 
  7 files changed, 109 insertions(+), 108 deletions(-)
  create mode 100644 meta-oe/recipes-multimedia/minidlna/minidlna.inc
  create mode 100644 
 meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
  create mode 100644 
 meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
  create mode 100644 meta-oe/recipes-multimedia/minidlna/minidlna_1.1.4.bb
  delete mode 100644 
 meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/fix-makefile.patch
  delete mode 100644 
 meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/search-for-headers-in-staging-dir.patch
  delete mode 100644 
 meta-oe/recipes-multimedia/nonworking/minidlna/minidlna_1.0.25.bb
 
 diff --git a/meta-oe/recipes-multimedia/minidlna/minidlna.inc 
 b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
 new file mode 100644
 index 000..9d6568d
 --- /dev/null
 +++ b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
 @@ -0,0 +1,39 @@
 +DESCRIPTION = MiniDLNA (aka ReadyDLNA) is server software with the aim of \
 +being fully compliant with DLNA/UPnP-AV clients.
 +LICENSE = GPL-2.0|BSD
 +DEPENDS = virtual/gettext ffmpeg flac libav jpeg sqlite3 libexif libogg 
 libid3tag libvorbis
 +
 +inherit autotools update-rc.d systemd
 +
 +SRC_URI = git://git.code.sf.net/p/minidlna/git;branch=master;module=git
 +SRC_URI =+ ${@base_contains('DISTRO_FEATURES', 'systemd', 
 'file://minidlna-daemon.service', 'file://minidlna-daemon.init.d', d)}

DISTRO_FEATUES may contain both systemd and sysvinit at the same time.

 +
 +S = ${WORKDIR}/git
 +B = ${S}

inherit autotools-brokensep instead of setting B.

 +
 +do_configure_prepend() {
 +cd ${S}
 +./autogen.sh
 +}

Is this required?

 +
 +do_install_initd() {
 + install -d ${D}${sysconfdir}/init.d
 + install -m 0755 ${WORKDIR}/minidlna-daemon.init.d 
 ${D}${sysconfdir}/init.d/minidlna
 +}
 +
 +do_install_systemd() {
 + install -d ${D}${nonarch_base_libdir}/systemd/system
 + install -m 0755 ${WORKDIR}/minidlna-daemon.service 
 ${D}${nonarch_base_libdir}/systemd/system
 +}
 +
 +do_install_append(){
 + install -d ${D}${sysconfdir}
 + install -m 0755 minidlna.conf ${D}${sysconfdir}
 + ${@base_contains('DISTRO_FEATURES', 'systemd', 'do_install_systemd', 
 'do_install_initd', d)}
 +}
 +
 +SYSTEMD_SERVICE_${PN} = minidlna-daemon.service
 +
 +INITSCRIPT_NAME = minidlna
 +INITSCRIPT_PARAMS = defaults 90

If the systemd unit had the same basename as the init script, i.e.
minidlna.service, then systemd would be able to ignore the init script
automatically.

Regards,
Andreas

 +
 diff --git 
 a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d 
 b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
 new file mode 100644
 index 000..61b5b54
 --- /dev/null
 +++ b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
 @@ -0,0 +1,54 @@
 +#!/bin/sh
 +
 +NAME=minidlna
 +DAEMON=/usr/sbin/minidlnad
 +SCRIPTNAME=/etc/init.d/$NAME
 +PIDFILE=/var/run/$NAME.pid
 +CONF=/etc/$NAME.conf
 +ARGS=-f $CONF
 +
 +# Exit if the package is not installed
 +[ -x $DAEMON ] || exit 0
 +
 +start_function() {
 +
 +export PATH=$PWD:$PATH
 +
 +if [ -f ${PIDFILE} ]; then
 +echo $SCRIPTNAME already running with PID #`cat $PIDFILE` ( 
 according to ${PIDFILE} );
 +exit 0
 +fi
 +
 +$DAEMON $ARGS
 + 
 +pid=$!
 +
 +if [ $pid !=  ]; then
 +echo -n $pid  ${PIDFILE}
 +fi
 +} 
 +
 +stop_function() {
 +
 +export PATH=$PWD:$PATH
 +
 +if [ ! -e ${PIDFILE} ]; then
 +echo ${SCRIPTNAME} not running ( according to ${PIDFILE} );
 +exit 1;
 +fi
 +PID=`cat ${PIDFILE}`
 +kill -INT ${PID}
 +rm -f ${PIDFILE}
 +}
 +
 +case $1 in
 +start)
 +start_function
 +;;
 +stop)
 +stop_function
 +;;
 +*)
 +echo Usage: $0 {start | stop}
 +
 +esac
 diff --git 
 a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service 
 b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
 new file mode 100644
 index 000..656100b
 --- /dev/null
 +++ b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
 @@ -0,0 +1,10 @@
 +[Unit]
 +Description=Minidlna Daemon
 +After=network.target
 +
 +[Service]
 +Type=notify
 +ExecStart=/usr/sbin/minidlnad -f /etc/minidlna.conf
 +
 +[Install]
 

Re: [oe] [meta-oe][PATCH] minidlna: Fix package and bump to version 1.1.4

2015-03-05 Thread Baptiste Durand
Thanks for your comment , I will resend a updated patch according to it.
Regards,

Baptiste


2015-03-05 9:14 GMT+01:00 Andreas Oberritter o...@opendreambox.org:

 Hello Baptiste,

 On 04.03.2015 13:56, Baptiste DURAND wrote:
  Signed-off-by: Baptiste DURAND baptiste.dur...@gmail.com
  ---
   meta-oe/recipes-multimedia/minidlna/minidlna.inc   | 39 
   .../minidlna/minidlna/minidlna-daemon.init.d   | 54
 ++
   .../minidlna/minidlna/minidlna-daemon.service  | 10 
   .../recipes-multimedia/minidlna/minidlna_1.1.4.bb  |  6 +++
   .../minidlna/minidlna/fix-makefile.patch   | 41 
   .../search-for-headers-in-staging-dir.patch| 48
 ---
   .../nonworking/minidlna/minidlna_1.0.25.bb | 19 
   7 files changed, 109 insertions(+), 108 deletions(-)
   create mode 100644 meta-oe/recipes-multimedia/minidlna/minidlna.inc
   create mode 100644
 meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
   create mode 100644
 meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
   create mode 100644 meta-oe/recipes-multimedia/minidlna/
 minidlna_1.1.4.bb
   delete mode 100644
 meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/fix-makefile.patch
   delete mode 100644
 meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/search-for-headers-in-staging-dir.patch
   delete mode 100644 meta-oe/recipes-multimedia/nonworking/minidlna/
 minidlna_1.0.25.bb
 
  diff --git a/meta-oe/recipes-multimedia/minidlna/minidlna.inc
 b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
  new file mode 100644
  index 000..9d6568d
  --- /dev/null
  +++ b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
  @@ -0,0 +1,39 @@
  +DESCRIPTION = MiniDLNA (aka ReadyDLNA) is server software with the aim
 of \
  +being fully compliant with DLNA/UPnP-AV clients.
  +LICENSE = GPL-2.0|BSD
  +DEPENDS = virtual/gettext ffmpeg flac libav jpeg sqlite3 libexif
 libogg libid3tag libvorbis
  +
  +inherit autotools update-rc.d systemd
  +
  +SRC_URI = git://
 git.code.sf.net/p/minidlna/git;branch=master;module=git
  +SRC_URI =+ ${@base_contains('DISTRO_FEATURES', 'systemd',
 'file://minidlna-daemon.service', 'file://minidlna-daemon.init.d', d)}

 DISTRO_FEATUES may contain both systemd and sysvinit at the same time.

  +
  +S = ${WORKDIR}/git
  +B = ${S}

 inherit autotools-brokensep instead of setting B.

  +
  +do_configure_prepend() {
  +cd ${S}
  +./autogen.sh
  +}

 Is this required?

  +
  +do_install_initd() {
  + install -d ${D}${sysconfdir}/init.d
  + install -m 0755 ${WORKDIR}/minidlna-daemon.init.d
 ${D}${sysconfdir}/init.d/minidlna
  +}
  +
  +do_install_systemd() {
  + install -d ${D}${nonarch_base_libdir}/systemd/system
  + install -m 0755 ${WORKDIR}/minidlna-daemon.service
 ${D}${nonarch_base_libdir}/systemd/system
  +}
  +
  +do_install_append(){
  + install -d ${D}${sysconfdir}
  + install -m 0755 minidlna.conf ${D}${sysconfdir}
  + ${@base_contains('DISTRO_FEATURES', 'systemd',
 'do_install_systemd', 'do_install_initd', d)}
  +}
  +
  +SYSTEMD_SERVICE_${PN} = minidlna-daemon.service
  +
  +INITSCRIPT_NAME = minidlna
  +INITSCRIPT_PARAMS = defaults 90

 If the systemd unit had the same basename as the init script, i.e.
 minidlna.service, then systemd would be able to ignore the init script
 automatically.

 Regards,
 Andreas

  +
  diff --git
 a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
 b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
  new file mode 100644
  index 000..61b5b54
  --- /dev/null
  +++ b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
  @@ -0,0 +1,54 @@
  +#!/bin/sh
  +
  +NAME=minidlna
  +DAEMON=/usr/sbin/minidlnad
  +SCRIPTNAME=/etc/init.d/$NAME
  +PIDFILE=/var/run/$NAME.pid
  +CONF=/etc/$NAME.conf
  +ARGS=-f $CONF
  +
  +# Exit if the package is not installed
  +[ -x $DAEMON ] || exit 0
  +
  +start_function() {
  +
  +export PATH=$PWD:$PATH
  +
  +if [ -f ${PIDFILE} ]; then
  +echo $SCRIPTNAME already running with PID #`cat $PIDFILE` (
 according to ${PIDFILE} );
  +exit 0
  +fi
  +
  +$DAEMON $ARGS
  +
  +pid=$!
  +
  +if [ $pid !=  ]; then
  +echo -n $pid  ${PIDFILE}
  +fi
  +}
  +
  +stop_function() {
  +
  +export PATH=$PWD:$PATH
  +
  +if [ ! -e ${PIDFILE} ]; then
  +echo ${SCRIPTNAME} not running ( according to ${PIDFILE} );
  +exit 1;
  +fi
  +PID=`cat ${PIDFILE}`
  +kill -INT ${PID}
  +rm -f ${PIDFILE}
  +}
  +
  +case $1 in
  +start)
  +start_function
  +;;
  +stop)
  +stop_function
  +;;
  +*)
  +echo Usage: $0 {start | stop}
  +
  +esac
  diff --git
 a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
 b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
  new file mode 100644
  index 000..656100b
  --- 

[oe] [meta-oe][PATCH] minidlna: Fix package and bump to version 1.1.4

2015-03-04 Thread Baptiste DURAND
Signed-off-by: Baptiste DURAND baptiste.dur...@gmail.com
---
 meta-oe/recipes-multimedia/minidlna/minidlna.inc   | 39 
 .../minidlna/minidlna/minidlna-daemon.init.d   | 54 ++
 .../minidlna/minidlna/minidlna-daemon.service  | 10 
 .../recipes-multimedia/minidlna/minidlna_1.1.4.bb  |  6 +++
 .../minidlna/minidlna/fix-makefile.patch   | 41 
 .../search-for-headers-in-staging-dir.patch| 48 ---
 .../nonworking/minidlna/minidlna_1.0.25.bb | 19 
 7 files changed, 109 insertions(+), 108 deletions(-)
 create mode 100644 meta-oe/recipes-multimedia/minidlna/minidlna.inc
 create mode 100644 
meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
 create mode 100644 
meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
 create mode 100644 meta-oe/recipes-multimedia/minidlna/minidlna_1.1.4.bb
 delete mode 100644 
meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/fix-makefile.patch
 delete mode 100644 
meta-oe/recipes-multimedia/nonworking/minidlna/minidlna/search-for-headers-in-staging-dir.patch
 delete mode 100644 
meta-oe/recipes-multimedia/nonworking/minidlna/minidlna_1.0.25.bb

diff --git a/meta-oe/recipes-multimedia/minidlna/minidlna.inc 
b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
new file mode 100644
index 000..9d6568d
--- /dev/null
+++ b/meta-oe/recipes-multimedia/minidlna/minidlna.inc
@@ -0,0 +1,39 @@
+DESCRIPTION = MiniDLNA (aka ReadyDLNA) is server software with the aim of \
+being fully compliant with DLNA/UPnP-AV clients.
+LICENSE = GPL-2.0|BSD
+DEPENDS = virtual/gettext ffmpeg flac libav jpeg sqlite3 libexif libogg 
libid3tag libvorbis
+
+inherit autotools update-rc.d systemd
+
+SRC_URI = git://git.code.sf.net/p/minidlna/git;branch=master;module=git
+SRC_URI =+ ${@base_contains('DISTRO_FEATURES', 'systemd', 
'file://minidlna-daemon.service', 'file://minidlna-daemon.init.d', d)}
+
+S = ${WORKDIR}/git
+B = ${S}
+
+do_configure_prepend() {
+cd ${S}
+./autogen.sh
+}
+
+do_install_initd() {
+   install -d ${D}${sysconfdir}/init.d
+   install -m 0755 ${WORKDIR}/minidlna-daemon.init.d 
${D}${sysconfdir}/init.d/minidlna
+}
+
+do_install_systemd() {
+   install -d ${D}${nonarch_base_libdir}/systemd/system
+   install -m 0755 ${WORKDIR}/minidlna-daemon.service 
${D}${nonarch_base_libdir}/systemd/system
+}
+
+do_install_append(){
+   install -d ${D}${sysconfdir}
+   install -m 0755 minidlna.conf ${D}${sysconfdir}
+   ${@base_contains('DISTRO_FEATURES', 'systemd', 'do_install_systemd', 
'do_install_initd', d)}
+}
+
+SYSTEMD_SERVICE_${PN} = minidlna-daemon.service
+
+INITSCRIPT_NAME = minidlna
+INITSCRIPT_PARAMS = defaults 90
+
diff --git 
a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d 
b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
new file mode 100644
index 000..61b5b54
--- /dev/null
+++ b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.init.d
@@ -0,0 +1,54 @@
+#!/bin/sh
+
+NAME=minidlna
+DAEMON=/usr/sbin/minidlnad
+SCRIPTNAME=/etc/init.d/$NAME
+PIDFILE=/var/run/$NAME.pid
+CONF=/etc/$NAME.conf
+ARGS=-f $CONF
+
+# Exit if the package is not installed
+[ -x $DAEMON ] || exit 0
+
+start_function() {
+
+export PATH=$PWD:$PATH
+
+if [ -f ${PIDFILE} ]; then
+echo $SCRIPTNAME already running with PID #`cat $PIDFILE` ( according 
to ${PIDFILE} );
+exit 0
+fi
+
+$DAEMON $ARGS
+ 
+pid=$!
+
+if [ $pid !=  ]; then
+echo -n $pid  ${PIDFILE}
+fi
+} 
+
+stop_function() {
+
+export PATH=$PWD:$PATH
+
+if [ ! -e ${PIDFILE} ]; then
+echo ${SCRIPTNAME} not running ( according to ${PIDFILE} );
+exit 1;
+fi
+PID=`cat ${PIDFILE}`
+kill -INT ${PID}
+rm -f ${PIDFILE}
+}
+
+case $1 in
+start)
+start_function
+;;
+stop)
+stop_function
+;;
+*)
+echo Usage: $0 {start | stop}
+
+esac
diff --git 
a/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service 
b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
new file mode 100644
index 000..656100b
--- /dev/null
+++ b/meta-oe/recipes-multimedia/minidlna/minidlna/minidlna-daemon.service
@@ -0,0 +1,10 @@
+[Unit]
+Description=Minidlna Daemon
+After=network.target
+
+[Service]
+Type=notify
+ExecStart=/usr/sbin/minidlnad -f /etc/minidlna.conf
+
+[Install]
+WantedBy=multi-user.target
diff --git a/meta-oe/recipes-multimedia/minidlna/minidlna_1.1.4.bb 
b/meta-oe/recipes-multimedia/minidlna/minidlna_1.1.4.bb
new file mode 100644
index 000..0531e69
--- /dev/null
+++ b/meta-oe/recipes-multimedia/minidlna/minidlna_1.1.4.bb
@@ -0,0 +1,6 @@
+require ${BPN}.inc
+
+SRCREV = v1_1_4
+LIC_FILES_CHKSUM = 
file://LICENCE.miniupnpd;md5=b0dabf9d8e0f871554e309d62ead8d2b
+SRC_URI[md5sum] = d966256baf2f9b068b9de871ab5dade5
+SRC_URI[sha256sum] =