Update FindOpenAL.cmake (emscripten-core#9830)

To follow the docs which indicate -lopenal is recommended, https://emscripten.org/docs/porting/Audio.html
hugoam · Nov 14, 2019 · 1b0bd8b · 1b0bd8b
1 parent fbb2897
commit 1b0bd8b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/AUTHORS b/AUTHORS
@@ -436,3 +436,4 @@ a license to everyone to use it as detailed in LICENSE.)
 * Jaikanth J <[email protected]>
 * Gernot Lassnig <[email protected]>
 * Christian Boos <[email protected]>
+* Erik Scholz <[email protected]>
diff --git a/cmake/Modules/FindOpenAL.cmake b/cmake/Modules/FindOpenAL.cmake
@@ -15,10 +15,9 @@ if (NOT OPENAL_FOUND)
 	# ... but the stock FindOpenAL.cmake would have returned this.
 	#SET(OPENAL_INCLUDE_DIR "${EMSCRIPTEN_ROOT_PATH}/system/include/AL")
 
-	# No library to link against for OpenAL, this is picked up automatically by library_openal.js,
-	# but need to report something, or CMake thinks we failed in the search.
-	SET(OPENAL_LIBRARY "nul")
-	SET(OPENAL_LIB "")
+	# Returning "-lopenal" is now considered mandatory
+	SET(OPENAL_LIBRARY "-lopenal")
+	SET(OPENAL_LIB "-lopenal")
 
 	set(CMAKE_INCLUDE_PATH ${CMAKE_INCLUDE_PATH} "${EMSCRIPTEN_ROOT_PATH}/system/include" "${EMSCRIPTEN_ROOT_PATH}/system/include/AL")