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

[Rafaël Carré]

Hi,

Shouldn't we check for sphelper.h, as well ?

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
>