Re: [Mlt-devel] Bug#677276: kdenlive: Rendering to file works, Rendering by a script not.
Hey Dan and Vincent, @Vincent: You are right, with your video(settings) it is also reproduceable on my PC. @Dan: Could you have a look on this issue? The rendering about the script just finishes within a second and leaves an empty file. I tried to reproduce it with some other videos and settings, but they worked, but not with his generated script. Full history: http://bugs.debian.org/677276 Thanks! Am 11.07.2012 22:45, schrieb Vincent Smeets: Hello Patrick, I have put a small project together with just one dv file. The files are packed in the file http://dl.dropbox.com/u/30033528/Bug_677276_script002.tar.gz which I have provided on my DropBox. I couldn't send it by mail because of it's size. Before starting the script you will probably have to correct the path names in the files because that are still pathnames for my computer. The dv file is probably not necessary because my previous strace showed that the rendering already stopped before accessing the dv file. Regards, Vincent On 10-07-12 22:00, Patrick Matthäi wrote: Am 09.07.2012 20:56, schrieb Vincent Smeets: On 07-07-12 23:58, Patrick Matthäi wrote: could you also send me please the output of: dpkg -l|egrep libav|mlt|melt thanks! Hello Patrick, Here is the output: Could you please upload an example file and also both scripts so I may reproduce it? -- /* Mit freundlichem Gruß / With kind regards, Patrick Matthäi GNU/Linux Debian Developer E-Mail: pmatth...@debian.org patr...@linux-dev.org */ signature.asc Description: OpenPGP digital signature -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
[Mlt-devel] Fwd: Bug#681339: melt: wrongly looks for ffpresets - should look for avpresets
Whops, forget to add mlt in the to field...:
Thanks Jonas, you are right, the naming schema of ffmpeg/libav has been
changed.
@Dan:
I have attached a patch for it. I think there shouldn't be a problem to
apply it to Debian, or not?
For you it may be not complete, since it would break mlt with older
ffmpeg releases..
Am 12.07.2012 14:14, schrieb Jonas Smedegaard:
Package: melt
Version: 0.8.0-1
Severity: normal
Melt (and all of MLT) wrongly uses old ffmpeg path and extension for
libav* specific preset files.
Instead of looking for /usr/share/ffpresets/*.ffpreset it should look
for /usr/share/avconv/*.avpreset
- Jonas
--
/*
Mit freundlichem Gruß / With kind regards,
Patrick Matthäi
GNU/Linux Debian Developer
E-Mail: pmatth...@debian.org
patr...@linux-dev.org
*/
# The path and naming schema of the preset files has been changed from:
# /usr/share/ffpresets/*.ffpreset
# to:
# /usr/share/avconv/*.avpreset
# Closes: #681339
diff -Naur mlt-0.8.0.orig/src/modules/avformat/configure mlt-0.8.0/src/modules/avformat/configure
--- mlt-0.8.0.orig/src/modules/avformat/configure 2012-06-01 22:51:32.0 +0200
+++ mlt-0.8.0/src/modules/avformat/configure 2012-07-12 21:41:31.506776608 +0200
@@ -86,7 +86,7 @@
then
if [ -d $static_ffmpeg ]
then
- echo CFLAGS+=-DAVDATADIR=\\\${static_ffmpeg}/ffpresets/\\\ config.mak
+ echo CFLAGS+=-DAVDATADIR=\\\${static_ffmpeg}/avconv/\\\ config.mak
echo CFLAGS+=-I$static_ffmpeg config.mak
echo LDFLAGS+=-L$static_ffmpeg/libavformat -L$static_ffmpeg/libavcodec -L$static_ffmpeg/libavutil config.mak
[ $targetos = Darwin ]
diff -Naur mlt-0.8.0.orig/src/modules/avformat/consumer_avformat.c mlt-0.8.0/src/modules/avformat/consumer_avformat.c
--- mlt-0.8.0.orig/src/modules/avformat/consumer_avformat.c 2012-06-01 22:51:32.0 +0200
+++ mlt-0.8.0/src/modules/avformat/consumer_avformat.c 2012-07-12 21:42:49.242780570 +0200
@@ -731,12 +731,12 @@
AVCodec *codec = avcodec_find_encoder( c-codec_id );
if ( codec )
{
- char *path = malloc( strlen(AVDATADIR) + strlen(codec-name) + strlen(vpre) + strlen(.ffpreset) + 2 );
+ char *path = malloc( strlen(AVDATADIR) + strlen(codec-name) + strlen(vpre) + strlen(.avpreset) + 2 );
strcpy( path, AVDATADIR );
strcat( path, codec-name );
strcat( path, - );
strcat( path, vpre );
- strcat( path, .ffpreset );
+ strcat( path, .avpreset );
mlt_properties_close( p );
p = mlt_properties_load( path );
signature.asc
Description: OpenPGP digital signature
--
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel
Re: [Mlt-devel] Bug#677276: kdenlive: Rendering to file works, Rendering by a script not.
@Vincent: Does it also work for you, if you remove the file:// prefix JUST in the SOURCE variable of the .sh script? Looks like melt does not handle it prop. anymore, with the TARGET prefix file:// everything is okay. Am 12.07.2012 20:00, schrieb Patrick Matthäi: Hey Dan and Vincent, @Vincent: You are right, with your video(settings) it is also reproduceable on my PC. @Dan: Could you have a look on this issue? The rendering about the script just finishes within a second and leaves an empty file. I tried to reproduce it with some other videos and settings, but they worked, but not with his generated script. Full history: http://bugs.debian.org/677276 Thanks! -- /* Mit freundlichem Gruß / With kind regards, Patrick Matthäi GNU/Linux Debian Developer E-Mail: pmatth...@debian.org patr...@linux-dev.org */ signature.asc Description: OpenPGP digital signature -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
Re: [Mlt-devel] Fwd: Bug#681339: melt: wrongly looks for ffpresets - should look for avpresets
On Thu, Jul 12, 2012 at 12:51 PM, Patrick Matthäi pmatth...@debian.org wrote: Whops, forget to add mlt in the to field...: Thanks Jonas, you are right, the naming schema of ffmpeg/libav has been changed. @Dan: I have attached a patch for it. I think there shouldn't be a problem to apply it to Debian, or not? certainly not a problem for package-specific patch... For you it may be not complete, since it would break mlt with older ffmpeg releases.. yes, it should switch based on what it is built against. I do have a build-time heuristic for that, and I can apply it with this patch. The heuristic is checking a unofficial DEFINE for the purpose, so I want to minimize it usage of it, but some things like this seem unavoidable. Maybe it can migrate to something more official if something has been added since I last looked. Am 12.07.2012 14:14, schrieb Jonas Smedegaard: Package: melt Version: 0.8.0-1 Severity: normal Melt (and all of MLT) wrongly uses old ffmpeg path and extension for libav* specific preset files. Instead of looking for /usr/share/ffpresets/*.ffpreset it should look for /usr/share/avconv/*.avpreset - Jonas -- /* Mit freundlichem Gruß / With kind regards, Patrick Matthäi GNU/Linux Debian Developer E-Mail: pmatth...@debian.org patr...@linux-dev.org */ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel -- +-DRD-+ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
Re: [Mlt-devel] [PATCH] extend melted with file probe command
On Tue, Jul 3, 2012 at 5:35 AM, Maksym Veremeyenko ve...@m1stereo.tv wrote: hi, attached set of patches extend melted functionality with PROBE command that return information for specified file how it /supported/ by mlt framework. it gives a chance to check if file is OK before adding one to playlist. This looks good except a couple of things. First, we need to update the documentation, which I am willing to do. Secondly, I found the SGI MVCP protocol after which this is modeled (loosely) uses the CINF command: http://techpubs.sgi.com/library/tpl/cgi-bin/getdoc.cgi/0650/bks/SGI_Developer/books/MSB_RP/sgi_html/mvcp.html#item_CINF -- +-DRD-+ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
Re: [Mlt-devel] [PATCH] extend USTA output with current clip start time and playlist duration
On Wed, Jul 4, 2012 at 10:57 AM, Maksym Veremeyenko ve...@m1stereo.tv wrote: 03.07.12 18:50, Dan Dennedy написав(ла): On Tue, Jul 3, 2012 at 5:42 AM, Maksym Veremeyenkove...@m1stereo.tv wrote: Hi, attached patch USTA output with current clip start time and playlist duration. Two more parameters helps detect global playlist playback and remaining time. that breaks ABI with previous version so lib number should be incremented. ok another issue is older mvcp client will not support status reading from newer version - they will reject to parse USTA output due to increased number of entries. I think we should make a new command USTATUS that also includes a version or new text format that affords extensibility. if you plan to reject this patch, i can implement USTA2 and STATUS2 commands with appropriate mvcp_status2 but that will require duplicating mvcp_status2.[c|h] and mvcp_notifier2.[h|c]... I do not want to reject it and make a new command. Instead, I would like to establish a pattern to make the client backwards compatible and adapt to future additions at the protocol level. So, instead of if ( 19 == r || 17 == r ) I think we should make it like: if ( r = 17 ) // if valid response ... if ( r = 17 ) status-start = atoi( mvcp_tokeniser_get_string( tokeniser, 17 ) ); else status-start = -1; if ( r = 18 ) status-dur = atoi( mvcp_tokeniser_get_string( tokeniser, 18 ) ); else status-dur = -1; -- +-DRD-+ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
Re: [Mlt-devel] Bug#677276: kdenlive: Rendering to file works, Rendering by a script not.
On Thu, Jul 12, 2012 at 1:17 PM, Patrick Matthäi pmatth...@debian.org wrote: @Vincent: Does it also work for you, if you remove the file:// prefix JUST in the SOURCE variable of the .sh script? Looks like melt does not handle it prop. anymore, with the TARGET prefix file:// everything is okay. Melt does not and never has accepted a file:// prefix on a MLT XML file. File specifications being sent to the avformat plugin accept a file:// prefix because libavformat does and because it is the default plugin. This is not a regression as far as I know, and I have no plans to accommodate it. I just tested v0.7.8 and v0.7.0, and it does not accept it either. How did file:// get added into the script? Kdenlive generates the script, but it might be a side effect of some API it is using. Looks like on June 20, kdenlive switched from KUrl::url() to QUrl::toEncoded(). Not sure if that is the problem. Am 12.07.2012 20:00, schrieb Patrick Matthäi: Hey Dan and Vincent, @Vincent: You are right, with your video(settings) it is also reproduceable on my PC. @Dan: Could you have a look on this issue? The rendering about the script just finishes within a second and leaves an empty file. I tried to reproduce it with some other videos and settings, but they worked, but not with his generated script. Full history: http://bugs.debian.org/677276 Thanks! -- /* Mit freundlichem Gruß / With kind regards, Patrick Matthäi GNU/Linux Debian Developer E-Mail: pmatth...@debian.org patr...@linux-dev.org */ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel -- +-DRD-+ -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ Mlt-devel mailing list Mlt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mlt-devel
