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

OSX and iOS unified AVFoundation video player. #3590

Merged
merged 1 commit into from
Feb 6, 2015

Conversation

julapy
Copy link
Member

@julapy julapy commented Jan 25, 2015

No description provided.

@ofTheo
Copy link
Member

ofTheo commented Jan 25, 2015

wow - awesome to see this @julapy
any regressions or possible issues to look out for?

@julapy
Copy link
Member Author

julapy commented Jan 25, 2015

@ofTheo, no regressions as far as im aware.
things should work all the same on OSX and iOS.
be good to give it a thorough test though...

@kylemcdonald
Copy link
Contributor

just for reference, this would help with #2283 and would close #3242. further description of this PR is in #3242 (comment)

i suggest where either:

  1. trust @julapy and merge immediately to give as much time to test before 0.9.0 is released :) (my preference)
  2. wait for 0.9.0, and include this in 0.9.1 as is currently planned (which could be around 6-8 months...)

@kylemcdonald kylemcdonald added this to the 0.9.1 milestone Feb 2, 2015
@julapy
Copy link
Member Author

julapy commented Feb 2, 2015

thats what i was thinking as well... since we are going forward with the unified video player, then its better to switch to it now so we can test it and improve it if any issues arise. it will also ensure that any future developments to the video player work on both osx and ios.

@kylemcdonald
Copy link
Contributor

@julapy great, then i'd say give this 2-3 more days, then merge if there are no objections from @openframeworks/video and we will make sure to squash any bugs before the next release :)

@bilderbuchi
Copy link
Member

tbh, I'd rather we not add more features/PRs/issues and testing debt to 0.9.0, and instead slate this for immediate merge after 0.9.0 release - that way, the testing time is maximised before this hits the "public", and we don't have to allocate yet more time to testing this.
either way, I just wanted to give my 2 cents from a release management p.o.v., and will defer to @openframeworks/video for the final decision.

@kylemcdonald
Copy link
Contributor

@bilderbuchi i generally agree, but given that the video player is already in flux and will need testing regardless i don't think this adds any testing debt -- just redirects it. and, ideally, it reduces debt by consolidating two implementations.

ofTheo added a commit that referenced this pull request Feb 6, 2015
OSX and iOS unified AVFoundation video player.
@ofTheo ofTheo merged commit ce8f258 into openframeworks:master Feb 6, 2015
@kylemcdonald kylemcdonald modified the milestones: 0.9.0, 0.9.1 Feb 6, 2015
@kylemcdonald
Copy link
Contributor

thanks @ofTheo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants