search

dubhater / vapoursynth-decross

Spatio-temporal derainbow filter
5 stars 0 forks source link

Missing install patch in meson.build #1

Open marillat opened 5 years ago

marillat commented 5 years ago

Under LInux the filter is installed in /usr/lib when vapoursynth search for plugin in /usr/lib/vapoursynth This simple patch fix this issue

--- a/meson.build
+++ b/meson.build
@@ -53,4 +53,5 @@ shared_module('decross',
               dependencies: deps,
               link_args: ldflags,
               cpp_args: cflags,
+              install_dir : join_paths(get_option('prefix'), get_option('libdir'), 'vapoursynth'),
               install: true)
dubhater commented 5 years ago

Or you can change the libdir option when you compile it?

marillat commented 5 years ago

Or you can change the libdir option when you compile it?

Why ? /usr/lib/vapoursynth is the default path defined by vapoursynth build. From vapoursynth 45 ./configure --help

--with-plugindir        The default value for the configuration option
                          'SystemPluginDir' in vapoursynth.conf.
                          (default=LIBDIR/vapoursynth)
dubhater commented 5 years ago

Because then I don't have to change anything here. :sunglasses:

But if you make this a pull request I'll merge it.

marillat commented 5 years ago

I don't want to bother you. You are the author of several vapoursynth plugins with the same issue.