SourceForge.net
2010-11-15 20:07:11 UTC
Bugs item #2919995, was opened at 2009-12-23 12:58
Message generated for change (Comment added) made by dekarl
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=2919995&group_id=39046
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tv_grab_za
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: kl1k kl1k (kl1k)
Assigned to: igitur (igitur)
Summary: tv_grab_za errors
Initial Comment:
when running the latest nightly build I get the follow error:
Initialising cookies
getting listings
cannot look up 'Crime & Investigation' in map at /usr/bin/tv_grab_za line 395.
This occurs on various channels. The problem is due to DSTV calling the channels different names.
Editing tv_grab_za and inserting the below and recompiling solves the problem:
my %dstvchannelfixups = (
'CNN International' => 'CNN',
'SABC 1' => 'SABC1',
'SABC 2' => 'SABC2',
'SABC 3' => 'SABC3',
'MagicWorld' => 'Magic World',
'Crime & Investigation Network' => 'Crime & Investigation',
'E! Entertainment Television' => 'E',
'Deutsche Welle' => 'Deutchse Welle',
'SuperSport MaXimo 1' => 'SuperSport Maximo',
'SONY' => 'Sony Entertainment');
With the above error can the code be changed so that it skips the incorrectly named instead if dying with a error? (eg Soweto TV is in the channel list but not on the DSTV site so the application dies when it tries to pull data for that channel. I know I could comment it out but it would be better if the error was handled.
----------------------------------------------------------------------
Message:
Is this fix still important after application of the last two patches? I'd
prefer to remove the fixups completely as the MNET fallback got disabled
anyway, so there's no reason to convert back and forth between channel ids
and channel names...
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2010-04-19 14:02
Message:
Thanks, knowledgejunkie. I'll try to implement this as soon as possible.
----------------------------------------------------------------------
Comment By: kl1k kl1k (kl1k)
Date: 2010-04-18 20:05
Message:
Good idea knowledgejunkie. As now the Hallmark channel has also been
renamed as well as a few others.
----------------------------------------------------------------------
Comment By: Nick Morrott (knowledgejunkie)
Date: 2010-04-12 19:45
Message:
It might be an idea to add these mappings to a file that can be loaded at
run-time via XMLTV::Supplement. This way, a new release of the grabber does
not have to be made when channel names change.
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2010-03-10 09:32
Message:
kl1k, have you been able to test the fixes?
----------------------------------------------------------------------
Comment By: kl1k kl1k (kl1k)
Date: 2009-12-26 09:27
Message:
Great - thanks for the feedback.
Feel free to contact me - I'd love to be a tester.
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2009-12-24 17:44
Message:
Thanks. I've fixed tv_grab_za a few months ago but was too lazy to commit
the changes. Apart from your suggestions there are some other fixes too.
I'm on holiday at the moment but will commit in once i'm back.
Promise↲↲You're the only other Saffa i know of that uses it so I'd like
to work with you in testing future enhancements. I'll contact you
directly.↲↲Regards,↲Francois
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=2919995&group_id=39046
Message generated for change (Comment added) made by dekarl
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=2919995&group_id=39046
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tv_grab_za
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: kl1k kl1k (kl1k)
Assigned to: igitur (igitur)
Summary: tv_grab_za errors
Initial Comment:
when running the latest nightly build I get the follow error:
Initialising cookies
getting listings
cannot look up 'Crime & Investigation' in map at /usr/bin/tv_grab_za line 395.
This occurs on various channels. The problem is due to DSTV calling the channels different names.
Editing tv_grab_za and inserting the below and recompiling solves the problem:
my %dstvchannelfixups = (
'CNN International' => 'CNN',
'SABC 1' => 'SABC1',
'SABC 2' => 'SABC2',
'SABC 3' => 'SABC3',
'MagicWorld' => 'Magic World',
'Crime & Investigation Network' => 'Crime & Investigation',
'E! Entertainment Television' => 'E',
'Deutsche Welle' => 'Deutchse Welle',
'SuperSport MaXimo 1' => 'SuperSport Maximo',
'SONY' => 'Sony Entertainment');
With the above error can the code be changed so that it skips the incorrectly named instead if dying with a error? (eg Soweto TV is in the channel list but not on the DSTV site so the application dies when it tries to pull data for that channel. I know I could comment it out but it would be better if the error was handled.
----------------------------------------------------------------------
Comment By: Karl Dietz (dekarl)
Date: 2010-11-15 21:07Message:
Is this fix still important after application of the last two patches? I'd
prefer to remove the fixups completely as the MNET fallback got disabled
anyway, so there's no reason to convert back and forth between channel ids
and channel names...
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2010-04-19 14:02
Message:
Thanks, knowledgejunkie. I'll try to implement this as soon as possible.
----------------------------------------------------------------------
Comment By: kl1k kl1k (kl1k)
Date: 2010-04-18 20:05
Message:
Good idea knowledgejunkie. As now the Hallmark channel has also been
renamed as well as a few others.
----------------------------------------------------------------------
Comment By: Nick Morrott (knowledgejunkie)
Date: 2010-04-12 19:45
Message:
It might be an idea to add these mappings to a file that can be loaded at
run-time via XMLTV::Supplement. This way, a new release of the grabber does
not have to be made when channel names change.
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2010-03-10 09:32
Message:
kl1k, have you been able to test the fixes?
----------------------------------------------------------------------
Comment By: kl1k kl1k (kl1k)
Date: 2009-12-26 09:27
Message:
Great - thanks for the feedback.
Feel free to contact me - I'd love to be a tester.
----------------------------------------------------------------------
Comment By: igitur (igitur)
Date: 2009-12-24 17:44
Message:
Thanks. I've fixed tv_grab_za a few months ago but was too lazy to commit
the changes. Apart from your suggestions there are some other fixes too.
I'm on holiday at the moment but will commit in once i'm back.
Promise↲↲You're the only other Saffa i know of that uses it so I'd like
to work with you in testing future enhancements. I'll contact you
directly.↲↲Regards,↲Francois
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=2919995&group_id=39046