[vlc-devel] [vlc-commits] contrib: use speexdsp 1.2rc2

Felix Paul Kühne fkuehne at videolan.org
Sun Dec 7 16:52:02 CET 2014


Hello Tristan,

> On 07.12.2014, at 00:12, Tristan Matthews <git at videolan.org> wrote:
> 
> vlc | branch: master | Tristan Matthews <tmatth at videolan.org> | Sat Dec  6 18:08:35 2014 -0500| [67066bf6dccc61ff15b6385dd80451a025a5217d] | committer: Tristan Matthews
> 
> contrib: use speedups 1.2rc2

This appears to be a bad idea tbh.

When compiling for neon, it breaks on resample.c because resample_neon.h is missing from the tarball.

Additionally, compilation also fails on buffer.c because 'speex/speex_buffer.h’ is not installed by speex 1.2rc2.

Seeing that I had to add SHA512SUMS manually, I’m left wondering if you actually tested your patches before committing them and even more before back-porting them to the _stable_ branch.

Please fix or revert.

Thanks,

Felix




More information about the vlc-devel mailing list