search

oe-alliance / XMLTV-Import

Import's EPG data from rytec xml data sources.
15 stars 37 forks source link

Latest PLI merge #28

Closed DougMac closed 5 years ago

DougMac commented 5 years ago

The latest PLI merge (2e43d64) marks a server as invalid if it doesn't have a 'LastUpdate.log' file. This file is not present on some servers so the merge has resulted in some servers no longer working.

Later commits on the PLI branch have added a 'nocheck' attribute so that you can bypass this check if needed. Would it be possible to merge this commit?

RobvanderDoes commented 5 years ago

Hmmm, that commit says "

But we strongly recommend not to use this attribute but ensure to have the LastUpdate.log created once all the files are properly updated.

In other words: it's an unsafe work-around for an improper source. So not sure if it's a good idea to embed that.

DougMac commented 5 years ago

It would be good to have the option.

The LastUpdate.log check has only been introduced in the recent merge and this commit just gives an option to restore previous behaviour - the default is still use the new server validation.

Huevos commented 5 years ago

This is currently being worked on.

davesayers2014 commented 5 years ago

Pull request to add here https://github.com/oe-alliance/XMLTV-Import/pull/29

DougMac commented 5 years ago

Thanks.