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

Hugo Beauzée-Luyssen hugo at beauzee.fr
Fri Jan 6 12:53:34 UTC 2017


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!


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(-)


More information about the flac-dev mailing list