Merge pull request #260 from gusano/topic/LadspaUGen

Make LadspaUGen optional

Author: mossheim

CMakeLists.txt

@@ -64,6 +64,7 @@ else()
 endif()
 
 option(AY "Build with AY ugens" ON)
+option(LADSPA "Build with Ladspa ugen" ON)
 option(QUARKS "Install plugins as quarks")
 option(OSX_PACKAGE "Package dmg for Apple")
 option(IN_PLACE_BUILD "Build and install in cmake build folder" ON)

README.md

@@ -84,19 +84,27 @@ ambisonics) subproject.
     * (default on linux `/usr/local`)
     * `cmake -DCMAKE_INSTALL_PREFIX=/usr/local ..`
 + Install in cmake build folder instead of `CMAKE_INSTALL_PREFIX`
-    * (OSX ONLY, default=ON)
+    * (macOS ONLY, default=ON)
     * `cmake -DIN_PLACE_BUILD=ON`
 + Build the plugins as quarks
-    * (default 'OFF')
+    * (default=OFF)
     * `cmake -DQUARKS=ON ..`
 + Build supernova-plugins
-    * (default 'OFF')
+    * (default=OFF)
     * `cmake -DSUPERNOVA=ON ..`
++ Build Ladspa UGen
+    * (macOS AND LINUX ONLY, default=ON)
+    * `sc3-plugins/build/$ cmake -DLADSPA=ON ..`
 + Print all cmake options
     * `sc3-plugins/build/$ cmake -L ..`
 
 ### Troubleshooting
 
+#### Known issues
+
+**On some Linux distributions** it has been reported that Ladspa UGen could prevent SuperCollider server from booting ([sc3-plugins/#210](https://github.com/supercollider/sc3-plugins/issues/210), [sc3-plugins/#23](https://github.com/supercollider/sc3-plugins/issues/23) and [supercollider/#4421](https://github.com/supercollider/supercollider/issues/4421)).
+See above how to build sc3-plugins without Ladspa if needed.
+
 #### Build errors
 
 If you get an error while building that files are missing, it probably means that you didn't clone all the SuperCollider

source/CMakeLists.txt

@@ -115,7 +115,6 @@ set(PLUGIN_DIRS_EXTRA
     BetablockerUGens
     ChaosUGens
     DEINDUGens
-    LadspaUGen
     MembraneUGens
     NCAnalysisUGens
     OteyPianoUGens
@@ -243,7 +242,8 @@ BUILD_PLUGIN(MembraneUGens "MembraneUGens/Membrane_shape.c;MembraneUGens/Membran
              "" "MembraneUGens")
 
 # LadspaUGen
-if(NOT WIN32)
+if(LADSPA AND NOT WIN32)
+    list(APPEND PLUGIN_DIRS_EXTRA LadspaUGen)
     BUILD_PLUGIN(LadspaUGen "LadspaUGen/LadspaUGen.cpp;LadspaUGen/search.c" "${CMAKE_DL_LIBS}" "LadspaUGen")
     add_executable(ladspalist LadspaUGen/ladspalist.c LadspaUGen/search.c)
     target_link_libraries(ladspalist "${CMAKE_DL_LIBS}")
@@ -357,7 +357,7 @@ if (NOT APPLE)
             DESTINATION ${INSTALL_DESTINATION})
 endif()
 
-if(NOT WIN32)
+if(LADSPA AND NOT WIN32)
   install(TARGETS ladspalist
     DESTINATION ${INSTALL_DESTINATION_LADSPALIST})
 endif()