[mythtv-commits] Ticket #10071: Bugfixes to mythburn.py

MythTV noreply at mythtv.org
Thu Feb 2 20:59:28 UTC 2012


#10071: Bugfixes to mythburn.py
----------------------------------+-----------------------------
 Reporter:  t.brackertz@…         |          Owner:  paulh
     Type:  Patch - Bug Fix       |         Status:  new
 Priority:  minor                 |      Milestone:  0.25
Component:  Plugin - MythArchive  |        Version:  Unspecified
 Severity:  medium                |     Resolution:
 Keywords:                        |  Ticket locked:  0
----------------------------------+-----------------------------

Comment (by t.brackertz@…):

 That's great.

 But wait a minute because during the work on the patch I did a few more
 changes that

 - tidy up the code
 - allow better scaling on multiple CPUs
 - prevent mythburn from eating up all the available disk-IO

 But I haven't finished testing yet. I hope I will have enough time on the
 weekend.
 Maybe you can answer my following questions till then:

 - How should we handle sound format:
 Until 0.24 the default behaviour was to leave the format as it is if it's
 mp2 or ac3 or to reencode everything to ac3 if explcitely chosen by user.
 There are two competing patches now: One by J.Pilk here in the ticket
 which allows to chose to encode everything to mp2 and one in commit
 c5aca5cf4f3690c95a7d4a4a64c7ec0f6ec56dce which enforces encoding to ac3
 all the time. As the comment to the commit suggests the problem are DVDs
 with mixed sound formats. So my proposition would be to let the user chose
 the sound format but take care that there aren't different formats on the
 same DVD. Do you agree? I would integrate this stuff on weekend if you do.

 - How should I supply the path:
 Because of commit c5aca5cf4f3690c95a7d4a4a64c7ec0f6ec56dce all the diffs
 above are outdated. I plan to provide one big diff including all the
 changes to the current git. Is this appropiate or should the different
 changes get split? Maybe I can also add a backport.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10071#comment:5>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center


More information about the mythtv-commits mailing list