dubhater / vapoursynth-nnedi3

nnedi3 filter for VapourSynth
92 stars 6 forks source link

add '-lm' to linker flags #6

Closed darealshinji closed 9 years ago

dubhater commented 9 years ago

The question is, why do you need this?

If I'm not mistaken, extra libraries should go in libnnedi3_la_LIBADD instead of libnnedi3_la_LDFLAGS.

darealshinji commented 9 years ago

I've added it to libnnedi3_la_LIBADD and removed it from libnnedi3_la_LDFLAGS.

dubhater commented 9 years ago

You still haven't explained why you need this. If it fails to compile or run, say so.

darealshinji commented 9 years ago

The plugin uses symbols from libm but it doesn't automatically link against it.

dubhater commented 9 years ago

I didn't like your solution.