[vlc-devel] [PATCH] configure: disable SAPI if sapi.h cannot be used

[Steve Lhomme]

On Mon, Feb 1, 2016 at 3:31 PM, Rafaël Carré <funman at videolan.org> wrote:
> Hi,
>
> Shouldn't we check for sphelper.h, as well ?

You're right. I sent a patch to fix this.

> It is not yet in debian:
> https://packages.debian.org/sid/all/mingw-w64-x86-64-dev/filelist
>
> So the build is failing:
>
> ../../extras/package/win32/../../../modules/text_renderer/sapi.cpp:44:22: fatal
> error: sphelper.h: No such file or directory
>
> http://buildbot.videolan.org/builders/vlc-cont-win64-x86_64/builds/4660/steps/Compile/logs/stdio
>
> I can disable sapi in the buildbot while a new mingw-w64 appears in debian
>
> On 01/26/2016 04:21 PM, Steve Lhomme wrote:
>> From: Steve Lhomme <slhomme at matroska.org>
>>
>> for Winstore apps the code is hidden
>> ---
>>  configure.ac | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 1a26bf1..ba5d910 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -3273,10 +3273,10 @@ AM_CONDITIONAL([HAVE_HARFBUZZ], [test "${have_harfbuzz}" = "yes"])
>>  dnl
>>  dnl SAPI (text to Speech renderer for Windows)
>>  dnl
>> -AC_CHECK_HEADERS(sapi.h, [
>> +AC_CHECK_TYPES([ISpObjectToken],[
>>                have_sapi="yes"
>> -            ],[AC_MSG_WARN([sapi.h not found. Text to Speech renderer for Windows disabled])])
>> -
>> +            ],[AC_MSG_WARN([sapi.h not found. Text to Speech renderer for Windows disabled])],
>> +              [#include <sapi.h>])
>>  AM_CONDITIONAL([HAVE_SAPI], [test "${have_sapi}" = "yes"])
>>
>>  dnl
>>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel