[Buildroot] [PATCH 2/2] pacage/xbmc-addon-xvdr: new package

[Maxime Hadjinlian]

Hi Yann, all

On Fri, Mar 28, 2014 at 1:03 AM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> From: "Yann E. MORIN" <yann.morin.1998 at free.fr>
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Maxime Hadjinlian <maxime.hadjinlian at gmail.com>
> ---
>  package/xbmc-addon-xvdr/Config.in                  | 14 ++++++++++++
>  .../xbmc-addon-xvdr-0001-foreign-automake.patch    | 25 ++++++++++++++++++++++
>  package/xbmc-addon-xvdr/xbmc-addon-xvdr.mk         | 25 ++++++++++++++++++++++
>  package/xbmc/Config.in                             |  1 +
>  4 files changed, 65 insertions(+)
>  create mode 100644 package/xbmc-addon-xvdr/Config.in
>  create mode 100644 package/xbmc-addon-xvdr/xbmc-addon-xvdr-0001-foreign-automake.patch
>  create mode 100644 package/xbmc-addon-xvdr/xbmc-addon-xvdr.mk
>
> diff --git a/package/xbmc-addon-xvdr/Config.in b/package/xbmc-addon-xvdr/Config.in
> new file mode 100644
> index 0000000..7329a9a
> --- /dev/null
> +++ b/package/xbmc-addon-xvdr/Config.in
> @@ -0,0 +1,14 @@
> +config BR2_PACKAGE_XBMC_ADDON_XVDR
> +       bool "xbmc-addon-xvdr"
> +       help
> +         This is a PVR add-on for XBMC to add VDR (http://tvdr.de/)
> +         as a TV/PVR Backend to XBMC.
> +
> +         It adds support for Live TV watching, replaying of Recordings,
> +         programming Timers and EPG TV Guide to use on same computer or
> +         over the Network.
> +
> +         https://github.com/pipelka/xbmc-addon-xvdr
> +
> +         Note: since the VDR server is not packaged in Buildroot, using
> +         this addon requires that a remote VDR server be used.
I am shocked that you did not provide a patch for this :).
Would it be any difficult or you don't see the point in us doing so ?
> diff --git a/package/xbmc-addon-xvdr/xbmc-addon-xvdr-0001-foreign-automake.patch b/package/xbmc-addon-xvdr/xbmc-addon-xvdr-0001-foreign-automake.patch
> new file mode 100644
> index 0000000..6996e85
> --- /dev/null
> +++ b/package/xbmc-addon-xvdr/xbmc-addon-xvdr-0001-foreign-automake.patch
> @@ -0,0 +1,25 @@
> +commit 846b0b5a8b559f94dffadf5b2f95ca845b6ba9f1
> +Author: Yann E. MORIN <yann.morin.1998 at free.fr>
> +Date:   Wed Mar 26 07:45:58 2014 +0100
> +
> +    configure: set the foreign flag
> +
> +    This is required since we do not bundle the necessary files (NEWS,
> +    INSTALL et al.) Setting it in configure.in is needed to be able
> +    to run autoreconf (in lieu of the weird autogen.sh)
> +
> +    Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> +
> +diff --git a/configure.in b/configure.in
> +index 6e692e1..7be0d18 100755
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -12,7 +12,7 @@ AC_CANONICAL_HOST
> + AC_CANONICAL_TARGET
> +
> + dnl Setup for automake
> +-AM_INIT_AUTOMAKE
> ++AM_INIT_AUTOMAKE([foreign])
> +
> + dnl Check for tools
> +
Do you think there is a chance for this one to make it mainline ?
> diff --git a/package/xbmc-addon-xvdr/xbmc-addon-xvdr.mk b/package/xbmc-addon-xvdr/xbmc-addon-xvdr.mk
> new file mode 100644
> index 0000000..9fa0a13
> --- /dev/null
> +++ b/package/xbmc-addon-xvdr/xbmc-addon-xvdr.mk
> @@ -0,0 +1,25 @@
> +################################################################################
> +#
> +# xbmc-addon-xvdr
> +#
> +################################################################################
> +
> +# This cset is on the branch 'xbmc-frodo'
Little typo.
> +# When XBMC is updated, then this should be updated to the corresponding branch
> +XBMC_ADDON_XVDR_VERSION = acd4e145fc3220cf708aaf40d895904732dce2c7
> +XBMC_ADDON_XVDR_SITE = $(call github,pipelka,xbmc-addon-xvdr,$(XBMC_ADDON_XVDR_VERSION))
> +XBMC_ADDON_XVDR_LICENSE = GPLv2+
> +XBMC_ADDON_XVDR_LICENSE_FILES = COPYING
> +
> +# There's no ./configure in the git tree, we need to generate it
> +# xbmc-addon-xvdr uses a weird autogen.sh script, which
> +# is even incorrect (it's missing the #! ) Sigh... :-(
> +# Fortunately, with our little patch, it autoreconfs nicely! :-)
> +XBMC_ADDON_XVDR_AUTORECONF = YES
> +
> +# This really is a runtime dependency, but we need XBMC to be isntalled
Little typo here too.
> +# first, since we'll install files in XBMC's directories _after_ XBMC has
> +# installed his own files
> +XBMC_ADDON_XVDR_DEPENDENCIES = xbmc
> +
> +$(eval $(autotools-package))
> diff --git a/package/xbmc/Config.in b/package/xbmc/Config.in
> index eb6ef55..d88821a 100644
> --- a/package/xbmc/Config.in
> +++ b/package/xbmc/Config.in
> @@ -160,6 +160,7 @@ config BR2_PACKAGE_XBMC_WAVPACK
>
>  comment "XBMC addons"
>
> +source "package/xbmc-addon-xvdr/Config.in"
>  source "package/xbmc-pvr-addons/Config.in"
>
>  endif # BR2_PACKAGE_XBMC
> --
> 1.8.3.2
>

As for the other patch, a quick review, build and test tomorrow :).