Re: [Libvisual-devel] lvdisplay, libvisual auto* patch

[Dennis S. <sy...@yo...>]

Patch has been merged! Thanks a lot it's a nice cleanup.

the LIBVISUAL_PLUGIN_BASE_DIR path has also been merged.

however I don't think it's a good idea to make the loader
recursive. I gave the reason in an earlier post.

Btw, do you have time to do this auto cleanup style for
the other packages as well ? 

Thanks a lot,
Dennis


On Thu, 2004-11-25 at 20:19 +0200, Vitaly V. Bursov wrote:
> regarding to autotools. Patch modifies/adds these options
> --enable-debug
> --enable-extra-optimization
> 
> it also defines LIBVISUAL_PLUGINS_BASE_DIR variable to specify base
> dir for plugins. i.e. dir that contains actor, input, morph.
> 
> Please review.
> 
> If you'll keep BASE_DIR portion of a patch, it would be nice if
> libvisual's plugin loader loaded  LIBVISUAL_PLUGINS_BASE_DIR/*/*.so
> libraries, not LIBVISUAL_PLUGINS_BASE_DIR/{actor,...}/*.so.
> 
> ps. lvdisplay autotoolsification is almost done :) it only depends
> on what you, Dennis, think about LIBVISUAL_PLUGINS_BASE_DIR etc.
>