Skip to content

Commit

Permalink
engine: wscript: move libasound uselib under linux condition, as the …
Browse files Browse the repository at this point in the history 
…ALSA interface is only relevant for Linux
  • Loading branch information
@a1batross
a1batross committed Oct 27, 2023
1 parent e3934af commit 48e44f0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion engine/wscript
View file
Original file line number Diff line number Diff line change
Expand Up @@ -144,12 +144,13 @@ def build(bld):
libs += ['USER32', 'SHELL32', 'GDI32', 'ADVAPI32', 'DBGHELP', 'PSAPI', 'WS2_32' ]
source += bld.path.ant_glob(['platform/win32/*.c'])
elif bld.env.DEST_OS not in ['dos', 'nswitch', 'psvita']: #posix
libs += [ 'M', 'RT', 'PTHREAD', 'ASOUND']
libs += [ 'M', 'RT', 'PTHREAD']
if not bld.env.STATIC:
libs += ['DL']
source += bld.path.ant_glob(['platform/posix/*.c'])

if bld.env.DEST_OS == 'linux':
libs += ['ASOUND']
source += bld.path.ant_glob(['platform/linux/*.c'])

if bld.env.DEST_OS == 'irix':
Expand Down

2 comments on commit 48e44f0

@nekonomicon
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ALSA interface is only relevant for Linux

It's incorrect because alsalib exists in userland of many other POSIX-compliant OSes.
And there no other native backends.

@a1batross
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nekonomicon wait so alsalib does exist for other systems? Ohhhh. That should be reverted then!

Please sign in to comment.