← Back to team overview

mythbuntu-bugs team mailing list archive

[Bug 647914] Re: Mythexport doesn't call ffmpeg for OTG exports

 

I have pushed the fix for this bug to the Mythbuntu Testing PPA:
https://launchpad.net/~mythbuntu/+archive/testing

Please leave any feedback if you are able to test it.

-- 
Mythexport doesn't call ffmpeg for OTG exports
https://bugs.launchpad.net/bugs/647914
You received this bug notification because you are a member of Mythbuntu
Bug Team, which is subscribed to mythexport in ubuntu.

Status in MythExport: Fix Released
Status in “mythexport” package in Ubuntu: Fix Released

Bug description:
Binary package hint: mythexport

After selecting a recording for an OTG (On The Go) export, mythexport processes the request, but never calls ffmpeg. This is because of a but in the code around line 340:

    foreach (@params){

        if ($_ =~ m/starttime/) {
            $starttime = (split(/\=/,$_))[1];
        }
        elsif ($_ =~ m/chanid/) {
            $chanid = (split(/\=/,$_))[1];
        }
        elsif ($_ =~ m/config/) {
            $block = (split(/\=/,$_))[1];
        }
#        elsif ($_ =~ m/otg/) {
#            $block = (split(/\=/,$_))[1];
#        }
    }


I commented out the last elsif to correct the issue, however this may cause a problem elsewhere, since I don't know what the original intent for parsing the value of otg was. With these three lines uncommented, $block is then set to "true", not the name of the configuration to use. As such, several lines just below this block of code fail to retrieve parameters from the config file, including the following:

my $args = $export_cfg->param("$block.ffmpegArgs") unless ref $export_cfg->param("$block.ffmpegArgs") eq 'ARRAY';

Since the code ends up looking for "true.ffmpegArgs" instead of "Steve_Podcast.ffmpegArgs", $args is empty. This causes the @args array to be blank and the foreach loop around line 485 is never entered.

Perhaps some extra debug code should be added to exit earlier when parsing the config fails?

NOTE: my line numbers are estimates, since I added several "logdebug" lines while tracking this down.





References