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

Removecrystalhd #5105

Merged
merged 2 commits into from
Aug 9, 2014
Merged

Removecrystalhd #5105

merged 2 commits into from
Aug 9, 2014

Conversation

fritsch
Copy link
Member

@fritsch fritsch commented Jul 27, 2014

This was discussed in the internal forum for years now and was timelined for helix. CrystalHD is not supported anymore and no dev is testing / running it.

As discussed and acked by @davilla the original author - I tried to pull out everything crystalhd related.

If and only if really someone wants to get back crystalhd support, we should use the ffmpeg hw accel for this, which is not on my list anywhere.

@fritsch
Copy link
Member Author

fritsch commented Jul 27, 2014

jenkins build this please

@fritsch
Copy link
Member Author

fritsch commented Jul 27, 2014

I will squash down once i get okay from the team and give those people that use it time to volunteer to maintain crystalhd upstream.

@MartijnKaijser MartijnKaijser added this to the Pending for inclusion milestone Jul 27, 2014
@MartijnKaijser
Copy link
Member

@davilla
care to review if anything missed? thx

@fritsch
Copy link
Member Author

fritsch commented Jul 27, 2014

Don't merge this please - just keep it open.

@samnazarko tweeted me to become maintainer of crystalhd and migrating it to an ffmpeg hwaccel and seeing to get it optimized by adjusting it to the render / buffering changes we got through the years.

So let's wait - Helix won't be released tomorrow.

@t-nelson
Copy link
Contributor

Just rip it out of xbmc now. Its current state is broken. If a new
maintainer shows up they can PR it back in. There's no point in keeping
broken, unmaintained code around at the promise of someone new to pick it
up.

@fritsch
Copy link
Member Author

fritsch commented Jul 28, 2014

Okay. I hand over this PR to whoever wants it. I squashed the changes. Presquash code for easier review is here: https://github.com/fritsch/xbmc/commits/crystalremtmp

@t-nelson
Copy link
Contributor

@davilla any other special cases that might be hiding?

I hear there's some debate about @fritsch's involvement with openelec and this PR. My previous comment is my own and formed without any outside discussion. I have no involvement with openelec. I've never even downloaded it. Broken, abandoned code simply has no place in an actively developed project.

@davilla
Copy link
Contributor

davilla commented Jul 28, 2014

rip it.

@MartijnKaijser MartijnKaijser modified the milestones: Pending for inclusion, Helix 14.0-alpha2 Jul 28, 2014
@t-nelson
Copy link
Contributor

@FernetMenta your honors?

@FernetMenta
Copy link
Contributor

Would be my pleasure to press the button here but I am on vacation this merge window. Not sure if get a chance to get online during this time.

@MartijnKaijser
Copy link
Member

jenkins build this please

MartijnKaijser added a commit that referenced this pull request Aug 9, 2014
@MartijnKaijser MartijnKaijser merged commit 2ca4c9a into xbmc:master Aug 9, 2014
@MajorMeat
Copy link

So many people have an ATV 1, would be nice to continue the support for a working CrystalHD. No one is willing to rescue this for all of us ATV 1 users...

@da-anda
Copy link
Member

da-anda commented Sep 4, 2014

what's the issue - just keep using your current XBMC version - done. There is always a point when old software/hardware becomes obsolete - just like we don't support WindowsXP, Android 2.x and older Linux distris anymore. And don't blame us for not wanting to support unmaintained code - blame upstream.

@xbmc xbmc locked and limited conversation to collaborators Sep 4, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Cleanup non-breaking change which removes non-working or unmaintained functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants