mythbuntu-bugs team mailing list archive

[John Baab <rhpot1991@xxxxxxxxxx>]

This one was already fixed in trunk, the code section should look something like this:
elsif ($_ =~ m/otg/) {
    $otg = (split(/\=/,$_))[1];
}

I will push this fix along with some of the others to the testing ppa
once I have verified them.

** Changed in: mythexport (Ubuntu)
       Status: New => Fix Released

** Also affects: mythexport
   Importance: Undecided
       Status: New

** Changed in: mythexport
       Status: New => Fix Released

-- 
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.