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

Support for Mac OS X #51

Closed
wants to merge 3 commits into from
Closed

Support for Mac OS X #51

wants to merge 3 commits into from

Conversation

D-Alex
Copy link

@D-Alex D-Alex commented Feb 5, 2017

No description provided.

Alex added 3 commits January 28, 2017 23:35
This is bad practice because this will not work on systems which
does not support custom baudrates
@LeonJung
Copy link
Contributor

LeonJung commented Feb 6, 2017

Guten Tag, Alex

What a great job you've made :)
Thank you for all your contributions of your own free will.
I'll check this and take into account for the next update.
Though, your request will be pulled by passing some of our procedures... since we are now under developing the official MacOS version DynamixelSDK and I caught some differences of the coding rules or regulations, files, constructions.
But I really don't want to make your contribution valueless, so
in the circumstance, we may request you to modify your source which is hoped to be pulled.

@LeonJung LeonJung mentioned this pull request Feb 9, 2017
7 tasks
@LeonJung LeonJung closed this Aug 21, 2017
@LeonJung
Copy link
Contributor

Thank you for your advices
and now the DynamixelSDK from ver. 3.5.1 will support MacOS

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.

2 participants