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

Fixed missing include in IEncoders.h #1127

Merged
merged 2 commits into from
Mar 11, 2017

Conversation

francesco-romano
Copy link
Collaborator

If somebody includes only IEncoders.h than YARP_dev_API is not defined.

Thanks @claudia-lat

@francesco-romano
Copy link
Collaborator Author

@drdanz where should I put the fix description in the release notes?

@drdanz
Copy link
Member

drdanz commented Mar 10, 2017

In doc/release/v2_3_68_1.md

@francesco-romano
Copy link
Collaborator Author

@drdanz done

@drdanz drdanz self-assigned this Mar 11, 2017
@drdanz drdanz added Affects: YARP v2.3.68 This is a known issue affecting YARP v2.3.68 Component: Library - YARP_dev Resolution: Merged Target: YARP v2.3.70 Issue Type: Bug Involves some intervention from a system administrator labels Mar 11, 2017
@drdanz drdanz merged commit bc69d1b into robotology:master Mar 11, 2017
@drdanz
Copy link
Member

drdanz commented Mar 11, 2017

Merged. Thanks!

PeterBowman added a commit to roboticslab-uc3m/kinematics-dynamics that referenced this pull request May 18, 2017
@francesco-romano francesco-romano deleted the fix_iencoders branch June 20, 2017 14:27
@drdanz drdanz added PR Type: Bugfix This PR fixes some bug and removed Issue Type: Bug Involves some intervention from a system administrator labels Mar 23, 2018
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.

2 participants