[vlc-devel] [PATCH 22/27] modules: use VLC_PLUGIN_PATH if it exists

[Romain Vimont]

On Thu, Jun 25, 2020 at 07:41:11PM +0300, Rémi Denis-Courmont wrote:
> Le torstaina 25. kesäkuuta 2020, 18.17.43 EEST Romain Vimont a écrit :
> > On Thu, Jun 25, 2020 at 05:37:42PM +0300, Rémi Denis-Courmont wrote:
> > > Le torstaina 25. kesäkuuta 2020, 15.23.09 EEST Romain Vimont a écrit :
> > > > Do not overwrite the VLC_PLUGIN_PATH environment variable if it exists.
> > > 
> > > Overriding is very much done on purpose. Otherwise, it would be disastrous
> > > when people develop with the variable set and everything in the build tree
> > > suddently starts failing.
> > > 
> > > -1
> > 
> > Would it be ok to force VLC_PLUGIN_PATH to "modules:doc" instead of
> > "modules" when the flag is enabled? (or always?)
> 
> That's almost as bad - stale plugins causing weird failures in build and 
> tests.

To explore all possibilities, what would you think about adding samples
in modules/samples/ instead of doc/?