search

cntools / cnfa

Foundational audio drivers
MIT License
32 stars 16 forks source link

Move logic for platform selection to .h file. Refs #23 #25

Closed cnlohr closed 6 months ago

cnlohr commented 6 months ago

Also fix compile for wave_player on Linux

sellicott commented 6 months ago

This does look cleaner to me, I haven't tested for functionality on my system though.

AEFeinstein commented 6 months ago

I tested these changes on Windows, msys2, and they work fine. I did have to manually compile it though. -lasound fails and -lwinmm is required. Might be nice to update the makefile to be OS aware. You can see an example of that here:

https://github.com/AEFeinstein/Super-2024-Swadge-FW/blob/main/makefile

cnlohr commented 6 months ago

Added OS detection, I plan to merge in a few hours.