[meta-oe][PATCH 0/3] add dlm and it's dependencies

[meta-oe][PATCH 0/3] add dlm and it's dependencies

[leimaohui]

From: leimh <leimaohui@cn.fujitsu.com>

Add dlm control daemon and tool because the kernel dlm requires a user daemon to control membership.
corosync : the dependency of dlm
libqb: the dependency of corosync

leimh (3):
  libqb: add new recipe
  corosync_2.3.4: add new recipe
  dlm: add a new recipe

 .../recipes-extended/corosync/corosync_2.3.4.bb    | 59 ++++++++++++++++++++++
 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb          | 47 +++++++++++++++++
 meta-oe/recipes-extended/libqb/libqb_0.17.1.bb     | 26 ++++++++++
 3 files changed, 132 insertions(+)
 create mode 100644 meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
 create mode 100644 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
 create mode 100644 meta-oe/recipes-extended/libqb/libqb_0.17.1.bb

-- 
1.8.4.2

[meta-oe][PATCH 1/3] libqb: add new recipe

[leimaohui]

From: leimh <leimaohui@cn.fujitsu.com>

An IPC library for high performance servers

Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
---
 meta-oe/recipes-extended/libqb/libqb_0.17.1.bb | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)
 create mode 100644 meta-oe/recipes-extended/libqb/libqb_0.17.1.bb

diff --git a/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb b/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb
new file mode 100644
index 0000000..283f1d1
--- /dev/null
+++ b/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb
@@ -0,0 +1,26 @@
+SUMMARY = "An IPC library for high performance servers"
+DESCRIPTION = "libqb is a library with the primary purpose of providing high performance client server reusable features. \
+It provides high performance logging, tracing, ipc, and poll."
+
+HOMEPAGE = "https://github.com/clusterlabs/libqb/wiki"
+
+SECTION = "libs"
+
+inherit autotools pkgconfig
+
+SRC_URI = "https://fedorahosted.org/releases/q/u/quarterback/libqb-${PV}.tar.xz \
+          "
+
+SRC_URI[md5sum] = "5770b343baa4528f6fec90120ec55048"
+SRC_URI[sha256sum] = "7a2115f83bfe20eaa5f2e4ed235e8f2994235d3b87e3e5ca41ba47b320f12e29"
+
+LICENSE = "LGPL-2.1"
+
+LIC_FILES_CHKSUM = "file://COPYING;md5=321bf41f280cf805086dd5a720b37785"
+
+do_configure_prepend() {
+    ( cd ${S}
+      ${S}/autogen.sh )
+}
+
+
-- 
1.8.4.2

[meta-oe][PATCH 2/3] corosync_2.3.4: add new recipe

[leimaohui]

From: leimh <leimaohui@cn.fujitsu.com>

The Corosync Cluster Engine and Application Programming Interfaces

Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
---
 .../recipes-extended/corosync/corosync_2.3.4.bb    | 59 ++++++++++++++++++++++
 1 file changed, 59 insertions(+)
 create mode 100644 meta-oe/recipes-extended/corosync/corosync_2.3.4.bb

diff --git a/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb b/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
new file mode 100644
index 0000000..7a09862
--- /dev/null
+++ b/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
@@ -0,0 +1,59 @@
+SUMMARY = "The Corosync Cluster Engine and Application Programming Interfaces"
+DESCRIPTION = "This package contains the Corosync Cluster Engine Executive, several default \
+APIs and libraries, default configuration files, and an init script."
+HOMEPAGE = "http://corosync.github.io/corosync/"
+
+SECTION = "base"
+
+inherit autotools pkgconfig systemd
+
+SRC_URI = "http://build.clusterlabs.org/corosync/releases/${BPN}-${PV}.tar.gz \
+	   "
+
+SRC_URI[md5sum] = "4b0f36a1dc014527e5b192265dbd7e70"
+SRC_URI[sha256sum] = "3dae93fb1cf5c560295253b0560cbc25421ed053ee373852864f3a60c03247d4"
+
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=a85eb4ce24033adb6088dd1d6ffc5e5d"
+
+DEPENDS = "groff-native nss libqb"
+
+SYSTEMD_SERVICE_${PN} = "corosync.service corosync-notifyd.service"
+SYSTEMD_AUTO_ENABLE = "enable"
+
+INITSCRIPT_NAME = "corosync-daemon"
+
+PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'systemd', '', d)}"
+
+PACKAGECONFIG[systemd] = "--enable-systemd --with-systemddir=${systemd_unitdir}/system/,--with-systemddir="
+
+EXTRA_OECONF = "--disable-nss \
+                --with-upstartdir=%{_sysconfdir}/init \
+"
+EXTRA_OECONF += " --enable-nss "
+
+do_configure_prepend() {
+    ( cd ${S}
+    ${S}/autogen.sh )
+}
+
+do_install_append() {
+        install -d ${D}${sysconfdir}/sysconfig/
+	install -d ${D}/${sysconfdir}/init.d
+	install -m 0644 ${S}/init/corosync.sysconfig.example ${D}${sysconfdir}/sysconfig/corosync
+	install -m 0644 ${S}/init/corosync-notifyd.conf.in ${D}${sysconfdir}/sysconfig/corosync-notifyd.conf
+	install -m 0644 ${S}/init/corosync.conf.in ${D}${sysconfdir}/sysconfig/corosync.conf
+	install -m 0644 ${S}/init/corosync.in ${D}${sysconfdir}/init.d/corosync
+	install -m 0644 ${S}/init/corosync-notifyd.in ${D}${sysconfdir}/init.d/corosync-notifyd
+
+	if ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
+		install -d ${D}${systemd_unitdir}/system
+		install -m 0644 ${S}/init/corosync.service.in ${D}${systemd_unitdir}/system/corosync.service
+		install -m 0644 ${S}/init/corosync-notifyd.service.in ${D}${systemd_unitdir}/system/corosync-notifyd.service
+		sed -i -e 's,@INITWRAPPERSDIR@,${sysconfdir}/init.d,g' ${D}${systemd_unitdir}/system/corosync.service
+		sed -i -e 's,@SYSCONFDIR@,${sysconfdir},g' ${D}${systemd_unitdir}/system/corosync-notifyd.service
+		sed -i -e 's,@SBINDIR@,${base_sbindir},g' ${D}${systemd_unitdir}/system/corosync-notifyd.service
+	fi	
+}
+
+FILES_${PN}-dbg += "${libexecdir}/lcrso/.debug"
-- 
1.8.4.2

[meta-oe][PATCH 3/3] dlm: add a new recipe

[leimaohui]

From: leimh <leimaohui@cn.fujitsu.com>

The kernel dlm requires a user daemon to control membership.

Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
---
 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb | 47 +++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)
 create mode 100644 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb

diff --git a/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
new file mode 100644
index 0000000..b46c357
--- /dev/null
+++ b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
@@ -0,0 +1,47 @@
+DESCRIPTION = "dlm control daemon and tool"
+
+SECTION = "utils"
+HOMEPAGE = "https://fedorahosted.org/cluster/wiki/HomePage"
+
+SRC_URI = "https://git.fedorahosted.org/cgit/dlm.git/snapshot/dlm-${PV}.tar.xz"
+
+SRC_URI[md5sum] = "efc2ee6093aa6aa0a88aaad83e998a3f"
+SRC_URI[sha256sum] = "b89bc557aaffbab0ac005398025f247718a5589cff6574d902eaffe2b20e683e"
+
+LICENSE = "LGPLv2+&GPLv2&GPLv2+"
+LIC_FILES_CHKSUM = "file://README.license;md5=8f0bbcdd678df1bce9863492b6c8832d"
+
+DEPENDS = "corosync systemd"
+
+inherit pkgconfig systemd
+
+SYSTEMD_SERVICE_${PN} = "dlm.service"
+SYSTEMD_AUTO_ENABLE = "enable"
+
+export EXTRA_OEMAKE = ""
+
+do_compile_prepend() {
+    sed -i "s/libsystemd-daemon/libsystemd/g" ${S}/dlm_controld/Makefile
+}
+
+do_compile () {
+    oe_runmake 'CC=${CC}'
+}
+
+do_install_append (){
+	install -d ${D}${sysconfdir}/sysconfig/
+	install -d ${D}${sysconfdir}/init.d/
+	install -m 0644 ${S}/init/dlm.sysconfig ${D}${sysconfdir}/sysconfig/dlm
+	install -m 0644 ${S}/init/dlm.init ${D}${sysconfdir}/init.d/dlm
+
+	# install systemd unit files
+	if ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
+		install -d ${D}${systemd_unitdir}/system
+		install -m 0644 ${S}/init/dlm.service ${D}${systemd_unitdir}/system
+	fi
+}
+
+do_install() {
+    oe_runmake install DESTDIR=${D} LIBDIR=${libdir}
+}
+
-- 
1.8.4.2

Re: [meta-oe][PATCH 1/3] libqb: add new recipe

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1833 bytes --]

On Mon, Jan 05, 2015 at 03:19:52PM +0800, leimaohui wrote:
> From: leimh <leimaohui@cn.fujitsu.com>
> 
> An IPC library for high performance servers
> 
> Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
> ---
>  meta-oe/recipes-extended/libqb/libqb_0.17.1.bb | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/libqb/libqb_0.17.1.bb
> 
> diff --git a/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb b/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb
> new file mode 100644
> index 0000000..283f1d1
> --- /dev/null
> +++ b/meta-oe/recipes-extended/libqb/libqb_0.17.1.bb
> @@ -0,0 +1,26 @@
> +SUMMARY = "An IPC library for high performance servers"
> +DESCRIPTION = "libqb is a library with the primary purpose of providing high performance client server reusable features. \
> +It provides high performance logging, tracing, ipc, and poll."
> +
> +HOMEPAGE = "https://github.com/clusterlabs/libqb/wiki"
> +
> +SECTION = "libs"
> +
> +inherit autotools pkgconfig
> +
> +SRC_URI = "https://fedorahosted.org/releases/q/u/quarterback/libqb-${PV}.tar.xz \
> +          "

libqb-${PV} -> ${BP}

> +
> +SRC_URI[md5sum] = "5770b343baa4528f6fec90120ec55048"
> +SRC_URI[sha256sum] = "7a2115f83bfe20eaa5f2e4ed235e8f2994235d3b87e3e5ca41ba47b320f12e29"
> +
> +LICENSE = "LGPL-2.1"
> +
> +LIC_FILES_CHKSUM = "file://COPYING;md5=321bf41f280cf805086dd5a720b37785"
> +
> +do_configure_prepend() {
> +    ( cd ${S}
> +      ${S}/autogen.sh )
> +}
> +
> +
> -- 
> 1.8.4.2
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

Re: [meta-oe][PATCH 2/3] corosync_2.3.4: add new recipe

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 3757 bytes --]

On Mon, Jan 05, 2015 at 03:19:53PM +0800, leimaohui wrote:
> From: leimh <leimaohui@cn.fujitsu.com>
> 
> The Corosync Cluster Engine and Application Programming Interfaces
> 
> Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
> ---
>  .../recipes-extended/corosync/corosync_2.3.4.bb    | 59 ++++++++++++++++++++++
>  1 file changed, 59 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
> 
> diff --git a/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb b/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
> new file mode 100644
> index 0000000..7a09862
> --- /dev/null
> +++ b/meta-oe/recipes-extended/corosync/corosync_2.3.4.bb
> @@ -0,0 +1,59 @@
> +SUMMARY = "The Corosync Cluster Engine and Application Programming Interfaces"
> +DESCRIPTION = "This package contains the Corosync Cluster Engine Executive, several default \
> +APIs and libraries, default configuration files, and an init script."
> +HOMEPAGE = "http://corosync.github.io/corosync/"
> +
> +SECTION = "base"
> +
> +inherit autotools pkgconfig systemd
> +
> +SRC_URI = "http://build.clusterlabs.org/corosync/releases/${BPN}-${PV}.tar.gz \
> +	   "

${BPN}-${PV} -> ${BP}

> +
> +SRC_URI[md5sum] = "4b0f36a1dc014527e5b192265dbd7e70"
> +SRC_URI[sha256sum] = "3dae93fb1cf5c560295253b0560cbc25421ed053ee373852864f3a60c03247d4"
> +
> +LICENSE = "BSD"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=a85eb4ce24033adb6088dd1d6ffc5e5d"
> +
> +DEPENDS = "groff-native nss libqb"
> +
> +SYSTEMD_SERVICE_${PN} = "corosync.service corosync-notifyd.service"
> +SYSTEMD_AUTO_ENABLE = "enable"
> +
> +INITSCRIPT_NAME = "corosync-daemon"
> +
> +PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'systemd', '', d)}"
> +
> +PACKAGECONFIG[systemd] = "--enable-systemd --with-systemddir=${systemd_unitdir}/system/,--with-systemddir="
> +
> +EXTRA_OECONF = "--disable-nss \
> +                --with-upstartdir=%{_sysconfdir}/init \
> +"
> +EXTRA_OECONF += " --enable-nss "
> +
> +do_configure_prepend() {
> +    ( cd ${S}
> +    ${S}/autogen.sh )
> +}
> +
> +do_install_append() {
> +        install -d ${D}${sysconfdir}/sysconfig/
> +	install -d ${D}/${sysconfdir}/init.d
> +	install -m 0644 ${S}/init/corosync.sysconfig.example ${D}${sysconfdir}/sysconfig/corosync
> +	install -m 0644 ${S}/init/corosync-notifyd.conf.in ${D}${sysconfdir}/sysconfig/corosync-notifyd.conf
> +	install -m 0644 ${S}/init/corosync.conf.in ${D}${sysconfdir}/sysconfig/corosync.conf
> +	install -m 0644 ${S}/init/corosync.in ${D}${sysconfdir}/init.d/corosync
> +	install -m 0644 ${S}/init/corosync-notifyd.in ${D}${sysconfdir}/init.d/corosync-notifyd
> +
> +	if ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
> +		install -d ${D}${systemd_unitdir}/system
> +		install -m 0644 ${S}/init/corosync.service.in ${D}${systemd_unitdir}/system/corosync.service
> +		install -m 0644 ${S}/init/corosync-notifyd.service.in ${D}${systemd_unitdir}/system/corosync-notifyd.service
> +		sed -i -e 's,@INITWRAPPERSDIR@,${sysconfdir}/init.d,g' ${D}${systemd_unitdir}/system/corosync.service
> +		sed -i -e 's,@SYSCONFDIR@,${sysconfdir},g' ${D}${systemd_unitdir}/system/corosync-notifyd.service
> +		sed -i -e 's,@SBINDIR@,${base_sbindir},g' ${D}${systemd_unitdir}/system/corosync-notifyd.service
> +	fi	
> +}

Use 4 spaces for indentation.

> +
> +FILES_${PN}-dbg += "${libexecdir}/lcrso/.debug"
> -- 
> 1.8.4.2
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

Re: [meta-oe][PATCH 3/3] dlm: add a new recipe

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 2604 bytes --]

On Mon, Jan 05, 2015 at 03:19:54PM +0800, leimaohui wrote:
> From: leimh <leimaohui@cn.fujitsu.com>
> 
> The kernel dlm requires a user daemon to control membership.
> 
> Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
> ---
>  meta-oe/recipes-extended/dlm/dlm_4.0.2.bb | 47 +++++++++++++++++++++++++++++++
>  1 file changed, 47 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> 
> diff --git a/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> new file mode 100644
> index 0000000..b46c357
> --- /dev/null
> +++ b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> @@ -0,0 +1,47 @@
> +DESCRIPTION = "dlm control daemon and tool"
> +
> +SECTION = "utils"
> +HOMEPAGE = "https://fedorahosted.org/cluster/wiki/HomePage"
> +
> +SRC_URI = "https://git.fedorahosted.org/cgit/dlm.git/snapshot/dlm-${PV}.tar.xz"

dlm-${PV} -> ${BP} but more importantly is dlm.git/snapshots "stable"?
Doesn't it re-generated sometimes ?
> +
> +SRC_URI[md5sum] = "efc2ee6093aa6aa0a88aaad83e998a3f"
> +SRC_URI[sha256sum] = "b89bc557aaffbab0ac005398025f247718a5589cff6574d902eaffe2b20e683e"
> +
> +LICENSE = "LGPLv2+&GPLv2&GPLv2+"

Add spaces around &

> +LIC_FILES_CHKSUM = "file://README.license;md5=8f0bbcdd678df1bce9863492b6c8832d"
> +
> +DEPENDS = "corosync systemd"
> +
> +inherit pkgconfig systemd
> +
> +SYSTEMD_SERVICE_${PN} = "dlm.service"
> +SYSTEMD_AUTO_ENABLE = "enable"
> +
> +export EXTRA_OEMAKE = ""
> +
> +do_compile_prepend() {
> +    sed -i "s/libsystemd-daemon/libsystemd/g" ${S}/dlm_controld/Makefile
> +}
> +
> +do_compile () {
> +    oe_runmake 'CC=${CC}'
> +}
> +
> +do_install_append (){
> +	install -d ${D}${sysconfdir}/sysconfig/
> +	install -d ${D}${sysconfdir}/init.d/
> +	install -m 0644 ${S}/init/dlm.sysconfig ${D}${sysconfdir}/sysconfig/dlm
> +	install -m 0644 ${S}/init/dlm.init ${D}${sysconfdir}/init.d/dlm
> +
> +	# install systemd unit files
> +	if ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
> +		install -d ${D}${systemd_unitdir}/system
> +		install -m 0644 ${S}/init/dlm.service ${D}${systemd_unitdir}/system
> +	fi
> +}

Use 4 spaces for indentation

> +
> +do_install() {
> +    oe_runmake install DESTDIR=${D} LIBDIR=${libdir}
> +}
> +
> -- 
> 1.8.4.2
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

Re: [meta-oe][PATCH 3/3] dlm: add a new recipe

[Lei, Maohui]

Hi Martin

> > +SRC_URI =
> "https://git.fedorahosted.org/cgit/dlm.git/snapshot/dlm-${PV}.tar.xz"
> 
> dlm-${PV} -> ${BP} but more importantly is dlm.git/snapshots "stable"?
> Doesn't it re-generated sometimes ?

The download URL comes from https://git.fedorahosted.org/cgit/dlm.git/.
Besides the latest version, the old versions are also provided by "dlm.git/snapshots".
Viewed the generated time of the old versions, I think it is "stable" enough.
And you know, dlm comes from fedora, I can't find a better download URL.


Cheers

Lei



> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> Martin Jansa
> Sent: Monday, January 05, 2015 5:03 PM
> To: openembedded-devel@lists.openembedded.org
> Subject: Re: [oe] [meta-oe][PATCH 3/3] dlm: add a new recipe
> 
> On Mon, Jan 05, 2015 at 03:19:54PM +0800, leimaohui wrote:
> > From: leimh <leimaohui@cn.fujitsu.com>
> >
> > The kernel dlm requires a user daemon to control membership.
> >
> > Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
> > ---
> >  meta-oe/recipes-extended/dlm/dlm_4.0.2.bb | 47
> > +++++++++++++++++++++++++++++++
> >  1 file changed, 47 insertions(+)
> >  create mode 100644 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> >
> > diff --git a/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > new file mode 100644
> > index 0000000..b46c357
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > @@ -0,0 +1,47 @@
> > +DESCRIPTION = "dlm control daemon and tool"
> > +
> > +SECTION = "utils"
> > +HOMEPAGE = "https://fedorahosted.org/cluster/wiki/HomePage"
> > +
> > +SRC_URI =
> "https://git.fedorahosted.org/cgit/dlm.git/snapshot/dlm-${PV}.tar.xz"
> 
> dlm-${PV} -> ${BP} but more importantly is dlm.git/snapshots "stable"?
> Doesn't it re-generated sometimes ?
> > +
> > +SRC_URI[md5sum] = "efc2ee6093aa6aa0a88aaad83e998a3f"
> > +SRC_URI[sha256sum] =
> "b89bc557aaffbab0ac005398025f247718a5589cff6574d902eaffe2b20e683e"
> > +
> > +LICENSE = "LGPLv2+&GPLv2&GPLv2+"
> 
> Add spaces around &
> 
> > +LIC_FILES_CHKSUM =
> "file://README.license;md5=8f0bbcdd678df1bce9863492b6c8832d"
> > +
> > +DEPENDS = "corosync systemd"
> > +
> > +inherit pkgconfig systemd
> > +
> > +SYSTEMD_SERVICE_${PN} = "dlm.service"
> > +SYSTEMD_AUTO_ENABLE = "enable"
> > +
> > +export EXTRA_OEMAKE = ""
> > +
> > +do_compile_prepend() {
> > +    sed -i "s/libsystemd-daemon/libsystemd/g"
> > +${S}/dlm_controld/Makefile }
> > +
> > +do_compile () {
> > +    oe_runmake 'CC=${CC}'
> > +}
> > +
> > +do_install_append (){
> > +	install -d ${D}${sysconfdir}/sysconfig/
> > +	install -d ${D}${sysconfdir}/init.d/
> > +	install -m 0644 ${S}/init/dlm.sysconfig
> ${D}${sysconfdir}/sysconfig/dlm
> > +	install -m 0644 ${S}/init/dlm.init ${D}${sysconfdir}/init.d/dlm
> > +
> > +	# install systemd unit files
> > +	if
> ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
> > +		install -d ${D}${systemd_unitdir}/system
> > +		install -m 0644 ${S}/init/dlm.service
> ${D}${systemd_unitdir}/system
> > +	fi
> > +}
> 
> Use 4 spaces for indentation
> 
> > +
> > +do_install() {
> > +    oe_runmake install DESTDIR=${D} LIBDIR=${libdir} }
> > +
> > --
> > 1.8.4.2
> >
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

Re: [meta-oe][PATCH 3/3] dlm: add a new recipe

[Lei, Maohui]

Hi Martin

   I will modify my patches, and then submit V2 patches later.


Cheers
Lei

> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of
> Martin Jansa
> Sent: Monday, January 05, 2015 5:03 PM
> To: openembedded-devel@lists.openembedded.org
> Subject: Re: [oe] [meta-oe][PATCH 3/3] dlm: add a new recipe
> 
> On Mon, Jan 05, 2015 at 03:19:54PM +0800, leimaohui wrote:
> > From: leimh <leimaohui@cn.fujitsu.com>
> >
> > The kernel dlm requires a user daemon to control membership.
> >
> > Signed-off-by: leimh <leimaohui@cn.fujitsu.com>
> > ---
> >  meta-oe/recipes-extended/dlm/dlm_4.0.2.bb | 47
> > +++++++++++++++++++++++++++++++
> >  1 file changed, 47 insertions(+)
> >  create mode 100644 meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> >
> > diff --git a/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > new file mode 100644
> > index 0000000..b46c357
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/dlm/dlm_4.0.2.bb
> > @@ -0,0 +1,47 @@
> > +DESCRIPTION = "dlm control daemon and tool"
> > +
> > +SECTION = "utils"
> > +HOMEPAGE = "https://fedorahosted.org/cluster/wiki/HomePage"
> > +
> > +SRC_URI =
> "https://git.fedorahosted.org/cgit/dlm.git/snapshot/dlm-${PV}.tar.xz"
> 
> dlm-${PV} -> ${BP} but more importantly is dlm.git/snapshots "stable"?
> Doesn't it re-generated sometimes ?
> > +
> > +SRC_URI[md5sum] = "efc2ee6093aa6aa0a88aaad83e998a3f"
> > +SRC_URI[sha256sum] =
> "b89bc557aaffbab0ac005398025f247718a5589cff6574d902eaffe2b20e683e"
> > +
> > +LICENSE = "LGPLv2+&GPLv2&GPLv2+"
> 
> Add spaces around &
> 
> > +LIC_FILES_CHKSUM =
> "file://README.license;md5=8f0bbcdd678df1bce9863492b6c8832d"
> > +
> > +DEPENDS = "corosync systemd"
> > +
> > +inherit pkgconfig systemd
> > +
> > +SYSTEMD_SERVICE_${PN} = "dlm.service"
> > +SYSTEMD_AUTO_ENABLE = "enable"
> > +
> > +export EXTRA_OEMAKE = ""
> > +
> > +do_compile_prepend() {
> > +    sed -i "s/libsystemd-daemon/libsystemd/g"
> > +${S}/dlm_controld/Makefile }
> > +
> > +do_compile () {
> > +    oe_runmake 'CC=${CC}'
> > +}
> > +
> > +do_install_append (){
> > +	install -d ${D}${sysconfdir}/sysconfig/
> > +	install -d ${D}${sysconfdir}/init.d/
> > +	install -m 0644 ${S}/init/dlm.sysconfig
> ${D}${sysconfdir}/sysconfig/dlm
> > +	install -m 0644 ${S}/init/dlm.init ${D}${sysconfdir}/init.d/dlm
> > +
> > +	# install systemd unit files
> > +	if
> ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
> > +		install -d ${D}${systemd_unitdir}/system
> > +		install -m 0644 ${S}/init/dlm.service
> ${D}${systemd_unitdir}/system
> > +	fi
> > +}
> 
> Use 4 spaces for indentation
> 
> > +
> > +do_install() {
> > +    oe_runmake install DESTDIR=${D} LIBDIR=${libdir} }
> > +
> > --
> > 1.8.4.2
> >
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com