[vlc-devel] [PATCH 2/3] vlm: print deprecation notice upon loading

[Rémi Denis-Courmont]

Le lauantaina 19. syyskuuta 2020, 8.51.00 EEST Pierre Ynard via vlc-devel a 
écrit :
> diff --git a/src/input/vlm.c b/src/input/vlm.c
> index 116fc52..4b6707e 100644
> --- a/src/input/vlm.c
> +++ b/src/input/vlm.c
> @@ -127,6 +127,7 @@ vlm_t *vlm_New( libvlc_int_t *libvlc, const char
> *psz_vlmconf ) }
> 
>      msg_Dbg( p_this, "creating VLM" );
> +    msg_Info( p_this, "The VLM is deprecated. Instead, please consider
> using several separate VLC instances, or the task management and scheduling
> services provided by your operating system." );

1) That does not fit on a typical terminal window.

2) I would limit the recommendation to replacing broadcast media with VLC 
instances. The recommendation against schedules should probably be shown when 
creating a schedule only, if only to keep things concise.

3) While you can probably count their users with one hand, I don't think 
bridge and mosaic_bridge can work without VLM broadcasts at this stage, or can 
they? So those plugins need to be adjusted to move data between processes, or 
removed, before deprecating use of broadcasts.

>      p_vlm = vlc_custom_create( p_this, sizeof( *p_vlm ), "vlm daemon" );
>      if( !p_vlm )


-- 
Rémi Denis-Courmont
http://www.remlab.net/