[Buildroot] [PATCH 1/1] package/tvheadend: bump version

Yann E. MORIN yann.morin.1998 at free.fr
Wed Sep 6 16:35:55 UTC 2017


On 2017-09-06 08:55 +0200, Thomas Petazzoni spake thusly:
> Hello,
> 
> On Tue, 5 Sep 2017 22:16:38 +0200, Yann E. MORIN wrote:
> > Thomas, All,
> > 
> > On 2017-09-05 22:13 +0200, Thomas Petazzoni spake thusly:
> > > On Mon,  4 Sep 2017 20:57:38 +0200, Bernd Kuhls wrote:  
> > > > Add optional dependencies to
> > > > 
> > > > - libva
> > > >   ffmpeg depends on libva, libva depends on xorg, so no need to add them
> > > >   to _DEPENDENCIES  
> > > 
> > > I don't see ffmpeg depending on libva, where do you see that ?  
> > 
> > Yet it does, see line 243. ;-)
> 
> Well, it will depend on libva if it's enabled. But I don't see anything
> in tvheadend or ffmpeg that forces libva to be enabled.

No, but the commit log says it is adding _optional_ dependency to libva.

So indeed nothing forces libva to be on, but if it is, then support is
explicitly enabled, otherwise it is explicitly disabled.

That's how I understand the patch.

But yes, the patch does more than one thing at a time, so should be
split.

Regards,
Yann E. MORIN.

> Perhaps I'm missing something obvious here.
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com

-- 
.-----------------.--------------------.------------------.--------------------.
|  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