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.

[…]

>         r...@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 <[email protected]>
> ---
> 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
> [email protected]
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil

_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to