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

Add dummy implementations to satisfy new IOBluetoothDeviceInquiryDelegate protocol in 10.8 #3

Merged
merged 1 commit into from
Oct 23, 2014

Conversation

hans
Copy link
Contributor

@hans hans commented Apr 11, 2013

The author of #2 and I have both found that the latest versions of Mac OS X (10.8.2 here, and 10.8.3 for him) have new required methods in the IOBluetoothDeviceInquiryDelegate protocol. I've added some no-op methods in order to satisfy the protocol and to allow LightBlue to continue.

Thanks for posting this updated version! It's been a big help.

bartoszkopinski pushed a commit to bartoszkopinski/lightblue-0.4 that referenced this pull request Jan 13, 2014
bartoszkopinski pushed a commit to bartoszkopinski/lightblue-0.4 that referenced this pull request Jan 13, 2014
…lementations.

This patch is from: <pipt@601beb5>

I tested that it works with 10.8.2 and our current libpebble readme already states use with 10.8.3+ is required.

Without this patch 10.8.2 does not work either. This suggests that all versions of 10.8 might require the patch and given we only explicitly support 10.8 then I think we should just pull the patch into our master. (This would also mean we don't need to redirect people to <https://github.com/pipt/lightblue-0.4/>.)

Related issues/patches:

 * <0-1-0#2>

 * <0-1-0#3>

 * <0-1-0#4>
@harlanhaskins
Copy link

Please merge this.

0-1-0 added a commit that referenced this pull request Oct 23, 2014
Add dummy implementations to satisfy new IOBluetoothDeviceInquiryDelegate protocol in 10.8
@0-1-0 0-1-0 merged commit 35580e3 into 0-1-0:master Oct 23, 2014
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.

3 participants