[PATCH] libvalhalla: Add recipe for libvalhalla 1.0.1.

[PATCH] libvalhalla: Add recipe for libvalhalla 1.0.1.

[Paul Menzel]

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

1. `--enable-pic` is needed, otherwise it does not work for non-x86 architectures, which is only detected with Binutils >= 2.20.
2. Build tested with GCC 4.4.2 and Binutils 2.20.1 and the following in `local.conf`.

        DISTRO = "angstrom-2008.1"
        
        MACHINE = "beagleboard"
        BUILD_ARCH = "i686"
        BUILD_OS = "linux"

3. I tried to run this on the BeagleBoard and `libvalhalla-test` returns an error code. I still have to verify if it is a build problem or some configuration problem.

        # libvalhalla-test -v 
        [libvalhalla] [database.c:1089] Info: Database version : 1
        Default suffixes: flac,m4a,mp3,ogg,wav,wma
                          avi,mkv,mov,mpg,wmv
                          bmp,gif,jpeg,jpg,png,tga,tif,tiff
        Run: parser=2 loop=1 wait=0 priority=19 commit-int=128
        Grabbers available:
          ffmpeg
          local
          tmdb
          amazon
          lastfm
          lyricwiki
          tvdb
          tvrage
          imdb
          allocine
        Error code: -1
        root@beagleboard:~# libvalhalla-test -v -g local
        [libvalhalla] [database.c:1089] Info: Database version : 1
        Default suffixes: flac,m4a,mp3,ogg,wav,wma
                          avi,mkv,mov,mpg,wmv
                          bmp,gif,jpeg,jpg,png,tga,tif,tiff
        Run: parser=2 loop=1 wait=0 priority=19 commit-int=128
        Grabbers available:
          local
        Error code: -1

4. -doc, -dev, -static and “normal” packages are created automatically. Although I could not find the manual page of `libvalhalla-test` included.
5. Maybe a seperate package should be created for `libvalhalla-test`. It is 78K big.

        # ls -alh /usr/bin/libvalhalla-test
        -rwxr-xr-x 1 root root 78K Mar 16 13:48 /usr/bin/libvalhalla-test

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
Could someone else please test this and if the above mentioned issues can be corrected later on, please check it in. If someone has a quick fix for the above issues and can fix them that would also be awesome.

Thanks!
---
 recipes/libvalhalla/libvalhalla.inc      |   24 ++++++++++++++++++++++++
 recipes/libvalhalla/libvalhalla_1.0.1.bb |    5 +++++
 2 files changed, 29 insertions(+), 0 deletions(-)
 create mode 100644 recipes/libvalhalla/libvalhalla.inc
 create mode 100644 recipes/libvalhalla/libvalhalla_1.0.1.bb

diff --git a/recipes/libvalhalla/libvalhalla.inc b/recipes/libvalhalla/libvalhalla.inc
new file mode 100644
index 0000000..2ff1e4c
--- /dev/null
+++ b/recipes/libvalhalla/libvalhalla.inc
@@ -0,0 +1,24 @@
+DESCRIPTION = "libvalhalla is a library written in C. It is a media scanner, that stores various information in an SQLite database and relies on FFmpeg (libavformat and libavutil) and libcurl. It features many Internet grabbers that allows automatic download of covers, lyrics, informations on media files, tags retrival in video and music files and so on."
+HOMEPAGE = "http://libvalhalla.geexbox.org/"
+LICENSE = "LGPLv2.1"
+DEPENDS = "ffmpeg"
+
+INC_PR = "r0"
+
+SRC_URI = "http://libvalhalla.geexbox.org/releases/${P}.tar.bz2;name=libvalhalla"
+
+inherit autotools pkgconfig
+
+# the configure script is hand-crafted, it rejects some of the usual
+# configure arguments
+do_configure() {
+	${S}/configure \
+			--prefix=${prefix} \
+			--disable-strip \
+		--with-lavf-inc=${STAGING_INCDIR} \
+		--with-lavf-lib=${STAGING_LIBDIR} \
+		--with-lavc-inc=${STAGING_INCDIR} \
+		--with-lavc-lib=${STAGING_LIBDIR} \
+		--cross-compile \
+		--enable-pic
+}
diff --git a/recipes/libvalhalla/libvalhalla_1.0.1.bb b/recipes/libvalhalla/libvalhalla_1.0.1.bb
new file mode 100644
index 0000000..cbb1330
--- /dev/null
+++ b/recipes/libvalhalla/libvalhalla_1.0.1.bb
@@ -0,0 +1,5 @@
+require libvalhalla.inc
+PR = "${INC_PR}.0"
+
+SRC_URI[libvalhalla.md5sum] = "75d4a3383887c4c341eeba9ebe60d626"
+SRC_URI[libvalhalla.sha256sum] = "df78015e118c784e3ff3e8562bc96d287ac11997d3b9e45e2d8b67568b0600d3"
-- 
1.7.0

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

Re: [PATCH] libvalhalla: Add recipe for libvalhalla 1.0.1.

[Paul Menzel]

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

Dear OE folks,


please do not apply this patch. Some dependencies are missing. I will
improve it a little bit and resend it. Comments are still welcome. I
have one more question below.

Am Dienstag, den 16.03.2010, 17:45 +0100 schrieb Paul Menzel:

[…]

> 3. I tried to run this on the BeagleBoard and `libvalhalla-test` returns an error code. I still have to verify if it is a build problem or some configuration problem.

[…]

>         root@beagleboard:~# libvalhalla-test -v -g local
>         [libvalhalla] [database.c:1089] Info: Database version : 1
>         Default suffixes: flac,m4a,mp3,ogg,wav,wma
>                           avi,mkv,mov,mpg,wmv
>                           bmp,gif,jpeg,jpg,png,tga,tif,tiff
>         Run: parser=2 loop=1 wait=0 priority=19 commit-int=128
>         Grabbers available:
>           local
>         Error code: -1

That is a usage error. The directory where to look for media files was
missing.

> 4. -doc, -dev, -static and “normal” packages are created automatically. Although I could not find the manual page of `libvalhalla-test` included.

I forgot to add `--enable-doc`.

> 5. Maybe a seperate package should be created for `libvalhalla-test`. It is 78K big.
> 
>         # ls -alh /usr/bin/libvalhalla-test
>         -rwxr-xr-x 1 root root 78K Mar 16 13:48 /usr/bin/libvalhalla-test

6. It is possible to enable several grabbers. `./configure` looks for
itself if the necessary header files are installed and disables the
grabber if no headers are found.

Should I add all needed headers into `DEPENDS` or leave them out and
`./configure` will only enable the ones it finds in the staging
directories?


Thanks,

Paul

> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> ---
> Could someone else please test this and if the above mentioned issues can be corrected later on, please check it in. If someone has a quick fix for the above issues and can fix them that would also be awesome.
> 
> Thanks!
> ---
>  recipes/libvalhalla/libvalhalla.inc      |   24 ++++++++++++++++++++++++
>  recipes/libvalhalla/libvalhalla_1.0.1.bb |    5 +++++
>  2 files changed, 29 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/libvalhalla/libvalhalla.inc
>  create mode 100644 recipes/libvalhalla/libvalhalla_1.0.1.bb
> 
> diff --git a/recipes/libvalhalla/libvalhalla.inc b/recipes/libvalhalla/libvalhalla.inc
> new file mode 100644
> index 0000000..2ff1e4c
> --- /dev/null
> +++ b/recipes/libvalhalla/libvalhalla.inc
> @@ -0,0 +1,24 @@
> +DESCRIPTION = "libvalhalla is a library written in C. It is a media scanner, that stores various information in an SQLite database and relies on FFmpeg (libavformat and libavutil) and libcurl. It features many Internet grabbers that allows automatic download of covers, lyrics, informations on media files, tags retrival in video and music files and so on."
> +HOMEPAGE = "http://libvalhalla.geexbox.org/"
> +LICENSE = "LGPLv2.1"
> +DEPENDS = "ffmpeg"
> +
> +INC_PR = "r0"
> +
> +SRC_URI = "http://libvalhalla.geexbox.org/releases/${P}.tar.bz2;name=libvalhalla"
> +
> +inherit autotools pkgconfig
> +
> +# the configure script is hand-crafted, it rejects some of the usual
> +# configure arguments
> +do_configure() {
> +	${S}/configure \
> +			--prefix=${prefix} \
> +			--disable-strip \
> +		--with-lavf-inc=${STAGING_INCDIR} \
> +		--with-lavf-lib=${STAGING_LIBDIR} \
> +		--with-lavc-inc=${STAGING_INCDIR} \
> +		--with-lavc-lib=${STAGING_LIBDIR} \
> +		--cross-compile \
> +		--enable-pic
> +}
> diff --git a/recipes/libvalhalla/libvalhalla_1.0.1.bb b/recipes/libvalhalla/libvalhalla_1.0.1.bb
> new file mode 100644
> index 0000000..cbb1330
> --- /dev/null
> +++ b/recipes/libvalhalla/libvalhalla_1.0.1.bb
> @@ -0,0 +1,5 @@
> +require libvalhalla.inc
> +PR = "${INC_PR}.0"
> +
> +SRC_URI[libvalhalla.md5sum] = "75d4a3383887c4c341eeba9ebe60d626"
> +SRC_URI[libvalhalla.sha256sum] = "df78015e118c784e3ff3e8562bc96d287ac11997d3b9e45e2d8b67568b0600d3"
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


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

[PATCH v2] libvalhalla: Add recipe for libvalhalla 1.0.1.

[Paul Menzel]

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

1. `--enable-pic` is needed for non-x86 architectures.
2. Put test programs in separate package `${PN}-bin`. Manual pages are in `${PN}-doc` though.
3. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard"`.
4. Depend on `libnfo` since it is recommended on upstream’s Web site.

`do_configure()` is based on the recipe of `ushare`.

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
 recipes/libvalhalla/libvalhalla.inc      |   30 ++++++++++++++++++++++++++++++
 recipes/libvalhalla/libvalhalla_1.0.1.bb |    5 +++++
 2 files changed, 35 insertions(+), 0 deletions(-)
 create mode 100644 recipes/libvalhalla/libvalhalla.inc
 create mode 100644 recipes/libvalhalla/libvalhalla_1.0.1.bb

diff --git a/recipes/libvalhalla/libvalhalla.inc b/recipes/libvalhalla/libvalhalla.inc
new file mode 100644
index 0000000..03b7cfd
--- /dev/null
+++ b/recipes/libvalhalla/libvalhalla.inc
@@ -0,0 +1,30 @@
+DESCRIPTION = "libvalhalla is a library written in C. It is a media scanner, that stores various information in an SQLite database and relies on FFmpeg (libavformat and libavutil) and libcurl. It features many Internet grabbers that allows automatic download of covers, lyrics, informations on media files, tags retrival in video and music files and so on."
+HOMEPAGE = "http://libvalhalla.geexbox.org/"
+SECTION = "libs/multimedia"
+LICENSE = "LGPLv2.1"
+DEPENDS = "sqlite3 curl ffmpeg libnfo"
+
+INC_PR = "r0"
+
+SRC_URI = "http://libvalhalla.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
+
+inherit autotools pkgconfig
+
+# the configure script is hand-crafted, it rejects some of the usual
+# configure arguments
+do_configure() {
+	${S}/configure \
+			--prefix=${prefix} \
+			--disable-strip \
+		--with-lavf-inc=${STAGING_INCDIR} \
+		--with-lavf-lib=${STAGING_LIBDIR} \
+		--with-lavc-inc=${STAGING_INCDIR} \
+		--with-lavc-lib=${STAGING_LIBDIR} \
+		--cross-compile \
+		--enable-pic \
+                --enable-doc
+}
+
+PACKAGES =+ "${PN}-bin"
+
+FILES_${PN}-bin = "${bindir}/*"
diff --git a/recipes/libvalhalla/libvalhalla_1.0.1.bb b/recipes/libvalhalla/libvalhalla_1.0.1.bb
new file mode 100644
index 0000000..9fe1f5e
--- /dev/null
+++ b/recipes/libvalhalla/libvalhalla_1.0.1.bb
@@ -0,0 +1,5 @@
+require libvalhalla.inc
+PR = "${INC_PR}.0"
+
+SRC_URI[libvalhalla.md5sum] = "75d4a3383887c4c341eeba9ebe60d626"
+SRC_URI[libvalhalla.sha256sum] = "df78015e118c784e3ff3e8562bc96d287ac11997d3b9e45e2d8b67568b0600d3"
-- 
1.7.0

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

Re: [PATCH v2] libvalhalla: Add recipe for libvalhalla 1.0.1.

[Koen Kooi]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

All your geexbox patches look ok, feel free to commit them.

regards,

Koen

On 19-03-10 16:27, Paul Menzel wrote:
> 1. `--enable-pic` is needed for non-x86 architectures.
> 2. Put test programs in separate package `${PN}-bin`. Manual pages are in `${PN}-doc` though.
> 3. Build tested with GCC 4.4.3 and Binutils 2.20.1 for `MACHINE = "beagleboard"`.
> 4. Depend on `libnfo` since it is recommended on upstream’s Web site.
> 
> `do_configure()` is based on the recipe of `ushare`.
> 
> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> ---
>  recipes/libvalhalla/libvalhalla.inc      |   30 ++++++++++++++++++++++++++++++
>  recipes/libvalhalla/libvalhalla_1.0.1.bb |    5 +++++
>  2 files changed, 35 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/libvalhalla/libvalhalla.inc
>  create mode 100644 recipes/libvalhalla/libvalhalla_1.0.1.bb
> 
> diff --git a/recipes/libvalhalla/libvalhalla.inc b/recipes/libvalhalla/libvalhalla.inc
> new file mode 100644
> index 0000000..03b7cfd
> --- /dev/null
> +++ b/recipes/libvalhalla/libvalhalla.inc
> @@ -0,0 +1,30 @@
> +DESCRIPTION = "libvalhalla is a library written in C. It is a media scanner, that stores various information in an SQLite database and relies on FFmpeg (libavformat and libavutil) and libcurl. It features many Internet grabbers that allows automatic download of covers, lyrics, informations on media files, tags retrival in video and music files and so on."
> +HOMEPAGE = "http://libvalhalla.geexbox.org/"
> +SECTION = "libs/multimedia"
> +LICENSE = "LGPLv2.1"
> +DEPENDS = "sqlite3 curl ffmpeg libnfo"
> +
> +INC_PR = "r0"
> +
> +SRC_URI = "http://libvalhalla.geexbox.org/releases/${P}.tar.bz2;name=${PN}"
> +
> +inherit autotools pkgconfig
> +
> +# the configure script is hand-crafted, it rejects some of the usual
> +# configure arguments
> +do_configure() {
> +	${S}/configure \
> +			--prefix=${prefix} \
> +			--disable-strip \
> +		--with-lavf-inc=${STAGING_INCDIR} \
> +		--with-lavf-lib=${STAGING_LIBDIR} \
> +		--with-lavc-inc=${STAGING_INCDIR} \
> +		--with-lavc-lib=${STAGING_LIBDIR} \
> +		--cross-compile \
> +		--enable-pic \
> +                --enable-doc
> +}
> +
> +PACKAGES =+ "${PN}-bin"
> +
> +FILES_${PN}-bin = "${bindir}/*"
> diff --git a/recipes/libvalhalla/libvalhalla_1.0.1.bb b/recipes/libvalhalla/libvalhalla_1.0.1.bb
> new file mode 100644
> index 0000000..9fe1f5e
> --- /dev/null
> +++ b/recipes/libvalhalla/libvalhalla_1.0.1.bb
> @@ -0,0 +1,5 @@
> +require libvalhalla.inc
> +PR = "${INC_PR}.0"
> +
> +SRC_URI[libvalhalla.md5sum] = "75d4a3383887c4c341eeba9ebe60d626"
> +SRC_URI[libvalhalla.sha256sum] = "df78015e118c784e3ff3e8562bc96d287ac11997d3b9e45e2d8b67568b0600d3"
> 
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFLo51sMkyGM64RGpERAnX4AJ9iaRsLIE3qaNUJTsglOivn6pkI6ACfQFZK
mo8MrU9wPwaVwjc0DB0e7ZM=
=3P6a
-----END PGP SIGNATURE-----

Re: [PATCH v2] libvalhalla: Add recipe for libvalhalla 1.0.1.

[Paul Menzel]

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

Am Freitag, den 19.03.2010, 16:51 +0100 schrieb Koen Kooi:
> All your geexbox patches look ok,

Did you take a look at the version numbering of libplayer and libnfo,
please? I am not sure about those.

> feel free to commit them.

I do not have commit access. Could you please commit those.


Thanks,

Paul

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