[flac-dev] [PATCH 0/5] Allow multiple targets to be disabled

Hugo Beauzée-Luyssen hugo at beauzee.fr
Sat Jan 14 08:35:41 UTC 2017


On 01/06/2017 01:53 PM, Hugo Beauzée-Luyssen wrote:
> Hi,
> 
> This patchet allows a few targets to be disabled when unrequired.
> The rational is coming from VLC's contrib buildsystem, so far we use make -C to select only some subparts of the available targets.
> It would be easier and cleaner to use autoconf to do so IMHO.
> 
> There's an additional patch which fixes the build when building for WinRT/UWP platform, upstreamed from VLC. 
> We have a couple more for this platform, but ultimately, I think the proper solution would be to allow only libflac to be built, as most forbidden function calls are in the convenience libraries in flac/shared.
> I tried to switch those from noinst_LTLIBRARIES to EXTRA_LTLIBRARIES but failed miserably at doing so :( I'm thinking it would be way easier to achieve by moving to a non-recursive
> automake configuration, but that's kind of an intrusive change; that being said, I'm willing to do it.
> 
> Please let me know what you think!
> 
> Regards,
> 
> Hugo Beauzée-Luyssen (5):
>   configure.ac: Allow the programs to be disabled
>   configure.ac: Allow bench to be disabled
>   configure.ac: Don't build any tests when they are explicitely disabled
>   configure.ac: Allow examples to be disabled
>   win_utf8_io: Avoid forbidden functions when building for WinRT/UWP
> 
>  configure.ac                        | 10 ++++++++++
>  examples/Makefile.am                |  2 ++
>  microbench/Makefile.am              |  4 ++++
>  src/Makefile.am                     | 10 +++++++---
>  src/flac/Makefile.am                | 13 +++++++++----
>  src/metaflac/Makefile.am            |  8 ++++++--
>  src/share/win_utf8_io/win_utf8_io.c |  9 +++++++++
>  src/test_libFLAC++/Makefile.am      |  5 +++++
>  8 files changed, 52 insertions(+), 9 deletions(-)
> 

Ping :)

-- 
Hugo Beauzée-Luyssen



More information about the flac-dev mailing list