[Buildroot] [PATCH v3 01/38] package/libdvdcss: add Kodi-specific patches

Yann E. MORIN yann.morin.1998 at free.fr
Sun Feb 5 21:24:09 UTC 2017


Bernd, All,

On 2017-02-05 08:59 +0100, Bernd Kuhls spake thusly:
> "Yann E. MORIN" <yann.morin.1998 at free.fr> wrote in 
> news:20170204230821.GA3805 at free.fr:
> 
> > Bernd, All,
> > 
> > On 2017-02-04 12:44 +0100, Bernd Kuhls spake thusly:
> >> The Kodi build system needs .a files to create
> >> usr/lib/kodi/system/players/VideoPlayer/libdvdcss-i486-linux.so
> > 
> > This is really nasty.
> > 
> > What happens if one does not have the .a library, but just the .so one
> > instead? Does the build really fail?
> 
> Yes.
> 
> > I've had a quick look at cmake/modules/FindLibDvd.cmake and it only
> > requires the .a files in the case that it compiles its own version.
> 
> Kodi 17 still uses autoconf, support for it will be dropped in Kodi 18.

Ah, OK, I was mislead then.

Indeed their code to handle the libdvdcss, libdvdnav and libdvdread is
just ugly. Han me my flamethrower...

Instead, I believe their code could be simplified (and fixed) using
this:

   73 $(SYSDIR)/libdvdcss-$(ARCH).so: $(WRAPPER) $(WRAPPER_DEF)
   74     [ -d libdvdcss ] || mkdir libdvdcss
   75     cd libdvdcss; $(AR) x $(DVDCSS_A)
   76     $(CC) -o $@ $(SO_LDFLAGS) -Wl,--soname,$@ -Wl,--unresolved-symbols=ignore-all -ldvdcss -lm \
   77         `cat $(WRAPPER_DEF)` $(WRAPPER)

Or something similar...

Ditto for the other lib.

Since this code is slated for removal upstream, I think it is acceptable
that we carry that kind of patch, especially since the replacement
CMakeList.txt looks like it is sane.

Could you test this, please?

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