Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kodi, pvr.iptvarchive: Add support for IPTV Archive features #138

Merged
merged 1 commit into from Nov 6, 2018

Conversation

arthur-liberman
Copy link
Member

@arthur-liberman arthur-liberman commented Oct 26, 2018

This PR adds a modified pvr.iptvsimple add-on and the accompanying modifications to Kodi to support it.
It adds support for the Archive feature some IPTV services provide to their users.
The add-on adds the following improvements over the standard pvr.iptvsimple add-on:

  • Support for playing a show from the EPG, using the archive.
  • Practically unlimited timeshift support.
  • Lets the user select a show from the EPG and timeshift to it.

Copy link
Contributor

@Ray-future Ray-future left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@vpeter4
Copy link
Contributor

vpeter4 commented Oct 26, 2018

Fine by me.

I really hope one day we see this functionality in Kodi/pvr.iptvsimple itself.

@MartijnKaijser
Copy link

Sure being able to record and store any IPTV streams available doesn't ring any alarm bells. Not to start over patching Kodi itself as well.

@arthur-liberman
Copy link
Member Author

arthur-liberman commented Oct 26, 2018

@MartijnKaijser
First of all this change does NOT record, nor store any streams. All it lets you do is start playback from the specified timestamp. Using the same mechanism, it also allows the user to "seek" anywhere they please in the timeline.
I am aware that a custom Demux component was the better way to go, but Kodi has pretty much everything needed to get it working, but the support for it is just not there yet.
This feature is specific to CoreELEC, and I don't have any plans to upstream it as it is right now, of course.

Edit: Here's an idea though: The patches to Kodi are there to enable timeshifting. Get rid of that, and keep the EPG tag support in iptvsimple, kodi-pvr/pvr.iptvsimple#123 will be resolved by it.
If you'd be willing to consider such an addition, I can make a PR for it.

@Ray-future
Copy link
Contributor

@MartijnKaijser Team Kodi proved several times that it'd like to play Source code Police with comments like "Raping my code" etc. This is still open source and we will not be intimidated to NOT patch Kodi. You guys can relicense closed source that will solve your problem. There is no reason to play bullie just because YOU can.

As Arthur explained there is no reason so go and bother another Kodi fork like OSMC .

@vpeter4
Copy link
Contributor

vpeter4 commented Oct 26, 2018

This small kodi patch brings BIG and much, much, much better Kodi user experience for some users. Including me. And I like it very, very much! Timeshifting is something "must include". Of course even play something from EPG in pvr.iptvsimple is something. But better have 2 functionality :)

Wish Kodi would include something like that but then I wake up in the middle of the nigh soaked in sweat :)

@MartijnKaijser
Copy link

If this feature makes sense to have in the true upstream code then it should be PRed to that. If CoreELEC decides to go diverge too much than yes they will have to completely rebrand. Just like OSMC already did so that's the reason we will not bother them. @Raybuntu So you argument is invalid.

@arthur-liberman Just to be clear we are open to any features that make sense and fit our design, and some one is willing to do the work for it.

@spycat88
Copy link
Contributor

@MartijnKaijser CoreELEC has no desire to diverge and continues to align itself with LibreELEC as closely as possible, we (the team) and myself included have encouraged Arthur to send this PR and others upstream for the benefit of all.

@Ray-future
Copy link
Contributor

@MartijnKaijser We already have a bunch of non-upstreamed and non-upstreamable Patches/Hacks so does LibreELEC, Debian and other forks. So in a sense this is already diverged a lot.
But what is your problem anyway? Comments like this and other comments just make upstreaming pointless.
OSMC is not a real rebrand according to your trademark policy. Binary is still called kodi.bin for instance and log is called kodi.log.
If that's acceptable it's an easy fix for us when we fork a skin that doesn't say kodi or has any logo's.

I don't see a point in rebranding because
1.) We don't use any Kodi resources (forum, bugtracker, personal Team Kodi support) like others do.
2.) We don't add piracy stuff like others do and we don't break Kodi. Even if when we use hacks we use them to improve Kodi so there is no reason for Team Kodi to claim that we harm the brand.
3.) We only have approx. 2000 users, we don't make any money like OSMC does with sales or LibreELEC with Donations. So using the name Kodi does no benefit to us and falls into Fair use in our case.

But I will no longer discuss this in this PR here, if Team Kodi has a Problem or wants to shut us down they can write us an email. I'm just tired of being constantly harassed by Team Kodi Members.

@arthur-liberman arthur-liberman force-pushed the pvr-archive-support branch 2 times, most recently from 4adfddc to e6ace7a Compare November 5, 2018 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants