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

Bug fix during catkin build #164

Merged
merged 2 commits into from
Jul 23, 2018

Conversation

gsilano
Copy link
Contributor

@gsilano gsilano commented May 31, 2018

The bug that appears during the catkin build (Ubuntu 16.04, ROS Kinetic Kame) process, as before mentioned in the #132, has been fixed in according to the commit 78071a14 of the FFmpeg repository.

gsilano added 2 commits May 31, 2018 16:08
The bug during the catkin build process has been fixed in according to the issue AutonomyLab#132 and explained in 78071a14 of the FFmpeg repository
Updated changelog and package file into the bebop_driver folder.
Copy link
Collaborator

@thomas-bamford thomas-bamford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for finding a fix to this bug, it will really help in the release to ROS Melodic.

@thomas-bamford
Copy link
Collaborator

@Voidminded or @mani-monaj, I am not authorized to merge pull requests on this repo. Would you be able to merge this one so that I can continue work on PR #168? If this PR is merged I can work towards releasing bebop_autonomy for ROS Melodic.

@robohacker robohacker merged commit b0cf0a7 into AutonomyLab:indigo-devel Jul 23, 2018
lincolnxlw added a commit to lincolnxlw/bebop_autonomy that referenced this pull request Jul 29, 2018
@SchwesterKatze
Copy link

@thomas-bamford
I am using the Ubuntu 18.04 and Melodic with 'bebop_autonomy' version 0.8.0 with the version_gte= 3_14_1 and I have an issue with the libbebop_driver_nodelet.so.

@WiPhi7600
Copy link

@SchwesterKatze I am as well any luck fixing it?

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.

5 participants