Skip to content

Added info to README#133

Closed
Asmodasis wants to merge 1 commit into
kodi-pvr:Piersfrom
Asmodasis:Piers
Closed

Added info to README#133
Asmodasis wants to merge 1 commit into
kodi-pvr:Piersfrom
Asmodasis:Piers

Conversation

@Asmodasis
Copy link
Copy Markdown

Updated README to reflect building procedure for Kodi version Piers.

Updated README to reflect building procedure for Kodi version Piers.
Comment thread README.md
When building the add-on you have to use the correct branch depending on which version of Kodi you're building against.
If you want to build the add-on to be compatible with the latest kodi `master` commit, you need to checkout the branch with the current kodi codename.
Also make sure you follow this README from the branch in question.
The addon `inputstream.adaptive` must be built at the same time. Also make sure you follow this README from the branch in question.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The inputstream.adaptive add-on can be build at any time, this is completely unrelated to building pvr.plutotv, as it is a run time dependency, not a build time dependency, thus, not related to pvr.plutotv build procedure.

We need to distinguish between build requirements, which are to be described here and runtime-requirements, which should not be part of this README.

Please remove all inputstream.adaptive information you have added to this file.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have considered this suggestion, I agree with it. Without it, however, I believe the changes to the README are insignificant. I will be closing the pull request. Thank you.

@Asmodasis Asmodasis closed this Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants