mythexport generates names like ---20100418192800 and thus fails ffmpeg.

Bug #598439 reported by DaveQB
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
mythexport (Ubuntu)
Fix Released
Undecided
rhpot1991

Bug Description

Binary package hint: mythexport

Running user jobs from myth frontend run, but the same jobs won't run from mythexport.
By line 230 it has the right name, by line 430 it has lost channame, title, subtitle and currentstarttime and producs --- as the name which in turn borks ffmpeg and errors out.

I have logs to show this where I have added some additional logdebug() lines to help narrow it down but don't understand the script (nor perl) well enough to work out the problem.

Revision history for this message
DaveQB (david-dward) wrote :
Revision history for this message
DaveQB (david-dward) wrote :

It seems the DB query around like 416 for channame comes back empty. Yet earlier on line 215 it had it correct.

Why is it accessing the DB again? Seems a lot of doubled up work in this script, but like I said in the original report, I don't understand it well enough to follow what it is trying to do and limited commenting makes it harder to catch up.

Revision history for this message
rhpot1991 (rhpot1991) wrote :

What type of job are you running? Mind pasting your config /etc/mythtv/mythexport/mythexport_settings.cfg ?

Revision history for this message
DaveQB (david-dward) wrote :

[Desktop2]
removeCommercials=
extension=.avi
codec=
sizeY=
sizeX=
aspect=
videoBR=
threads=
device=custom
podcastName=
deletePeriod=
audioBR=
ffmpegArgs=-y -acodec libmp3lame -ab 128kb -ar 44100 -ac 2 -vcodec mpeg4 -b 1500kb -mbd 2 -trellis 2 -s 720x400
audioChannels=
deinterlace=

That is that I am running.
It runs fine if I select it using Myth Frontend. And I have tried it manually on the command line with an output WITHOUT leading dashes and it works. If I set the output to "---something.avi" or "/somewhere/here/---something.avi" ffmpeg complains and errors straight away.

Not sure why mythtv frontend can name it correctly and mythexport web interface can not.

Revision history for this message
rhpot1991 (rhpot1991) wrote :

When you say the job wont run from MythExport, are you running an On The Go job there?

rhpot1991 (rhpot1991)
Changed in mythexport (Ubuntu):
status: New → Incomplete
Revision history for this message
DaveQB (david-dward) wrote :

Hi John,

That is a good question. I can't recall off the top of my head as I haven't touched MythExport since logging this ticket.
I will job my memory tonight and report back.

Sorry for the delay.

Thanks.

Revision history for this message
DaveQB (david-dward) wrote :

John,

Had a look and yes. I am ticking multiple shows to encode.
I am now resorting to using MythFrontend and setting each show up one at a time. A very slow process if you have 20 shows to do, but it works.

Thanks
If you need any more info, let me know.

Revision history for this message
Steve Switzer (steve-switzer) wrote :

See bug report 647911, 647908 and 647914 for a full explanation of each symptom here, and how to fix them. I'm on the fence about if each should be marked as a duplicate.

Changed in mythexport (Ubuntu):
status: Incomplete → Confirmed
rhpot1991 (rhpot1991)
Changed in mythexport (Ubuntu):
status: Confirmed → In Progress
assignee: nobody → John Baab (rhpot1991)
rhpot1991 (rhpot1991)
Changed in mythexport (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
rhpot1991 (rhpot1991) wrote :

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.

Revision history for this message
Dave Walker (davewalker) wrote :

mythexport (2.2.2-0ubuntu1) maverick; urgency=low

  [ John Baab ]
  * debian/mythexport.config and debian/mythexport.templates
    - Added warning about new configuration
  * usr/bin/mythexport-daemon
    - Fixed issue with OTG exports (LP #598439)
    - Added lockfile, to keep mythtv-backend informed of the job status (LP #596251)
    - Removed commented out code from past release
  * usr/bin/mythexport_addjob
    - Added lockfile, to keep mythtv-backend informed of the job status (LP #596251)
  * usr/share/mythtv/mythexport.sql
    - Changed param column to longtext (LP #647908)
  * usr/share/mythtv/mythexport/otg.cgi
    - Fixed issue where old configuration file was still in use (LP #652322)
  * usr/share/mythtv/mythexport/otg_job.cgi
    - Renamed variable, no longer using "block" from old configuration
  * usr/share/mythtv/mythexport/setup.cgi
    - Fixed issue where form was submitting to the wrong page (LP #652322)
    - Changed configuration input to radio buttons
  * usr/share/mythtv/mythexport/save_system_setup.cgi
    - Fixed typo
  * usr/share/mythtv/mythexport/otg.cgi
    - Fixed issue where old configuration file was still in use
  * usr/share/mythtv/mythexport/setupsave.cgi
    - Removed debugging line
  * debial/rules:
    - Synced BZR_REVISION

  [ Angel Abad ]
  * debian/rules: Fix typo in TARFILE rule (LP: #654088)

 -- John Baab <email address hidden> Sun, 03 Oct 2010 22:46:48 -0400

Changed in mythexport (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.