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

Package Catkinized #79

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@boris-il-forte

boris-il-forte commented Nov 15, 2013

I've opened a new pull request due to the change of brach...

boris-il-forte added some commits Aug 14, 2013

Migration to Catkin build system
- deleted manifest.xml
- updated CMakeLists.txt
- created package.xml

This patch shuold be tested. It's also neeed to change the e-mail in package.xml, it was choosen a random e-mail to make the package compile with catkin
Added install rules
- added rules to install ardrone_driver binary in the correct location specified by --install-space of catkin_make command
@mani-monaj

This comment has been minimized.

Show comment
Hide comment
@mani-monaj

mani-monaj Nov 22, 2013

Member

Thank you very much @boris-il-forte . I merged your changes into catkin branch. I am working on streamlining ARDroneLib build with catkin. Documentation needs to be updated too.

Member

mani-monaj commented Nov 22, 2013

Thank you very much @boris-il-forte . I merged your changes into catkin branch. I am working on streamlining ARDroneLib build with catkin. Documentation needs to be updated too.

@boris-il-forte

This comment has been minimized.

Show comment
Hide comment
@boris-il-forte

boris-il-forte Nov 22, 2013

ook, if you need help in doing so, let me know.
thank you very much for the official catkinized branch ;)
I can now close safely the old repository?
however you should close the pull request...

boris-il-forte commented Nov 22, 2013

ook, if you need help in doing so, let me know.
thank you very much for the official catkinized branch ;)
I can now close safely the old repository?
however you should close the pull request...

@mani-monaj

This comment has been minimized.

Show comment
Hide comment
@mani-monaj

mani-monaj Nov 22, 2013

Member

👍 Thank you for the offer. Sure.

Please keep an eye on catkin branch :)

Member

mani-monaj commented Nov 22, 2013

👍 Thank you for the offer. Sure.

Please keep an eye on catkin branch :)

@mani-monaj mani-monaj closed this Nov 22, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment