Add connection manager support to wait for a valid M3U file before starting the add-on instance

[phunkyfish]

v21.7.0

• Add connection manager support to wait for a valid M3U file before starting the add-on instance
• The minimum refresh interval for the M3U should be 1 minutes and not zero, as that would be infinite refresh

[phunkyfish]

New testbuilds: https://jenkins.kodi.tv/blue/organizations/jenkins/kodi-pvr%2Fpvr.iptvsimple/detail/PR-823/3/artifacts

Question, what behaviour do you get/expect if there is no M3U URL set?

[phunkyfish]

Question, what behaviour do you get/expect if there is no M3U URL set?

Any response here? Actually NVM. I'll just have it wait for URL to be set before validating it.

New testbuilds: https://jenkins.kodi.tv/blue/organizations/jenkins/kodi-pvr%2Fpvr.iptvsimple/detail/PR-823/5/artifacts

[matthuisman]

hmm. something seems off now. after booting - i get empty channels and epg.
I see it being loaded in the log though.
If i go Clear PVR data then it all appears

It doesnt show as pvr manager starting up either like it used too.

I went back to https://jenkins.kodi.tv/blue/organizations/jenkins/kodi-pvr%2Fpvr.iptvsimple/detail/PR-823/1/artifacts and that works

https://jenkins.kodi.tv/blue/organizations/jenkins/kodi-pvr%2Fpvr.iptvsimple/detail/PR-823/2 has no windows x64
but /3 is were it "breaks"

[phunkyfish]

Hmmm, I won’t have history so hard to figure out the change that breaks it.

Can you explain your set up to me?

[phunkyfish]

I bet it's because we removed the notification. My guess is it also updates Kodi PVR of the state change.

Yup, that was it. New builds: https://jenkins.kodi.tv/blue/organizations/jenkins/kodi-pvr%2Fpvr.iptvsimple/detail/PR-823/8/artifacts

[matthuisman]

thank you. have tested and works good now :)

[matthuisman]

@phunkyfish I forgot to test local paths!

Just tried them now and they dont seem to work.

Localpath: special://userdata/addon_data/plugin.program.iptv.merge/playlist.m3u8

Just constantly get logged:
AddOnLog: pvr.iptvsimple: pvr.iptvsimple - Check Unable to open url: special://userdata/addon_data/plugin.program.iptv.merge/playlist.m3u8

I still think be nice if it retried for local paths as well, but maybe the check method needs to handle local paths?

[phunkyfish]

I implemented it using local paths, but just the regular filesystem and not special paths. They should just work too though.

Will take a look later.

[phunkyfish]

Please test the fix here: #826