Bug 388503 - Changing "acodec=vorbis" to "acodec=libvorbis" fixes a SIGSEGV
Summary: Changing "acodec=vorbis" to "acodec=libvorbis" fixes a SIGSEGV
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Display & Export (show other bugs)
Version: 17.12.0
Platform: Debian unstable Linux
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
: 388696 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-01-03 23:22 UTC by kingsley
Modified: 2018-01-13 20:33 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kingsley 2018-01-03 23:22:55 UTC
First of all, thanks again for maintaining kdenlive.

I love how expressive and open it is.

The main reason I'm writing is to humbly suggest an improvement.

When asked to render to the .webm format, kdenlive currently passes

    acodec=vorbis
    
to melt.

I saw it in a kind of log file named something like

    <output-file-name>,webm.txt

Maybe the line of kdenlive's source code is

    https://github.com/KDE/kdenlive/blob/640d4467558db92b5d2dbcc0baaf74a4a5f664c2/data/profiles.xml#L6

But but but!

Rendering to .webm, crashes!

My original bug report is at

    https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884268

Why does it crash?

ffmpeg's web site says the vorbis audio encoder 

    a.) is experimental and 

    b.) not recommended.

You can see this is so by searching for 

    "Please note it is not recommended to use the experimental vorbis for Vorbis encoding; use libvorbis instead."

at

    https://trac.ffmpeg.org/wiki/Encode/HighQualityAudio#AudioencodersFFmpegcanuse

I tried it..

I changed line 6 of

    /usr/share/kdenlive/export/profiles.xml

from

   acodec=vorbis

to

   acodec=libvorbis

It worked!

It rendered to .webm!

Maybe you should change it too.

Thanks again,
Kingsley

PS: Someone using the nick name "furq" on freenode's #ffmpeg channel was the source of much of the above info. furq also seemed to think kdenlive really should be using libopus. I tested it. It didn't seg fault. But the video was silent. I didn't try libopus again, but furq suggested changing quality=good and aq=%audioquality, using something like -ab 128k, and said he didn't know what -quality good even does. He'd never heard of it.
Comment 1 Jean-Baptiste Mardelle 2018-01-11 08:59:19 UTC
Git commit 5de2633cc59424eb64968534d01ebe4e9186830f by Jean-Baptiste Mardelle.
Committed on 11/01/2018 at 08:59.
Pushed by mardelle into branch 'Applications/17.12'.

Fix webM encoding (auto replace vorbis by libvorbis if installed)

M  +1    -2    src/dialogs/renderwidget.cpp

https://commits.kde.org/kdenlive/5de2633cc59424eb64968534d01ebe4e9186830f
Comment 2 Ovidiu-Florin BOGDAN 2018-01-13 20:33:21 UTC
*** Bug 388696 has been marked as a duplicate of this bug. ***