Discussion:
[Xmltv-devel] [ xmltv-Bugs-3534807 ] Fresh configuration of uk_rt fails on 0.5.62
SourceForge.net
2012-06-13 08:30:20 UTC
Permalink
Bugs item #3534807, was opened at 2012-06-13 01:30
Message generated for change (Tracker Item Submitted) made by knowledgejunkie
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3534807&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_uk_rt
Group: None
Status: Open
Resolution: None
Priority: 9
Private: No
Submitted By: Nick Morrott (knowledgejunkie)
Assigned to: Nick Morrott (knowledgejunkie)
Summary: Fresh configuration of uk_rt fails on 0.5.62

Initial Comment:
Observation:

As reported on xmltv-users, fresh configuration of uk_rt fails in 0.5.62 when requesting available lineups.


Implication:


XMLTV::Supplement works with path contents in either the directory of filename component of its call _if_ the path to the requested file already exists. The uk_rt grabber had a single direction component in the file parameter which does not exist at configuraiton time. This coupled with a testing failure which meant this directory already existed resulted in this issue not revealing itself.

Fresh installation/configuration of any grabbers using XMLTV::Supplement with deeply nested files must therefore ensure all path components are in the first parameter passed to GetSupplement (which is likely how it is intended to be used).


Recommendation:

Always call GetSupplement with:

my $string = GetSupplement("full/path/to/file", "$file");

instead of

my $string = GetSupplement("full/path/to", "dir/$file");


Developer Response:

Confirmed here. Patch committed to CVS.
Will will need to get a new dot release out the door with this in.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3534807&group_id=39046
SourceForge.net
2012-06-13 08:36:05 UTC
Permalink
Bugs item #3534807, was opened at 2012-06-13 01:30
Message generated for change (Comment added) made by knowledgejunkie
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3534807&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_uk_rt
Group: None
Status: Closed
Resolution: Fixed
Priority: 9
Private: No
Submitted By: Nick Morrott (knowledgejunkie)
Assigned to: Nick Morrott (knowledgejunkie)
Summary: Fresh configuration of uk_rt fails on 0.5.62

Initial Comment:
Observation:

As reported on xmltv-users, fresh configuration of uk_rt fails in 0.5.62 when requesting available lineups.


Implication:


XMLTV::Supplement works with path contents in either the directory of filename component of its call _if_ the path to the requested file already exists. The uk_rt grabber had a single direction component in the file parameter which does not exist at configuraiton time. This coupled with a testing failure which meant this directory already existed resulted in this issue not revealing itself.

Fresh installation/configuration of any grabbers using XMLTV::Supplement with deeply nested files must therefore ensure all path components are in the first parameter passed to GetSupplement (which is likely how it is intended to be used).


Recommendation:

Always call GetSupplement with:

my $string = GetSupplement("full/path/to/file", "$file");

instead of

my $string = GetSupplement("full/path/to", "dir/$file");


Developer Response:

Confirmed here. Patch committed to CVS.
Will will need to get a new dot release out the door with this in.

----------------------------------------------------------------------
Comment By: Nick Morrott (knowledgejunkie)
Date: 2012-06-13 01:36

Message:
Patch committed to CVS. Developer will get more sleep

We will need to get a new dot release out.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424135&aid=3534807&group_id=39046
Loading...