[PATCH] ell: Initial recipe for Embedded Linux Library

[PATCH] ell: Initial recipe for Embedded Linux Library

[Ankit Navik]

Add generic recipe for ELL.

Signed-off-by: Ankit Navik <ankit.tarot@gmail.com>
---
 meta-oe/recipes-connectivity/ell/ell_git.bb | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta-oe/recipes-connectivity/ell/ell_git.bb

diff --git a/meta-oe/recipes-connectivity/ell/ell_git.bb b/meta-oe/recipes-connectivity/ell/ell_git.bb
new file mode 100644
index 0000000..d259f52
--- /dev/null
+++ b/meta-oe/recipes-connectivity/ell/ell_git.bb
@@ -0,0 +1,17 @@
+SUMMARY  = "Embedded Linux Library"
+DESCRIPTION = "ELL is a DBUS library which provides DBUS bindings."
+LICENSE  = "LGPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
+SECTION = "libs"
+
+inherit autotools pkgconfig
+
+S = "${WORKDIR}/git"
+SRCREV = "399099946abaeb2f3718a0667cc11436a7c0be14"
+SRC_URI = "git://git.kernel.org/pub/scm/libs/ell/ell.git"
+
+do_configure_prepend () {
+    mkdir ${S}/build-aux
+}
+
+DEPENDS = "dbus"
-- 
1.9.1

Re: [PATCH] ell: Initial recipe for Embedded Linux Library

[Burton, Ross]

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

On 25 February 2017 at 15:47, Ankit Navik <ankit.tarot@gmail.com> wrote:

> I think, gdbus APIs will be replaced soon with ell APIs, I was the early
> adopter of ell APIs in OMA DM IMEI Sync (ODIS) Implementation.
> When I was implementing ODIS, there was license issue since gdbus uses GPL
> v2. I took help maintainer and they suggested ell. I remember they were
> mentioned that soon gdbus API will changes to ell in other components e.g.
> ofono, bluez, etc.
> I dont know thier exect timelines.
>

In that case ell can be considered for oe-core when a dependency of it
needs it - until then feel free to submit it to meta-oe or similar.

Ross

[-- Attachment #2: Type: text/html, Size: 1079 bytes --]

Re: [PATCH] ell: Initial recipe for Embedded Linux Library

[Burton, Ross]

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

On 24 February 2017 at 18:42, Ankit Navik <ankit.tarot@gmail.com> wrote:

> Add generic recipe for ELL.
>

This doesn't answer the question of "why?".

Ross

[-- Attachment #2: Type: text/html, Size: 587 bytes --]

[PATCH] ell: Initial recipe for Embedded Linux Library

[Ankit Navik]

Add generic recipe for ELL.

Signed-off-by: Ankit Navik <ankit.tarot@gmail.com>
---
 meta/recipes-core/ell/ell_git.bb | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta/recipes-core/ell/ell_git.bb

diff --git a/meta/recipes-core/ell/ell_git.bb b/meta/recipes-core/ell/ell_git.bb
new file mode 100644
index 0000000..d259f52
--- /dev/null
+++ b/meta/recipes-core/ell/ell_git.bb
@@ -0,0 +1,17 @@
+SUMMARY  = "Embedded Linux Library"
+DESCRIPTION = "ELL is a DBUS library which provides DBUS bindings."
+LICENSE  = "LGPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
+SECTION = "libs"
+
+inherit autotools pkgconfig
+
+S = "${WORKDIR}/git"
+SRCREV = "399099946abaeb2f3718a0667cc11436a7c0be14"
+SRC_URI = "git://git.kernel.org/pub/scm/libs/ell/ell.git"
+
+do_configure_prepend () {
+    mkdir ${S}/build-aux
+}
+
+DEPENDS = "dbus"
-- 
1.9.1

Re: [PATCH] ell: Initial recipe for Embedded Linux Library

[Burton, Ross]

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

On 24 February 2017 at 16:00, Ankit Navik <ankit.tarot@gmail.com> wrote:

>  meta/recipes-core/ell/ell.inc    | 10 ++++++++++
>  meta/recipes-core/ell/ell_git.bb |  9 +++++++++
>

Unless you have other recipes using the .inc file, then just merge these
into a single .bb.


> +DEPENDS  = "dbus"
> +RDEPENDS_${PN} += "dbus"
>

Why the RDEPENDS?  Any linking will add libdbus as required.


> +do_configure_prepend () {
> +    ${S}/bootstrap
> +}
>

This just does a mkdir and then calls autoreconf which we do straight away
afterwards, so is mostly a waste of time.  If we need to create the
directory before autoreconf then just do the mkdir in _prepend.

However currently this isn't used by anything in oe-core, so what's the
rationale for adding it to oe-core instead of another layer?

Ross

[-- Attachment #2: Type: text/html, Size: 1779 bytes --]

[PATCH] ell: Initial recipe for Embedded Linux Library

[Ankit Navik]

Based on the ofono recipe, add ell.inc for generic recipe components
and use it from ell_git.bb.

Signed-off-by: Ankit Navik <ankit.tarot@gmail.com>
---
 meta/recipes-core/ell/ell.inc    | 10 ++++++++++
 meta/recipes-core/ell/ell_git.bb |  9 +++++++++
 2 files changed, 19 insertions(+)
 create mode 100644 meta/recipes-core/ell/ell.inc
 create mode 100644 meta/recipes-core/ell/ell_git.bb

diff --git a/meta/recipes-core/ell/ell.inc b/meta/recipes-core/ell/ell.inc
new file mode 100644
index 0000000..b5e185e
--- /dev/null
+++ b/meta/recipes-core/ell/ell.inc
@@ -0,0 +1,10 @@
+SUMMARY  = "Embedded Linux Library"
+DESCRIPTION = "ELL is a DBUS library which provides DBUS bindings."
+LICENSE  = "LGPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=fb504b67c50331fc78734fed90fb0e09"
+SECTION = "libs"
+
+inherit autotools pkgconfig
+
+DEPENDS  = "dbus"
+RDEPENDS_${PN} += "dbus"
diff --git a/meta/recipes-core/ell/ell_git.bb b/meta/recipes-core/ell/ell_git.bb
new file mode 100644
index 0000000..fd438be
--- /dev/null
+++ b/meta/recipes-core/ell/ell_git.bb
@@ -0,0 +1,9 @@
+require ell.inc
+
+S = "${WORKDIR}/git"
+SRCREV = "399099946abaeb2f3718a0667cc11436a7c0be14"
+SRC_URI = "git://git.kernel.org/pub/scm/libs/ell/ell.git"
+
+do_configure_prepend () {
+    ${S}/bootstrap
+}
-- 
1.9.1