Menu

#26 Solaris build wrongly uses /dev/dsp

closed-fixed
None
5
2004-11-22
2004-11-18
Mark Round
No

Hi,

When building on Solaris (gcc or Sun's CC), the
resulting binary attempts to open /dev/dsp, which does
not exist on Solaris. I can see through a quick look at
the source that you do support Sun's /dev/audio device
- but for some reason, this is not getting picked up by
the build process.

This occurs on Solaris 8, both Sparc and Intel.

Discussion

  • Philip Kendall

    Philip Kendall - 2004-11-18

    Logged In: YES
    user_id=29214

    * Which version of Fuse are you using?
    * Which user interface are you using?
    * Could you post your config.h (crucially, the lines
    mentioning HAVE_SYS_SOUNDCARD_H and HAVE_SYS_AUDIO_H)?
    * Do <sys/soundcard.h> and/or <sys/audio.h> exist on your
    system?

    Cheers!

     
  • Mark Round

    Mark Round - 2004-11-18

    config.h from Solaris x86

     
  • Mark Round

    Mark Round - 2004-11-18

    Logged In: YES
    user_id=1161605

    Version of fuse : 0.7.0
    User interface : GTK2
    Config.h - attached

    bash-2.03$ ls -l /usr/include/sys/soundcard.h
    -rw-r--r-- 1 root other 61889 Sep 14 2003
    /usr/include/sys/soundcard.h
    bash-2.03$ ls -l /usr/include/sys/audio.h
    -rw-r--r-- 1 root bin 2504 Jun 2 06:38
    /usr/include/sys/audio.h

    Thanks for looking at this :)

    -Mark

     
  • Philip Kendall

    Philip Kendall - 2004-11-18

    Logged In: YES
    user_id=29214

    The "problem" is that <sys/soundcard.h> exists; the Solaris
    box which I formerly had access didn't have <sys/soundcard.h>.

    Given that none of the OSS-using boxes I have access to have
    <sys/audio.h>, swapping the order of the tests in
    sound/lowlevel.h should be OK. Can you apply the attached
    patch and tell me if it fixes the problem?

     
  • Philip Kendall

    Philip Kendall - 2004-11-18

    Swap order of <sys audio.h=""> and <sys soundcard.h=""> tests</sys></sys>

     
  • Mark Round

    Mark Round - 2004-11-18

    Logged In: YES
    user_id=1161605

    Excellent - this looks like it works, thanks. Both the Sparc
    and x86 builds now try to open /dev/audio (I can't verify
    this actually works, as I don't have sound on my Sun boxes -
    but I will get friends to test on theirs). Thanks again for
    the quick response - and for a quality bit of software in
    the first place... I'm now off to dust off my Manic Miner
    skills :)

     
  • Philip Kendall

    Philip Kendall - 2004-11-18

    Logged In: YES
    user_id=29214

    Cool. I'll commit the patch to HEAD (on, say, Monday) unless
    anyone thinks it will break something else.

    In the longer term, we should probably have configure
    options to select the sound code used.

     
  • Philip Kendall

    Philip Kendall - 2004-11-22

    Logged In: YES
    user_id=29214

    Patch applied, bug closed.

     
  • Philip Kendall

    Philip Kendall - 2004-11-22
    • assigned_to: nobody --> pak21
    • status: open --> closed-fixed
     

Log in to post a comment.