This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118545/ |
On June 5th, 2014, 10:58 a.m. UTC, David Faure wrote:
Correct, video/* isn't supported. In any case, how can you be sure that this application will support *ALL* video codecs?On June 5th, 2014, 11:05 a.m. UTC, Hrvoje Senjan wrote:
i cannot - this was used in 4.x version, i've added a few specific ones validated they work. will adjust the review, and remove the /* completely...
Unless I'm wrong, video/* didn't work in kdelibs4 either. (IIRC it worked for ServiceMenus, but not for application desktop files, which have to follow the Desktop Entry Spec).
- David
On June 5th, 2014, 10:47 a.m. UTC, Hrvoje Senjan wrote:
Review request for KDE Multimedia and David Faure.
By Hrvoje Senjan.
Updated June 5, 2014, 10:47 a.m.
Repository:
ffmpegthumbs
Description
Testing
Diffs
|