On Thu, 12 Oct 2017 19:17:32 +0200 Diego Biurrun <[email protected]> wrote:
> On Thu, Oct 12, 2017 at 01:02:57AM -0300, James Almer wrote: > > libxavs may require pthreads and libm at link time, and without > > said ldflags available as global extralibs, the check will fail. > > > > Regression since 7cb1d9e2dbbe5bf4652be5d78cdd68e956fa3d63 > > --- > > I tried replacing the require() check with a require_pkg_config() > > one, and while it included the pthreads ldflag when libxavs was > > compiled with pthreads support, it didn't include the libm one. > > Considering the project seems dead, trying to get the installed > > pkg-config file fixed is probably futile. > > configure | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- a/configure > > +++ b/configure > > @@ -4781,7 +4781,7 @@ enabled libx264 && require_pkg_config > > libx264 x264 "stdint.h x264.h" x > > -enabled libxavs && require libxavs "stdint.h xavs.h" > > xavs_encoder_encode -lxavs > > +enabled libxavs && require libxavs "stdint.h xavs.h" > > xavs_encoder_encode "-lxavs $pthreads_extralibs $libm_extralibs" > > Hmmmmm.... > > I don't like this. It's clearly a bug in libxavs that you are just working > around here. It should be fixed at the source. Even if libxavs itself may > be unfixable, it's certainly not unforkable. Forking would imply acquiring ownership, which I'm sure nobody wants to do. You'd be better off arguing dropping libaxavs - but for such a small reason? _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
