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

Remove Support for OSX 10.6 #3507

Closed
3 tasks done
bakercp opened this issue Dec 29, 2014 · 7 comments
Closed
3 tasks done

Remove Support for OSX 10.6 #3507

bakercp opened this issue Dec 29, 2014 · 7 comments

Comments

@bakercp
Copy link
Member

bakercp commented Dec 29, 2014

  • ofConstants (Update)
  • Legacy QuickTimePlayer (do we still need this for windows?)
  • Makefiles
@bakercp bakercp changed the title Remove Support for OSX 10.7 Remove Support for OSX 10.6-10.7 Dec 29, 2014
@bilderbuchi bilderbuchi added this to the 0.9.0 milestone Dec 29, 2014
@pizthewiz pizthewiz self-assigned this Jan 4, 2015
@pizthewiz
Copy link
Member

I have these changes in a local branch and if we can get #3502 merged, should be able to test and PR shortly.

@pizthewiz
Copy link
Member

ofQuickTimePlayer appears to be used by Windows when without MinGW, see ofConstants.h#L308-323:

    #elif defined(TARGET_WIN32) && !defined(__MINGW32__)
        #define OF_VIDEO_PLAYER_DIRECTSHOW
    // (...)
    #else
        #define OF_VIDEO_PLAYER_QUICKTIME

ofQuickTimeGrabber.h would now be dead via preprocessor ofConstants.h#L269-282:

    #elif defined (TARGET_WIN32)

        // comment out this following line, if you'd like to use the
        // quicktime capture interface on windows
        // if not, we default to videoInput library for
        // direct show capture...

        #define OF_SWITCH_TO_DSHOW_FOR_WIN_VIDCAP

        #ifdef OF_SWITCH_TO_DSHOW_FOR_WIN_VIDCAP
            #define OF_VIDEO_CAPTURE_DIRECTSHOW
        #else
            #define OF_VIDEO_CAPTURE_QUICKTIME
        #endif

@pizthewiz
Copy link
Member

This is in limbo given #3543

@kylemcdonald
Copy link
Contributor

changing this issue to "remove 10.6 support" which will happen when @tgrerer's PR #3627 is pulled.

0.9.0 will not support 10.6 due to 64-bit issues.
0.9.0 will support 10.7 with ofQTKitPlayer but it will be deprecated.
0.9.0 on 10.8+ will use AVFoundation.

@kylemcdonald kylemcdonald changed the title Remove Support for OSX 10.6-10.7 Remove Support for OSX 10.6 Feb 8, 2015
@pizthewiz
Copy link
Member

@tgfrerer now that #3627 has been integrated is OS X 10.7 is our new minimum and this can be closed yeah?

@tgfrerer
Copy link
Member

Yes

@pizthewiz
Copy link
Member

Closing, OS X minimum is now 10.7 👍

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

Successfully merging a pull request may close this issue.

5 participants