Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-10 Thread dinkypumpkin
On 09/10/2011 22:36, dinkypumpkin wrote: On 09/10/2011 08:39, Vangelis forthnet wrote: it has pid=p00kdr09. With the workaround suggested, AP (latest binary) tags the m4a file, but only PARTIALLY: The Comment and ©lyr fields (in MediaInfo) contain only the first word of the description, i.e.

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-09 Thread Vangelis forthnet
On Sat Oct 8 21:21:25 BST 2011, I wrote: A short-term workaround until I get a patch together is to comment out lines 4286-4288 in get_iplayer{.pl}, which should look like: # while ( my ($key, $val) = each %{$meta} ) { # $meta-{$key} = decode(utf8, $val); # } The

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-09 Thread Shevek
On 8 October 2011 16:33, dinkypumpkin dinkypump...@gmail.com wrote: On 08/10/2011 16:12, Shevek wrote: I'm also getting this error now (having never seen it before): INFO: Recorded D:\Users\Cyrus\Downloads\iPlayer\Stuart_Maconies_Freakier_Zone_-_08_10_2011_b015gv6n_default.m4a Wide

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-09 Thread dinkypumpkin
On 09/10/2011 08:39, Vangelis forthnet wrote: it has pid=p00kdr09. With the workaround suggested, AP (latest binary) tags the m4a file, but only PARTIALLY: The Comment and ©lyr fields (in MediaInfo) contain only the first word of the description, i.e. Were (???). Man, when it rains it pours.

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread dinkypumpkin
On 07/10/2011 21:53, Vangelis forthnet wrote: Wide character in print at get_iplayer.pl line 3530. This is only a warning message, though it's related to the overall problem (see below). Cannot decode string with wide characters at Encode.pm line 162. This is a separate error that

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread Kevin Reilly
On 08/10/2011 15:05, dinkypumpkin wrote: On 07/10/2011 22:24, Kevin Reilly wrote: https://bitbucket.org/dinkypumpkin/atomicparsley/downloads/AtomicParsley-0.9.4-hg-472d5fe6fb04.zip Thanks to both of you for testing. I'm satisfied that an update is warranted, unless anyone has an

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread Shevek
On 8 October 2011 13:18, dinkypumpkin dinkypump...@gmail.com wrote: On 07/10/2011 21:53, Vangelis forthnet wrote: Wide character in print at get_iplayer.pl line 3530. This is only a warning message, though it's related to the overall problem (see below). Cannot decode string with wide

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread David Woodhouse
On Sat, 2011-10-08 at 13:18 +0100, dinkypumpkin wrote: Long Version: The overall problem is that get_iplayer doesn't anticipate programme metadata will contain character entities that are outside the the ISO 8859-1 character set. Ick. That's kind of broken. We should be using UTF-8

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread dinkypumpkin
On 08/10/2011 16:20, David Woodhouse wrote: If you *decode* UTF-8, that implies turning it into actual letters... A different decoding in this case. What I mean here is that HTML::Entities::decode_entities() returns a decoded string with the utf8 flag set due to the presence of the expanded

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread David Woodhouse
On Sat, 2011-10-08 at 18:37 +0100, dinkypumpkin wrote: On 08/10/2011 16:20, David Woodhouse wrote: If you *decode* UTF-8, that implies turning it into actual letters... A different decoding in this case. What I mean here is that HTML::Entities::decode_entities() returns a decoded string

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-08 Thread Vangelis forthnet
On Sat Oct 8 16:33:28 BST 2011, dinkypumpkin wrote: Same problem as Vangelis - the dreaded curly quote. ... A short-term workaround until I get a patch together is to comment out lines 4286-4288 in get_iplayer{.pl}, which should look like: # while ( my ($key, $val) = each %{$meta} ) { #

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-07 Thread Vangelis forthnet
On Thu Oct 6 00:31:17 BST 2011, dinkypumpkin wrote: So that is now 2 programmes where the latest version of AP appears to fix the the crashing problem. It would be grateful if both of you could check this on your systems. I've temporarily uploaded a copy of the latest AP build here:

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-07 Thread Kevin Reilly
On 06/10/2011 00:41, Shevek wrote: https://bitbucket.org/dinkypumpkin/atomicparsley/downloads/AtomicParsley-0.9.4-hg-472d5fe6fb04.zip I re-enabled the Lyrics tagging and can confirm success with the Night Waves episode: Ditto here. 3x downloads of that same programme using the new AP, and

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread Kevin Reilly
On 02/10/2011 20:26, dinkypumpkin wrote: 2. There is reason to think that pushing the above-mentioned AtomicParsley build could be premature. There is a very recent patch - not yet merged into AtomicParsley - that may also bear on the problem get_iplayer users have experienced. See this

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread dinkypumpkin
On 05/10/2011 09:28, Kevin Reilly wrote: Sod's Law has dictated, of course, that the four programmes I've downloaded so far have caused no problem with the original version. I'll keep using this method, though, just in case I stumble upon a programme that holds the key to resolving this, or

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread Shevek
On 5 October 2011 20:32, dinkypumpkin dinkypump...@gmail.com wrote: If the patch I referenced is the root cause of AP crashes with get_iplayer, the problem it addresses is something that will only manifest occasionally and won't always be reproducible on another system, even with the same

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread Kevin Reilly
On 05/10/2011 22:09, Shevek wrote: Well, how's that for tempting fate! I've just had AP (the alternate build) crap out on me: Night_Waves_-_Midnight_in_Paris_Fiona_Shaw_Madness_St_Pauls_Cathedral_b0159w8g Confirmed. Crashes here with both the installer-provided AP and the one you referenced

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread dinkypumpkin
On 05/10/2011 22:21, Kevin Reilly wrote: On 05/10/2011 22:09, Shevek wrote: Well, how's that for tempting fate! I've just had AP (the alternate build) crap out on me: Night_Waves_-_Midnight_in_Paris_Fiona_Shaw_Madness_St_Pauls_Cathedral_b0159w8g Confirmed. Crashes here with both the

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-05 Thread Shevek
On 6 October 2011 00:31, dinkypumpkin dinkypump...@gmail.com wrote: So that is now 2 programmes where the latest version of AP appears to fix the the crashing problem.  It would be grateful if both of you could check this on your systems.  I've temporarily uploaded a copy of the latest AP

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-02 Thread dinkypumpkin
On 01/10/2011 10:41, Shevek wrote: For the adventurous among you, give it a try! https://bitbucket.org/jonhedgerows/atomicparsley/downloads/AtomicParsley-0.9.4-hg83.814077d09d34.zip Two things here: 1. To anyone having trouble with AtomicParsley in Windows - at least tell us which programmes

Re: Possible solution to the Windows AtomicParsley crashes?

2011-10-01 Thread Kevin Reilly
On 01/10/2011 10:41, Shevek wrote: [...] For the adventurous among you, give it a try! https://bitbucket.org/jonhedgerows/atomicparsley/downloads/AtomicParsley-0.9.4-hg83.814077d09d34.zip Of course, YMMV! My downloads tend to be mostly TV shows rather than radio, but I've installed this