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

[John Pilkington]

On 11/09/13 19:43, Paul Harrison wrote:
> 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.

Good.  I'm not sufficiently into python to spot that 'obvious' problem; 
  I thought it was just normal python magic.  And it seems that info.xml 
isn't written by mythburn.py, so my intention to test that has been put 
on hold :-)