Re: More ESD ./configure bogosity

ST> Why is the default "native, esd" and not "everything we can detect"? Good question. How do people feel about this more comprehensible user interface: --with-nas-sound=yes|no (default autodetect) --with-esd-sound=yes|no (default autodetect) --with-native-sound=yes|no (default autodetect) --with-sound=no is a convenience option equivalent to --with-nas-sound=no --with-esd-sound=no --with-native-sound=no --with-sound=yes is ignored (still autodetect all sound, or perhaps abort configure if no sound support can be autodetected) Here is a start.... (THIS UNTESTED PATCH DOES NOT WORK). Perhaps Robert Bihlmeyer would like to continue this work? --- configure.in 1999/12/08 07:43:35 1.111.2.53 +++ configure.in 1999/12/08 07:52:57 ＠＠ -3479,26 +3479,30 ＠＠ AC_EGREP_HEADER(AuXtErrorJump,audio/Xtutil.h,,[AC_DEFINE(NAS_NO_ERROR_JUMP)]) fi -AC_CHECK_PROG(have_esd_config, esd-config, yes, no) -if test -z "$with_esd"; then - with_esd=$have_esd_config -fi - -if test "$with_esd" = "yes"; then - if test "$have_esd_config" = "no"; then - echo "*** esd-config was not found in your PATH. Disabling esd support." - with_esd=no - else - AC_DEFINE(HAVE_ESD_SOUND) +dnl ESD Sound support +if test "$with_esd" != "no"; then + AC_CHECK_PROG(have_esd_config, esd-config, yes, no) + if test "$have_esd_config" = "yes"; then + esd_config_libs=`esd-config --libs` + save_libs_x="$libs_x" + XE_PREPEND(`esd-config --libs`, libs_x) + AC_CHECK_FUNC(esd_play_stream, + have_esd=yes, + have_esd=no libs_x="$save_libs_x") + fi + if test "$have_esd" = "yes"; then + with_esd=yes need_miscplay=yes XE_ADD_OBJS(esd.o) - XE_PREPEND(`esd-config --libs`, libs_x) + AC_DEFINE(HAVE_ESD_SOUND) + else + test "$with_esd" = "yes" && \ + XE_DIE("Required ESD module support cannot be provided.") + with_esd=no fi -fi - -if test "$need_miscplay" = "yes"; then - XE_ADD_OBJS(miscplay.o) fi + +test "$need_miscplay" = "yes" && XE_ADD_OBJS(miscplay.o) dnl --------------------- dnl TTY-dependent options