On Sun, Feb 14, 2016 at 5:04 AM Jean-Baptiste Mardelle <j...@kdenlive.org>
wrote:
> > Hi!
> >
> > I accidentally found an issue that probably caused several bug reports
> > against movit!
> >
> > Looks like it does not correctly handle locales (the infamous dot/comma
> > issue)...
> >
> > Playing a clip in terminal:
> >
> > melt myclip.mp4 .glsl=1 -attach movit.saturation saturation=0 -consumer
> > xgl
> >
> > Works fine.
> > But if I use a locale with comma separator:
> >
> > LC_ALL=fr_FR.UTF-8 melt myclip.mp4 .glsl=1 -attach movit.saturation
> > saturation=0 -consumer xgl
> >
> > I get the following:
> > Movit system requirements: Needs at least GLSL version 1.30 (has version
> > 1,0)
> > [filter avcolor_space] Invalid format glsl
> >
> > It clearly looks like the locale is not handled correctly, making it
> > believe it has an older GLSL (1.0) version. Not sure if the problem is in
> > MLT or in movit, will keep you posted if I find where it is.
>
>
> After some tests, here is a solution:
> When used through MLT, Movit's locale is set to the current user's locale.
> The version check is than made in movit (init.cpp) with an atof with is
> locale sensitive.
>
> So trying to parse "1.30" fails on comma as numerical separator.
>
> Patching movit with the patch below fixes the problem for me (patch against
> movit git master).
>
>
FYI - this patch was applied to Movit Git recently:
https://git.sesse.net/?p=movit;a=commit;h=fc1e9e9f7f047e348a34fd691ceda0cd383e9d4f
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel
