[Buildroot] [PATCH 2/2] pacage/xbmc-addon-xvdr: new package
Yann E. MORIN
yann.morin.1998 at free.fr
Tue Apr 1 22:54:27 UTC 2014
Maxime, All,
On 2014-04-02 00:41 +0200, Maxime Hadjinlian spake thusly:
> 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 ?
I did not look into it so far. I can have a look at it later, but I'm
currently focusing on using tvheadend for my own setup...
> > 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 ?
It has already been accepted:
https://github.com/pipelka/xbmc-addon-xvdr/commit/84e5860f488e67a3a3386c779f9a56adeb3dc220
and more:
https://github.com/pipelka/xbmc-addon-xvdr/pull/130
I have already added this info in the patch I have in my branch:
https://www.gitorious.org/buildroot/buildroot/commit/d419c3eb9a0bcfeb32f81b08126fc1c5bd25dfa8
> > 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.
Same brain-fart on your side, again. ;-)
> > +# 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.
OK.
> > +# 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 :).
Thanks, will fix the typo.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list