Commit 13fb436d authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

Turn --services-discovery into a string advanced option

Modules list was intended for plain static module names. It does not
work right for most SD plugins, including:
 - LUA (dependent on string chain),
 - udev (submodules),
 - media directories (submodules).

It works but is totally useless for those plugins that are fast,
including XCB screen capture and Windows optical drives.

For network-based discovery and MTP, it could be used to prefetch
items. But this is not necessarily a good idea as it sends extra
network traffic and consumes memory even when the services discovery
is not used. Furthermore, podcast can prevent Qt4 and Skins2 UI from
starting, as it depends on HTTP which depends on libproxy which
depends on Xlib.
parent 89a6c143
......@@ -1245,8 +1245,8 @@ static const char *const ppsz_albumart_descriptions[] =
#define SD_TEXT N_( "Services discovery modules")
#define SD_LONGTEXT N_( \
"Specifies the services discovery modules to load, separated by " \
"colons. Typical values are sap, hal, ..." )
"Specifies the services discovery modules to preload, separated by " \
"colons. Typical value is \"sap\"." )
#define RANDOM_TEXT N_("Play files randomly forever")
#define RANDOM_LONGTEXT N_( \
......@@ -2118,8 +2118,7 @@ vlc_module_begin ()
ppsz_albumart_descriptions )
set_subcategory( SUBCAT_PLAYLIST_SD )
add_module_list_cat( "services-discovery", SUBCAT_PLAYLIST_SD, NULL,
SD_TEXT, SD_LONGTEXT, false )
add_string( "services-discovery", "", SD_TEXT, SD_LONGTEXT, true )
change_short('S')
/* Interface options */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment