[FFmpeg-devel] [PATCH] Use SLIBSUF instead of .so in vf_frei0r

[Måns Rullgård]

V?ctor Paesa <victorpaesa at googlemail.com> writes:

> Hi,
>
> Patch attached to use a more generic dynamic library suffix.
>
> Regards,
> V?ctor
>
> Index: configure
> ===================================================================
> --- configure	(revision 25211)
> +++ configure	(working copy)
> @@ -3214,6 +3214,7 @@
>  #define ASMALIGN(ZEROBITS) ".p2align " #ZEROBITS "\\n\\t"
>  #define EXTERN_PREFIX "${extern_prefix}"
>  #define EXTERN_ASM ${extern_prefix}
> +#define SLIBSUF "$SLIBSUF"
>  EOF

OK

>  test -n "$malloc_prefix" &&
> Index: libavfilter/vf_frei0r.c
> ===================================================================
> --- libavfilter/vf_frei0r.c	(revision 25211)
> +++ libavfilter/vf_frei0r.c	(working copy)
> @@ -191,7 +191,7 @@
>  {
>      char path[1024];
>  
> -    snprintf(path, sizeof(path), "%s%s.so", prefix, name);
> +    snprintf(path, sizeof(path), "%s%s%s", prefix, name, SLIBSUF);
>      av_log(ctx, AV_LOG_DEBUG, "Looking for frei0r effect in '%s'\n", path);
>      return dlopen(path, RTLD_NOW|RTLD_LOCAL);
>  }

Is this correct on macos too?  That is, are the plugins named *.dylib there?

-- 
M?ns Rullg?rd
mans at mansr.com