[mythtv-users] MythArchive in 0.27: title of iso image is bad

[Paul Harrison]

On 11/09/13 18:40, John Pilkington wrote:
> On 11/09/13 18:05, Paul Harrison wrote:
>> On 11/09/13 12:06, John Pilkington wrote:
>>> I've just been trying out MythArchive in Richard Shaw's latest build,
>>> which includes a batch of very recent patches by PaulH.  Many thanks
>>> to both.  The prettyxml is a great improvement for diagnostics, and I
>>> see an animated thumbnail fix has just gone in.
>>>
>>> A new bug has crept in with the iso title.  I don't really make use of
>>> that, but it's now appearing (when played with vlc) as a line of
>>> underscores. Here's the command being used, from the progress log of
>>> my (patched) version of mythburn.py.  The unpatched script has the
>>> same problem.
>>>
>>> 00:00:15 Command is ionice -c3 "mkisofs" -dvd-video  -V "
>>> _______" -o "/mnt/f10store/myth/arctmp/isos/mythburn.iso"
>>> "/home/John/arctmp/work/dvd"
>>>
>>> I thought that this might be a product of the shift to  utf-8 and this
>>> snippet:
>>>
>>>             # replace all non-ascii-characters
>>>             title.encode('ascii', 'replace').decode('ascii', 'replace')
>>>             title.strip()
>>>
>>> but I commented out the 'title.encode' line and still see the same.  I
>>> can play around a bit more but hints would be welcome.
>>>
>>> John P
>>
>> Everything looks OK here. What was the title of the show?
>>
>> There is some code to replace non alphanumeric characters with '_' maybe
>> that is what you are seeing but that would have to be one strange
>> programme title. You could stick a few 'print title' lines around where
>> the title is processed to see how it ends up being what it is.
>>
>> Paul H.
>        write("Title read from info.xml : %s : End"  %  title)
>
> 18:28:41 Title read from info.xml :
>         Knocked Up
>      : End
>
>        write("Title passed for DVDISO : %s : End"  %  title)
>
> 18:28:41 Title passed for DVDISO :
> _______ : End
>


Yeah Jonatan on irc pointed out the obvious problem title.strip() should 
be title = title.strip() same with the line above it too.

I'll fix it shortly once I've found out how long the title can be. The 
original code set it to 32 chars but the patch from Stefan that changed 
the processing of the title changed it to just 7 chars which seems a 
little short.

Paul H.