[PATCH] enna: Move to own recipe folder and update to enna hg.

[PATCH] enna: Move to own recipe folder and update to enna hg.

[Paul Menzel]

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

1. Create `enna.inc`.
2. Use `INC_PR`.
3. Move to new repository location.
4. Update license to LGPLv2.1+.
5. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard".
6. Remove `dbus` from `DEPEND` since `edbus` already depends an it.
7. `do_configure_prepend()` fixes [1]. Thanks Koen.

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/thread.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
1. Trying to run this binary unfortunately gives me a symbol lookup error [2].

# enna --help
enna: symbol lookup error: /usr/lib/libelementary-ver-pre-svn-05.so.0: undefined symbol: ECORE_X_ATOM_E_ILLUME_TOP_SHELF_GEOMETRY

The GeeXboX folks say it might be related to an outdated Ecore_X but `ecore` with `--enable-ecore-x` is in `DEPENDS`.

[2] https://geexbox.org/sympa/arc/devel/2010-03/msg00116.html

2. Thanks to all your help!
---
 recipes/e17/enna_svn.bb |   14 --------------
 recipes/enna/enna.inc   |   16 ++++++++++++++++
 recipes/enna/enna_hg.bb |   10 ++++++++++
 3 files changed, 26 insertions(+), 14 deletions(-)
 delete mode 100644 recipes/e17/enna_svn.bb
 create mode 100644 recipes/enna/enna.inc
 create mode 100644 recipes/enna/enna_hg.bb

diff --git a/recipes/e17/enna_svn.bb b/recipes/e17/enna_svn.bb
deleted file mode 100644
index 8f4ddc4..0000000
--- a/recipes/e17/enna_svn.bb
+++ /dev/null
@@ -1,14 +0,0 @@
-DESCRIPTION = "Enna is a media center application based on EFL"
-LICENSE = "GPL"
-DEPENDS = "curl dbus eet evas edje ecore edbus emotion"
-DEPENDS += "libmusicbrainz lightmediascanner libxml2 taglib"
-SECTION = "x11/multimedia"
-PV = "0.2.0+svnr${SRCPV}"
-PR = "r1"
-
-inherit e
-
-SRC_URI = "svn://svn.enlightenment.org/svn/e/trunk/MISC;module=enna;proto=http"
-S = "${WORKDIR}/enna"
-
-FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
diff --git a/recipes/enna/enna.inc b/recipes/enna/enna.inc
new file mode 100644
index 0000000..4cc5b48
--- /dev/null
+++ b/recipes/enna/enna.inc
@@ -0,0 +1,16 @@
+DESCRIPTION = "Enna is a media center application based on EFL."
+HOMEPAGE = "http://${PN}.geexbox.org/"
+SECTION = "x11/multimedia"
+LICENSE = "LGPLv2.1+"
+DEPENDS = "curl eet evas edje ecore edbus elementary emotion"
+DEPENDS += "libxml2 libvalhalla libplayer"
+
+INC_PR = "r1"
+
+SRC_URI = "http://${PN}.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
+
+inherit autotools pkgconfig
+
+do_configure_prepend() {
+	autopoint || touch config.rpath
+}
diff --git a/recipes/enna/enna_hg.bb b/recipes/enna/enna_hg.bb
new file mode 100644
index 0000000..eb82b5e
--- /dev/null
+++ b/recipes/enna/enna_hg.bb
@@ -0,0 +1,10 @@
+require enna.inc
+PV = "0.4.0+hg"
+PR = "${INC_PR}.0"
+
+SRCREV = "0b97b26c8101"
+
+SRC_URI = "hg://hg.geexbox.org;proto=http;module=${PN};rev=${SRCREV}"
+
+S = "${WORKDIR}/${PN}"
-- 
1.7.0.3

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

[PATCH v2] enna: Move to own recipe folder and update to enna hg.

[Paul Menzel]

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

1. Create `enna.inc`.
2. Use `INC_PR`.
3. Move to new repository location.
4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.
5. Update license to LGPLv2.1+.
6. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard".
7. Build tested with GCC 4.3.3 and Binuitls 2.18.50 for `MACHINE = "beagleboard".
8. Remove `dbus` from `DEPEND` since `edbus` already depends an it.
9. `libmusicbrainz`, `lightmediascanner` and `taglib` are not needed anymore as `DEPENDS`.
10. `do_configure_prepend()` fixes [1]. Thanks Koen.
11. libvalhalla 2.0.0 (`libvalhalla_hg.bb`) is needed for this version.

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/thread.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
# v1 → v2
• Use `NOTQUITESRCREV`. Manually “improving” things during composing the message without testing is not a good idea.
• Update revision. (Just because.)
• Update commit message.

# comments
1. Trying to run this binary unfortunately gives me a symbol lookup error [2].

# enna --help
enna: symbol lookup error: /usr/lib/libelementary-ver-pre-svn-05.so.0: undefined symbol: ECORE_X_ATOM_E_ILLUME_TOP_SHELF_GEOMETRY

The GeeXboX folks say it might be related to an outdated Ecore_X but `ecore` with `--enable-ecore-x` is in `DEPENDS`.

[2] https://geexbox.org/sympa/arc/devel/2010-03/msg00116.html

2. Thanks to all your help!
3. There are three `ERROR: QA Issue with enna:` building the recipe. I reported those on geexbox-devel.
---
 recipes/e17/enna_svn.bb |   14 --------------
 recipes/enna/enna.inc   |   16 ++++++++++++++++
 recipes/enna/enna_hg.bb |    9 +++++++++
 3 files changed, 25 insertions(+), 14 deletions(-)
 delete mode 100644 recipes/e17/enna_svn.bb
 create mode 100644 recipes/enna/enna.inc
 create mode 100644 recipes/enna/enna_hg.bb

diff --git a/recipes/e17/enna_svn.bb b/recipes/e17/enna_svn.bb
deleted file mode 100644
index 8f4ddc4..0000000
--- a/recipes/e17/enna_svn.bb
+++ /dev/null
@@ -1,14 +0,0 @@
-DESCRIPTION = "Enna is a media center application based on EFL"
-LICENSE = "GPL"
-DEPENDS = "curl dbus eet evas edje ecore edbus emotion"
-DEPENDS += "libmusicbrainz lightmediascanner libxml2 taglib"
-SECTION = "x11/multimedia"
-PV = "0.2.0+svnr${SRCPV}"
-PR = "r1"
-
-inherit e
-
-SRC_URI = "svn://svn.enlightenment.org/svn/e/trunk/MISC;module=enna;proto=http"
-S = "${WORKDIR}/enna"
-
-FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
diff --git a/recipes/enna/enna.inc b/recipes/enna/enna.inc
new file mode 100644
index 0000000..4cc5b48
--- /dev/null
+++ b/recipes/enna/enna.inc
@@ -0,0 +1,16 @@
+DESCRIPTION = "Enna is a media center application based on EFL."
+HOMEPAGE = "http://${PN}.geexbox.org/"
+SECTION = "x11/multimedia"
+LICENSE = "LGPLv2.1+"
+DEPENDS = "curl eet evas edje ecore edbus elementary emotion"
+DEPENDS += "libxml2 libvalhalla libplayer"
+
+INC_PR = "r1"
+
+SRC_URI = "http://${PN}.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
+
+inherit autotools pkgconfig
+
+do_configure_prepend() {
+	autopoint || touch config.rpath
+}
diff --git a/recipes/enna/enna_hg.bb b/recipes/enna/enna_hg.bb
new file mode 100644
index 0000000..f927461
--- /dev/null
+++ b/recipes/enna/enna_hg.bb
@@ -0,0 +1,9 @@
+require enna.inc
+PV = "0.4.0+hg"
+PR = "${INC_PR}.0"
+
+NOTQUITESRCREV = "a00ab191771b"
+
+SRC_URI = "hg://hg.geexbox.org;proto=http;module=${PN};rev=${NOTQUITESRCREV}"
+
+S = "${WORKDIR}/${PN}"
-- 
1.7.0.3

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.

[Paul Menzel]

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

1. Create `enna.inc`.
2. Use `INC_PR`.
3. Move to new repository location.
4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.
5. Update license to LGPLv2.1+.
6. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard".
7. Build tested with GCC 4.3.3 and Binuitls 2.18.50 for `MACHINE = "beagleboard".
8. Remove `dbus` from `DEPEND` since `edbus` already depends an it.
9. `libmusicbrainz`, `lightmediascanner` and `taglib` are not needed anymore as `DEPENDS`.
10. `do_configure_prepend()` fixes [1]. Thanks Koen.
11. libvalhalla 2.0.0 (`libvalhalla_hg.bb`) is needed for this version.
12. Works on BeagleBoard with libplayer patches sent to list.

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/thread.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
# v1 → v2
• Use `NOTQUITESRCREV`. Manually “improving” things during composing the message without testing is not a good idea.
• Update revision. (Just because.)
• Update commit message.

# v2 → v3
• Update `SRCREV` which fixed `ERROR: QA Issue with enna:`.
• New efl revisions fixed `enna: symbol lookup error: /usr/lib/libelementary-ver-pre-svn-05.so.0: undefined symbol: ECORE_X_ATOM_E_ILLUME_TOP_SHELF_GEOMETRY` [2].

[2] https://geexbox.org/sympa/arc/devel/2010-03/msg00116.html

Thanks for all your help!
---
 recipes/e17/enna_svn.bb |   15 ---------------
 recipes/enna/enna.inc   |   16 ++++++++++++++++
 recipes/enna/enna_hg.bb |    9 +++++++++
 3 files changed, 25 insertions(+), 15 deletions(-)
 delete mode 100644 recipes/e17/enna_svn.bb
 create mode 100644 recipes/enna/enna.inc
 create mode 100644 recipes/enna/enna_hg.bb

diff --git a/recipes/e17/enna_svn.bb b/recipes/e17/enna_svn.bb
deleted file mode 100644
index f81990c..0000000
--- a/recipes/e17/enna_svn.bb
+++ /dev/null
@@ -1,15 +0,0 @@
-DESCRIPTION = "Enna is a media center application based on EFL"
-LICENSE = "GPL"
-DEPENDS = "curl dbus eet evas edje ecore edbus emotion"
-DEPENDS += "libmusicbrainz lightmediascanner libxml2 taglib"
-SECTION = "x11/multimedia"
-PV = "0.2.0+svnr${SRCPV}"
-PR = "r1"
-SRCREV = "${EFL_SRCREV}"
-
-inherit e
-
-SRC_URI = "svn://svn.enlightenment.org/svn/e/trunk/MISC;module=enna;proto=http"
-S = "${WORKDIR}/enna"
-
-FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
diff --git a/recipes/enna/enna.inc b/recipes/enna/enna.inc
new file mode 100644
index 0000000..4cc5b48
--- /dev/null
+++ b/recipes/enna/enna.inc
@@ -0,0 +1,16 @@
+DESCRIPTION = "Enna is a media center application based on EFL."
+HOMEPAGE = "http://${PN}.geexbox.org/"
+SECTION = "x11/multimedia"
+LICENSE = "LGPLv2.1+"
+DEPENDS = "curl eet evas edje ecore edbus elementary emotion"
+DEPENDS += "libxml2 libvalhalla libplayer"
+
+INC_PR = "r1"
+
+SRC_URI = "http://${PN}.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
+
+inherit autotools pkgconfig
+
+do_configure_prepend() {
+	autopoint || touch config.rpath
+}
diff --git a/recipes/enna/enna_hg.bb b/recipes/enna/enna_hg.bb
new file mode 100644
index 0000000..68f69ba
--- /dev/null
+++ b/recipes/enna/enna_hg.bb
@@ -0,0 +1,9 @@
+require enna.inc
+PV = "0.4.0+hg"
+PR = "${INC_PR}.0"
+
+NOTQUITESRCREV = "319d4702c427"
+
+SRC_URI = "hg://hg.geexbox.org;proto=http;module=${PN};rev=${NOTQUITESRCREV}"
+
+S = "${WORKDIR}/${PN}"
-- 
1.7.0.3

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.

[Khem Raj]

On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
<paulepanter@users.sourceforge.net> wrote:
> 1. Create `enna.inc`.
> 2. Use `INC_PR`.
> 3. Move to new repository location.
> 4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.
> 5. Update license to LGPLv2.1+.
> 6. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard".
> 7. Build tested with GCC 4.3.3 and Binuitls 2.18.50 for `MACHINE = "beagleboard".
> 8. Remove `dbus` from `DEPEND` since `edbus` already depends an it.
> 9. `libmusicbrainz`, `lightmediascanner` and `taglib` are not needed anymore as `DEPENDS`.
> 10. `do_configure_prepend()` fixes [1]. Thanks Koen.
> 11. libvalhalla 2.0.0 (`libvalhalla_hg.bb`) is needed for this version.
> 12. Works on BeagleBoard with libplayer patches sent to list.
>
> [1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/thread.html
>
> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> ---
> # v1 → v2
> • Use `NOTQUITESRCREV`. Manually “improving” things during composing the message without testing is not a good idea.
> • Update revision. (Just because.)
> • Update commit message.
>
> # v2 → v3
> • Update `SRCREV` which fixed `ERROR: QA Issue with enna:`.
> • New efl revisions fixed `enna: symbol lookup error: /usr/lib/libelementary-ver-pre-svn-05.so.0: undefined symbol: ECORE_X_ATOM_E_ILLUME_TOP_SHELF_GEOMETRY` [2].
>
> [2] https://geexbox.org/sympa/arc/devel/2010-03/msg00116.html
>
> Thanks for all your help!
> ---
>  recipes/e17/enna_svn.bb |   15 ---------------
>  recipes/enna/enna.inc   |   16 ++++++++++++++++
>  recipes/enna/enna_hg.bb |    9 +++++++++
>  3 files changed, 25 insertions(+), 15 deletions(-)
>  delete mode 100644 recipes/e17/enna_svn.bb
>  create mode 100644 recipes/enna/enna.inc
>  create mode 100644 recipes/enna/enna_hg.bb
>
> diff --git a/recipes/e17/enna_svn.bb b/recipes/e17/enna_svn.bb
> deleted file mode 100644
> index f81990c..0000000
> --- a/recipes/e17/enna_svn.bb
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -DESCRIPTION = "Enna is a media center application based on EFL"
> -LICENSE = "GPL"
> -DEPENDS = "curl dbus eet evas edje ecore edbus emotion"
> -DEPENDS += "libmusicbrainz lightmediascanner libxml2 taglib"
> -SECTION = "x11/multimedia"
> -PV = "0.2.0+svnr${SRCPV}"
> -PR = "r1"
> -SRCREV = "${EFL_SRCREV}"
> -
> -inherit e
> -
> -SRC_URI = "svn://svn.enlightenment.org/svn/e/trunk/MISC;module=enna;proto=http"
> -S = "${WORKDIR}/enna"
> -
> -FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
> diff --git a/recipes/enna/enna.inc b/recipes/enna/enna.inc
> new file mode 100644
> index 0000000..4cc5b48
> --- /dev/null
> +++ b/recipes/enna/enna.inc
> @@ -0,0 +1,16 @@
> +DESCRIPTION = "Enna is a media center application based on EFL."
> +HOMEPAGE = "http://${PN}.geexbox.org/"
> +SECTION = "x11/multimedia"
> +LICENSE = "LGPLv2.1+"
> +DEPENDS = "curl eet evas edje ecore edbus elementary emotion"
> +DEPENDS += "libxml2 libvalhalla libplayer"
> +
> +INC_PR = "r1"
> +
> +SRC_URI = "http://${PN}.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
> +
> +inherit autotools pkgconfig
> +
> +do_configure_prepend() {
> +       autopoint || touch config.rpath
> +}

empty config.rpath would build it wrongly isnt it. Probably this
should be removed

> diff --git a/recipes/enna/enna_hg.bb b/recipes/enna/enna_hg.bb
> new file mode 100644
> index 0000000..68f69ba
> --- /dev/null
> +++ b/recipes/enna/enna_hg.bb
> @@ -0,0 +1,9 @@
> +require enna.inc
> +PV = "0.4.0+hg"
> +PR = "${INC_PR}.0"
> +
> +NOTQUITESRCREV = "319d4702c427"
> +
> +SRC_URI = "hg://hg.geexbox.org;proto=http;module=${PN};rev=${NOTQUITESRCREV}"
> +
> +S = "${WORKDIR}/${PN}"
> --
> 1.7.0.3
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>

Re: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.

[Paul Menzel]

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

Am Freitag, den 02.04.2010, 11:28 -0700 schrieb Khem Raj:
> On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
> <paulepanter@users.sourceforge.net> wrote:

[…]

> > +do_configure_prepend() {
> > +       autopoint || touch config.rpath
> > +}
> 
> empty config.rpath would build it wrongly isnt it. Probably this
> should be removed

That is the work around Koen suggested [1]. Koen, can you comment
please.

[…]


Thanks,

Paul


[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018800.html

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

autopoint || touch config.rpath (was: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.)

[Paul Menzel]

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

Am Freitag, den 02.04.2010, 21:44 +0200 schrieb Paul Menzel:
> Am Freitag, den 02.04.2010, 11:28 -0700 schrieb Khem Raj:
> > On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
> > <paulepanter@users.sourceforge.net> wrote:
> 
> […]
> 
> > > +do_configure_prepend() {
> > > +       autopoint || touch config.rpath
> > > +}
> > 
> > empty config.rpath would build it wrongly isnt it. Probably this
> > should be removed
> 
> That is the work around Koen suggested [1]. Koen, can you comment
> please.

I do not know, if I should have answered in the thread of [1], but
anyway.

Is

        do_configure_prepend() {
                autopoint || automake --add-missing
        }

a viable solution [2]? Or is this just the same as removing
`EXTRA_AUTORECONF = "--exclude=autopoint"` from `autotools.bbclass`?

The configure went fine for me.


Thanks,

Paul


> [1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018800.html
[2] http://www.mail-archive.com/automake-patches@gnu.org/msg00479.html

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: autopoint || touch config.rpath (was: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.)

[Jens Seidel]

On Sat, Apr 03, 2010 at 01:20:20PM +0200, Paul Menzel wrote:
> Am Freitag, den 02.04.2010, 21:44 +0200 schrieb Paul Menzel:
> > Am Freitag, den 02.04.2010, 11:28 -0700 schrieb Khem Raj:
> > > On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel <paulepanter@users.sourceforge.net> wrote:
> > > > +do_configure_prepend() {
> > > > +       autopoint || touch config.rpath
> > > > +}
> > > 
> > > empty config.rpath would build it wrongly isnt it. Probably this
> > > should be removed

Right.
 
> > That is the work around Koen suggested [1]. Koen, can you comment
> > please.
> 
> Is
> 
>         do_configure_prepend() {
>                 autopoint || automake --add-missing
>         }
> 
> a viable solution [2]?

No, it isn't. I really wonder about all these guesses. I agree that the
autotools stuff is not very easy to understand but one should only provide
patches if one knows it.

Let me short summarize:

automake is a tool to generate Makefile templates (called Makefile.in) from
Makefile.am files. Normally a tgz ball ships already these generated files
(that's dictated by GNU). Nevertheless these files are often not committed
into version control systems and a script autogen.sh or better autoreconf is
used to generate Makefile.in by calling automake. But autoreconf calls even
more: e.g. autoconf (which generated configure) and now also autopoint.
autopoint generated the gettext infrastructure (the translation system to
support foreign languages) by copying all non config dependent gettext files
(such as Makefile.in.in, some scripts, ...) into po and m4 macros. That's
also a good thing as this avoids committing generated files.

But why do you want to call either autopoint or automake? The belong
together and autoreconf should care about it. If this doesn' work it could
be a bug in the package.

If you send me the project source and the command which fails (without
bitbake stuff as I never used it up to now) together with the
autoconf/automake versions I can try to inspect it. Since the error seems to
happen in configure or even before there should be no difficult relationship to
bitbake and recipes, right?

Jens

Re: autopoint || touch config.rpath

[Paul Menzel]

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

Am Samstag, den 03.04.2010, 15:51 +0200 schrieb Jens Seidel:
> On Sat, Apr 03, 2010 at 01:20:20PM +0200, Paul Menzel wrote:
> > Am Freitag, den 02.04.2010, 21:44 +0200 schrieb Paul Menzel:
> > > Am Freitag, den 02.04.2010, 11:28 -0700 schrieb Khem Raj:
> > > > On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel <paulepanter@users.sourceforge.net> wrote:
> > > > > +do_configure_prepend() {
> > > > > +       autopoint || touch config.rpath
> > > > > +}
> > > > 
> > > > empty config.rpath would build it wrongly isnt it. Probably this
> > > > should be removed
> 
> Right.
>  
> > > That is the work around Koen suggested [1]. Koen, can you comment
> > > please.
> > 
> > Is
> > 
> >         do_configure_prepend() {
> >                 autopoint || automake --add-missing
> >         }
> > 
> > a viable solution [2]?
> 
> No, it isn't. I really wonder about all these guesses. I agree that the
> autotools stuff is not very easy to understand but one should only provide
> patches if one knows it.

You are right. Great to have an expert joining the discussion.

> Let me short summarize:
> 
> automake is a tool to generate Makefile templates (called Makefile.in) from
> Makefile.am files. Normally a tgz ball ships already these generated files
> (that's dictated by GNU). Nevertheless these files are often not committed
> into version control systems and a script autogen.sh or better autoreconf is
> used to generate Makefile.in by calling automake. But autoreconf calls even
> more: e.g. autoconf (which generated configure) and now also autopoint.
> autopoint generated the gettext infrastructure (the translation system to
> support foreign languages) by copying all non config dependent gettext files
> (such as Makefile.in.in, some scripts, ...) into po and m4 macros. That's
> also a good thing as this avoids committing generated files.

Thank you for this great summary. I am still clueless about
`config.rpath` though.

> But why do you want to call either autopoint or automake? The belong
> together and autoreconf should care about it. If this doesn' work it could
> be a bug in the package.
> 
> If you send me the project source and the command which fails (without
> bitbake stuff as I never used it up to now) together with the
> autoconf/automake versions I can try to inspect it. Since the error seems to
> happen in configure or even before there should be no difficult relationship to
> bitbake and recipes, right?

Correct. Though as far as I understood it using `inherit autotools` OE
is using its own commands to set up the environment using
`autotools.bbclass` and there you will find the following line [2]
causing problems [3].

    EXTRA_AUTORECONF = "--exclude=autopoint"

Nobody knows why this was added in the first place, but if you try to
configure Enna [4] for example it will fail as I reported [5]. Other
packages seem to be affected too.

The argument is called in this line [6].

    oenote Executing autoreconf --verbose --install --force ${EXTRA_AUTORECONF} $acpaths
    autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || oefatal "autoreconf execution failed."

I cannot reproduce this using the provided files by Enna in my local
environment.


Thanks,

Paul


[1] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass
[2] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass#n38
[3] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018818.html
[4] http://enna.geexbox.org/
[5] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018796.html
[6] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass#n136

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: autopoint || touch config.rpath

[Jens Seidel]

Hi Paul,

I'm sorry for replying so late.

Originally you wrote
(http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018796.html):

> Now I am getting the following error message during `configure`.
> configure.ac:35: required file `./config.rpath' not found

To be honest I had trouble reproducing it without bitbake. Probably I should
take the time to learn it ...

Nevertheless according to my autotools knowledge this comes from ommitting
autopoint, which installs this file and is by default called from
autoreconf.
Have you tried removing the line EXTRA_AUTORECONF = "--exclude=autopoint"
in [2]?

But --exclude=autopoint is no valid option for version 2.65. Mhm ...

Try also to use the option -v of autoreconf, some warnings are only visible
in verbose mode (such as sometimes:
"autoreconf: configure.ac: not running autopoint: --install not given").

> So `autogen.sh` needs to be run and I just found the following
> hint.
>
>        do_configure_prepend() {
>               sh autogen.sh
>        }
>
> But this fails with the error that C programs cannot be executed.

This fails because this already starts configure without crosscompiling
options. Set the environment variable "NOCONFIGURE=yes" before calling
autogen.sh. Does it help?

On Sat, Apr 03, 2010 at 04:43:58PM +0200, Paul Menzel wrote:
> Am Samstag, den 03.04.2010, 15:51 +0200 schrieb Jens Seidel:
> > On Sat, Apr 03, 2010 at 01:20:20PM +0200, Paul Menzel wrote:
> > > Am Freitag, den 02.04.2010, 21:44 +0200 schrieb Paul Menzel:
> > > > Am Freitag, den 02.04.2010, 11:28 -0700 schrieb Khem Raj:
> > > > > On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel <paulepanter@users.sourceforge.net> wrote:
> > Let me short summarize:
> > 
> > automake is a tool to generate Makefile templates (called Makefile.in) from
> > Makefile.am files. Normally a tgz ball ships already these generated files
> > (that's dictated by GNU). Nevertheless these files are often not committed
> > into version control systems and a script autogen.sh or better autoreconf is
> > used to generate Makefile.in by calling automake. But autoreconf calls even
> > more: e.g. autoconf (which generated configure) and now also autopoint.
> > autopoint generated the gettext infrastructure (the translation system to

s/generated/generates/

> > support foreign languages) by copying all non config dependent gettext files
> > (such as Makefile.in.in, some scripts, ...) into po and m4 macros. That's

should be (sorry):
(such as Makefile.in.in, some scripts, m4 macros, ...) into the directory po/.

> > also a good thing as this avoids committing generated files.
 
> Thank you for this great summary. I am still clueless about
> `config.rpath` though.

Yep, try the hints above.
 
> Correct. Though as far as I understood it using `inherit autotools` OE
> is using its own commands to set up the environment using
> `autotools.bbclass` and there you will find the following line [2]
> causing problems [3].
> 
>     EXTRA_AUTORECONF = "--exclude=autopoint"
> 
> Nobody knows why this was added in the first place, but if you try to

What about git? Try git blame, git log.

I suggest to just remove it if it works without!

autopoint has dependance to cvs: it requires the client as it
ships a local CVS repository to handle various gettext versions (if a
package mentions it needs only version x.y of gettext, autopoint copies only
this version (no newer one) into the directory to verify that x.y. is indeed
sufficient). Maybe autopoint was excluded to avoid this dependence, there
were some similar problems in Debian (http://bugs.debian.org/432270) and
Gentoo, where CVS was not added as requirement for gettext (which ships
autopoint).

> configure Enna [4] for example it will fail as I reported [5]. Other
> packages seem to be affected too.
> 
> The argument is called in this line [6].
> 
>     oenote Executing autoreconf --verbose --install --force ${EXTRA_AUTORECONF} $acpaths
>     autoreconf -Wcross --verbose --install --force ${EXTRA_AUTORECONF} $acpaths || oefatal "autoreconf execution failed."
> 
> I cannot reproduce this using the provided files by Enna in my local
> environment.
 
> [1] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass
> [2] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass#n38
> [3] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018818.html
> [4] http://enna.geexbox.org/
> [5] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018796.html
> [6] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/classes/autotools.bbclass#n136

Re: autopoint || touch config.rpath

[Richard Purdie]

On Wed, 2010-04-07 at 17:44 +0200, Jens Seidel wrote:
> > Thank you for this great summary. I am still clueless about
> > `config.rpath` though.
> 
> Yep, try the hints above.

I'm late to this thread but poky has this in autoconf.bbclass:


  if grep "^[[:space:]]*AM_GLIB_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then 
	if grep "sed.*POTFILES" $CONFIGURE_AC >/dev/null; then
		: do nothing -- we still have an old unmodified configure.ac
	else
		oenote Executing glib-gettextize --force --copy
		echo "no" | glib-gettextize --force --copy 130
	fi
  else if grep "^[[:space:]]*AM_GNU_GETTEXT" $CONFIGURE_AC >/dev/null; then 
	cp ${STAGING_DATADIR}/gettext/config.rpath ${S}/
  fi

which generally makes sure config.rpath is available when its needed.

Cheers,

Richard

Re: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.

[Martin Jansa]

On Fri, Apr 02, 2010 at 11:28:10AM -0700, Khem Raj wrote:
> On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
> <paulepanter@users.sourceforge.net> wrote:
> > 1. Create `enna.inc`.
> > 2. Use `INC_PR`.
> > 3. Move to new repository location.
> > 4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.

You can use SRCREV again after
http://git.openembedded.org/cgit.cgi/openembedded/commit/?id=82b394467d9142edf2721176244c3a7eb26ba69a

And maybe you want rev shown in PV or PR
PV = "0.4.0+hg${SRCPV}" or 
PR_append = "+hg${SRCPV}"
is quire common and should work for hg too (haven't checked)

Cheers,

-- 
uin:136542059                jid:Martin.Jansa@gmail.com
Jansa Martin                 sip:jamasip@voip.wengo.fr 
JaMa                         

Setting `PV` in recipes using SCM repositories (was: [PATCH v3] enna: Move to own recipe folder and update to new Enna hg repository.)

[Paul Menzel]

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

Am Freitag, den 02.04.2010, 23:18 +0200 schrieb Martin Jansa:
> On Fri, Apr 02, 2010 at 11:28:10AM -0700, Khem Raj wrote:
> > On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
> > <paulepanter@users.sourceforge.net> wrote:
> > > 1. Create `enna.inc`.
> > > 2. Use `INC_PR`.
> > > 3. Move to new repository location.
> > > 4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.
> 
> You can use SRCREV again after
> http://git.openembedded.org/cgit.cgi/openembedded/commit/?id=82b394467d9142edf2721176244c3a7eb26ba69a

I guess I have to resend a v4.

> And maybe you want rev shown in PV or PR
> PV = "0.4.0+hg${SRCPV}" or 
> PR_append = "+hg${SRCPV}"
> is quire common and should work for hg too (haven't checked)

Unfortunately I could not find any information about `SRCPV` in the
BitBake or OE manual. What is the difference to `SRCREV`?

Anyway, the problem I have with appending the `SRCREV` to `PV` is, that
if you change the revision `bitbake -c clean recipe.bb` will not work
and the old files stay in the build tree. Is that true with `SRCPV`
also?

Could the TSC make a suggestion on how to set `PV` in recipes using SCM
repositories.


Thanks,

Paul

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: Setting `PV` in recipes using SCM repositories

[Marco Cavallini]

Paul Menzel ha scritto, Il 03/04/2010 00:56:
> Am Freitag, den 02.04.2010, 23:18 +0200 schrieb Martin Jansa:
>> On Fri, Apr 02, 2010 at 11:28:10AM -0700, Khem Raj wrote:
>>> On Fri, Apr 2, 2010 at 8:59 AM, Paul Menzel
>>> <paulepanter@users.sourceforge.net> wrote:
>>>> 1. Create `enna.inc`.
>>>> 2. Use `INC_PR`.
>>>> 3. Move to new repository location.
>>>> 4. Use `NOTQUITESRCREV` since `SRCREV` seems to get set to the one from the EFL recipes.
>> You can use SRCREV again after
>> http://git.openembedded.org/cgit.cgi/openembedded/commit/?id=82b394467d9142edf2721176244c3a7eb26ba69a
> 
> I guess I have to resend a v4.
> 
>> And maybe you want rev shown in PV or PR
>> PV = "0.4.0+hg${SRCPV}" or 
>> PR_append = "+hg${SRCPV}"
>> is quire common and should work for hg too (haven't checked)
> 
> Unfortunately I could not find any information about `SRCPV` in the
> BitBake or OE manual. What is the difference to `SRCREV`?
> 
> Anyway, the problem I have with appending the `SRCREV` to `PV` is, that
> if you change the revision `bitbake -c clean recipe.bb` will not work
> and the old files stay in the build tree. Is that true with `SRCPV`
> also?
> 
> Could the TSC make a suggestion on how to set `PV` in recipes using SCM
> repositories.
> 
> 
> Thanks,
> 
> Paul
> 
> 
> ------------------------------------------------------------------------

I'll work on this next week

Happy Easter :-)


Cordiali Saluti / Kindest Regards / mit freundlichen Grüssen
--
Marco Cavallini | KOAN sas | Bergamo - Italia
 embedded and real-time software engineering
Phone:+39-035-255.235 - Fax:+39-178-22.39.748
      http://www.KoanSoftware.com

[PATCH v4] enna: Move to own recipe folder and update to new Enna hg repository.

[Paul Menzel]

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

1. Create `enna.inc`.
2. Use `INC_PR`.
3. Move to new repository location.
4. Update license to LGPLv2.1+.
5. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard".
6. Build tested with GCC 4.3.3 and Binuitls 2.18.50 for `MACHINE = "beagleboard".
7. Remove `dbus` from `DEPEND` since `edbus` already depends an it.
8. `libmusicbrainz`, `lightmediascanner` and `taglib` are not needed anymore as `DEPENDS`.
9. `do_configure_prepend()` fixes [1]. Thanks Koen.
10. libvalhalla 2.0.0 (`libvalhalla_hg.bb`) is needed for this version.

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-March/018800.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
# v1 → v2
• Use `NOTQUITESRCREV`. Manually “improving” things during composing the message without testing is not a good idea.
• Update revision. (Just because.)
• Update commit message.

# v2 → v3
• Update `SRCREV` which fixed `ERROR: QA Issue with enna:`.
• New efl revisions fixed `enna: symbol lookup error: /usr/lib/libelementary-ver-pre-svn-05.so.0: undefined symbol: ECORE_X_ATOM_E_ILLUME_TOP_SHELF_GEOMETRY` [2].

[2] https://geexbox.org/sympa/arc/devel/2010-03/msg00116.html

# v3 → v4
• Use `SRCREV` as mentioned by Martin [3].
• Update revision to current tip.

[3] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-April/019018.html

Thanks for all your help!
---
 recipes/e17/enna_svn.bb |   15 ---------------
 recipes/enna/enna.inc   |   16 ++++++++++++++++
 recipes/enna/enna_hg.bb |    9 +++++++++
 3 files changed, 25 insertions(+), 15 deletions(-)
 delete mode 100644 recipes/e17/enna_svn.bb
 create mode 100644 recipes/enna/enna.inc
 create mode 100644 recipes/enna/enna_hg.bb

diff --git a/recipes/e17/enna_svn.bb b/recipes/e17/enna_svn.bb
deleted file mode 100644
index f81990c..0000000
--- a/recipes/e17/enna_svn.bb
+++ /dev/null
@@ -1,15 +0,0 @@
-DESCRIPTION = "Enna is a media center application based on EFL"
-LICENSE = "GPL"
-DEPENDS = "curl dbus eet evas edje ecore edbus emotion"
-DEPENDS += "libmusicbrainz lightmediascanner libxml2 taglib"
-SECTION = "x11/multimedia"
-PV = "0.2.0+svnr${SRCPV}"
-PR = "r1"
-SRCREV = "${EFL_SRCREV}"
-
-inherit e
-
-SRC_URI = "svn://svn.enlightenment.org/svn/e/trunk/MISC;module=enna;proto=http"
-S = "${WORKDIR}/enna"
-
-FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
diff --git a/recipes/enna/enna.inc b/recipes/enna/enna.inc
new file mode 100644
index 0000000..4cc5b48
--- /dev/null
+++ b/recipes/enna/enna.inc
@@ -0,0 +1,16 @@
+DESCRIPTION = "Enna is a media center application based on EFL."
+HOMEPAGE = "http://${PN}.geexbox.org/"
+SECTION = "x11/multimedia"
+LICENSE = "LGPLv2.1+"
+DEPENDS = "curl eet evas edje ecore edbus elementary emotion"
+DEPENDS += "libxml2 libvalhalla libplayer"
+
+INC_PR = "r1"
+
+SRC_URI = "http://${PN}.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
+
+inherit autotools pkgconfig
+
+do_configure_prepend() {
+	autopoint || touch config.rpath
+}
diff --git a/recipes/enna/enna_hg.bb b/recipes/enna/enna_hg.bb
new file mode 100644
index 0000000..89fa12b
--- /dev/null
+++ b/recipes/enna/enna_hg.bb
@@ -0,0 +1,9 @@
+require enna.inc
+PV = "0.4.0+hg"
+PR = "${INC_PR}.0"
+
+SRCREV = "2a9c5f249b47"
+
+SRC_URI = "hg://hg.geexbox.org;proto=http;module=${PN};rev=${SRCREV}"
+
+S = "${WORKDIR}/${PN}"
-- 
1.7.0.3

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 205 bytes --]