<div dir="ltr">Hi,<br><div><br>On Sun, Dec 7, 2014 at 10:52 AM, Felix Paul Kühne <span dir="ltr"><<a href="mailto:fkuehne@videolan.org" target="_blank">fkuehne@videolan.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Tristan,<br>
<br>
> On 07.12.2014, at 00:12, Tristan Matthews <<a href="mailto:git@videolan.org">git@videolan.org</a>> wrote:<br>
><br>
> vlc | branch: master | Tristan Matthews <<a href="mailto:tmatth@videolan.org">tmatth@videolan.org</a>> | Sat Dec  6 18:08:35 2014 -0500| [67066bf6dccc61ff15b6385dd80451a025a5217d] | committer: Tristan Matthews<br>
><br>
> contrib: use speedups 1.2rc2<br>
<br>
This appears to be a bad idea tbh.<br>
<br>
When compiling for neon, it breaks on resample.c because resample_neon.h is missing from the tarball.<br>
<br>
Additionally, compilation also fails on buffer.c because 'speex/speex_buffer.h’ is not installed by speex 1.2rc2.<br>
<br>
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.<br></blockquote><div><br></div><div>I had tested locally but only a x86-64 build (not NEON) and a) forgot to git add the sha512sums and b) wasn't aware of the missing header bug in speexdsp until testing without having speex installed on my system.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Please fix or revert.<br>
<br></blockquote><div><br></div><div>Sorry about that, I'll revert the speexdsp change until I've properly fixed the tarball (didn't now about the resample_neon.h header being missing either)..<br><br></div><div>Best,<br></div><div>Tristan<br></div></div></div></div></div>