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

revert ISO support #53

Merged
merged 1 commit into from Apr 1, 2021
Merged

revert ISO support #53

merged 1 commit into from Apr 1, 2021

Conversation

DaVukovic
Copy link
Member

Kodi itself has ISO support. So we should probably revert ISO support for this add-on as ppl are complaining that their DVD and BR ISO files aren't played anymore or do behave pretty strange. For example:

xbmc/xbmc#18897

This is a first wild shot to fix that. But as long as there's no other fix (which I'm not able to do because of lacking development skills), this add-on should not conflict with Kodi core functionalities.

@fuzzard
Copy link
Contributor

fuzzard commented Mar 6, 2021

Probably want to ping @zach-morris

As he introduced that change in 1021869 maybe he can give some context into the use case it was intended for

@wsnipex
Copy link
Member

wsnipex commented Mar 6, 2021

This also needs a version bump and tag in order to be deployed to mirrors.

@DaVukovic
Copy link
Member Author

Version bump is done. Can't do anything for the tags as I don't have the rights for it (and I don't necessarily need them ;) )

@zach-morris
Copy link
Contributor

zach-morris commented Mar 6, 2021

I did not test that, but it seems like a good idea to revert. It would only minorly inconvenience users of retroplayer. Including these two formats supported opening the archive/container and launching the correct files for retroplayer.

Eventually, playing ISO games and playing DVD/BRs can hopefully be sorted out.

The context of this change, and a different one previously suggested for rar support is for retroplayer. The issue is that some formats (rar, iso being good examples) are historically containerizing videos to play, but with the introduction of retroplayer, many games are containerized in the same formats, so there's a conflict. Kodi can't discern between the two currently i guess. If/when retroplayer becomes more utilized maybe a 'play using' option is the best workaround, or perhaps Kodi can detect what type of media is to be played / what addon to use to open the container in another fashion

@DaVukovic
Copy link
Member Author

I see more and more ppl who have issue while that addon being installed. Is there something which stops this PR to be merged?

@fuzzard
Copy link
Contributor

fuzzard commented Apr 1, 2021

Needs someone with commit rights, going to have to find someone on slack to get it in

@jjd-uk
Copy link
Member

jjd-uk commented Apr 1, 2021

Agreed this really needs to be merged.

@AlwinEsch @notspiff

@jjd-uk
Copy link
Member

jjd-uk commented Apr 1, 2021

Now this has been merged I assume a PR needs to done to https://github.com/xbmc/repo-binary-addons to push it to the repo? or does it just need a new tag ???

@notspiff
Copy link
Collaborator

notspiff commented Apr 1, 2021

Nothing else is required. Just the tag.

@DaVukovic DaVukovic deleted the revert_iso branch April 3, 2021 07:14
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.

None yet

7 participants