[PATCH] [alsa-utils] Fix build with library options disabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

It's been a perennial problem with alsa-utils that there is no
configure option to disable the seq and amidi apps. These apps depend
on how you configured alsa-lib: If you pass --disable-seq
--disable-instr to the alsa-lib configure, there does not currently
exist a configure-time option to allow alsa-utils to build against
this particular alsa-lib.

Of course, you could hack makefiles and stuff so it doesn't build seq
and amidi, but I thought it would be nice to commit changes to the
configure.in and Makefile.am to upstream, so that we can disable seq
and amidi the "right" way.

The attached patches are diffed against the latest revision of
alsa-utils. With this change, you can:

1. Disable building amidi by passing --disable-midi to the configure
in alsa-utils.
2. Disable building seq by passing --disable-seq to the configure in alsa-utils.

With this change, we can get a "good" build of alsa-utils (albeit,
missing two of the utilities, which is intended) in the following
circumstances:

1. seq and instr are DISABLED in alsa-lib; seq and midi are DISABLED
in alsa-utils
2. seq and instr are ENABLED in alsa-lib; seq and midi are ENABLED in
alsa-utils (the default).
3. seq and instr are ENABLED in alsa-lib; seq and midi are DISABLED in
alsa-utils.

Of course, it does not build cleanly if seq and instr are DISABLED in
alsa-lib, but seq and midi are ENABLED in alsa-utils.

Patches are attached -- please let me know if there any issues, and if
not, could one of the committers please commit them.

Thanks,

Sean

Attachment: patch-alsa-utils-no-midi.tar.bz2
Description: BZip2 compressed data

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux