add-ons / service.iptv.manager

Integrate TV and radio channels with EPG data from other add-ons in Kodi PVR.
https://github.com/add-ons/service.iptv.manager/wiki
GNU General Public License v3.0
98 stars 28 forks source link

Kodi repository submission? #47

Closed Lunatixz closed 3 years ago

Lunatixz commented 3 years ago

@dagwieers any plans to submit this to the Kodi repository?

dagwieers commented 3 years ago

@michaelarnauts Are we ready for this? I am ok to release.

dagwieers commented 3 years ago

Let me already update the GitHub CI release workflow to make releasing super-easy.

Lunatixz commented 3 years ago

I recommend adding a setting to allow manual m3u/xmltv path setup, only noticed the "automatic" option.

Also, https://github.com/add-ons/service.iptv.manager/blob/master/addon.xml#L4 needs to be set up for matrix, 3.0.0

Looking forward to adding support for this manager!

dagwieers commented 3 years ago

@michaelarnauts proposed IPTV Manager to the official Kodi repository here: https://github.com/xbmc/repo-scripts/pull/1710

michaelarnauts commented 3 years ago

I recommend adding a setting to allow manual m3u/xmltv path setup, only noticed the "automatic" option.

Also, https://github.com/add-ons/service.iptv.manager/blob/master/addon.xml#L4 needs to be set up for matrix, 3.0.0

Looking forward to adding support for this manager!

You can still manually configure Simple IPTV, the item in the settings is just there for convenience. It's not easy to find the exact path to use, and the settings to configure. This could be improved by using something like a http://127.0.0.1:xxxx/epg url, but it's easier to just write the file to the filesystem instead of having Simple IPTV fetch it trough HTTP. Also, what port should we use, what if the port is already in use, ...

We probably can change some things for Matrix, so I guess the best way forward is to maintain two branches unfortunatly.

dagwieers commented 3 years ago

We probably can change some things for Matrix, so I guess the best way forward is to maintain two branches unfortunately.

As discussed, I think you can still avoid having 2 branches by making some of the Leia-specific stuff conditional. Just like we already do for Python 2/3 compatibility.

skank01 commented 3 years ago

OH, this wont take long anymore before released in official kodi repository?

piejanssens commented 3 years ago

I just installed it on Leia through the repo 🎉

dagwieers commented 3 years ago

Yes, it is in Leia already. We will have to spend some more time before we can make it available for Matrix, as the code could look a bit different because of difference in IPTV Simple.

We also are not sure if we want to manage 2 branches (one for each) or do something else instead.