Closed brainstorm closed 4 years ago
Works on XCode 11.4.1 locally, although might need some tweaks on platformcontext.mm
to avoid future deprecations:
$ make -j4
[ 36%] Building CXX object CMakeFiles/openpnp-capture.dir/common/libmain.cpp.o
[ 36%] Building CXX object CMakeFiles/openpnp-capture.dir/common/logging.cpp.o
[ 36%] Building CXX object CMakeFiles/openpnp-capture.dir/common/context.cpp.o
[ 36%] Building CXX object CMakeFiles/openpnp-capture.dir/common/stream.cpp.o
[ 45%] Building CXX object CMakeFiles/openpnp-capture.dir/mac/platformcontext.mm.o
[ 54%] Building CXX object CMakeFiles/openpnp-capture.dir/mac/platformstream.mm.o
[ 63%] Building CXX object CMakeFiles/openpnp-capture.dir/mac/uvcctrl.mm.o
/Users/romanvg/dev/cchs/openpnp-capture/mac/platformcontext.mm:74:54: warning: 'devicesWithMediaType:' is deprecated: first deprecated in macOS 10.15 - Use AVCaptureDeviceDiscoverySession instead. [-Wdeprecated-declarations]
for (AVCaptureDevice* device in [AVCaptureDevice devicesWithMediaType:AVMediaTypeVideo])
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/System/Library/Frameworks/AVFoundation.framework/Headers/AVCaptureDevice.h:102:1: note: 'devicesWithMediaType:' has been explicitly marked deprecated here
+ (NSArray<AVCaptureDevice *> *)devicesWithMediaType:(AVMediaType)mediaType API_DEPRECATED("Use AVCaptureDeviceDiscoverySession instead.", ios(4.0, 10.0), macos(10.7, 10.15));
^
1 warning generated.
[ 72%] Linking CXX shared library libopenpnp-capture.dylib
[ 72%] Built target openpnp-capture
[ 81%] Building CXX object mac/tests/CMakeFiles/openpnp-capture-test.dir/__/__/common/logging.cpp.o
[ 90%] Building CXX object mac/tests/CMakeFiles/openpnp-capture-test.dir/main.cpp.o
[100%] Linking CXX executable openpnp-capture-test
[100%] Built target openpnp-capture-test
Oh dear, it turns out XCode 11.4 is fine, but TravisCI still does not support Catalina?:
https://travis-ci.community/t/macos-catalina-build-environment/5608/16
https://docs.travis-ci.com/user/reference/osx/#macos-version
... sigh
Possible OS-version-dependent fix spotted here: https://github.com/opencv/opencv/pull/15517/files
Thanks @brainstorm - all sounds good. I appreciate the detailed response. Let's give it a try!
This is a followup on the failed deploy for PR https://github.com/openpnp/openpnp-capture/pull/32.
The underlying issue has been resolved by travisci support folks, as explained/tracked on twitter:
https://twitter.com/braincode/status/1253981272885477376
This pullrequest should now deploy, as demonstrated on this fork:
https://github.com/brainstorm/openpnp-capture/releases and https://travis-ci.com/github/brainstorm/openpnp-capture/builds/162954136
As long as this particular repository is migrated to travis-ci.com instead of travis-ci.org (change to be operated by @vonnieda):
https://docs.travis-ci.com/user/migrate/open-source-repository-migration