[Mplayer-dev-eng] Sun audio trouble broke CVS build

[Marcus Comstedt]

>>>>> "Bertrand" == Bertrand BAUDET <bertrand_baudet at yahoo.com> writes:

  Bertrand> The script just check in /dev for filename! In my case,
  Bertrand> I have both /dev/audio & /dev/audioctl which can
  Bertrand> be the same on most of Un*x bases system. 

  Bertrand> I don't think an autodection based on filename is a good thing,
  Bertrand> we must use the same way as for detecting ALSA drivers or ESD
  Bertrand> -ie- by doing a small c file that test for specific include file or
  Bertrand> variable name or for a lib at compilation time.

[...]

  Bertrand> And I think this will be the same trouble with OSS audio, just
  Bertrand> checking for /dev/dsp is not enough!

Both the Sun audio and OSS test should check for header files
instead.  It's the headers that allow you to compile the module.
Whether the /dev node exists should just be a runtime test.

Checking for header files in a reliable way is simple enough once the
current configure hack has been thrown out in favour of a real
autoconf script.


  // Marcus




_______________________________________________
Mplayer-dev-eng mailing list
Mplayer-dev-eng at lists.sourceforge.net
http://lists.sourceforge.net/lists/listinfo/mplayer-dev-eng