Gazebo ROS interface documentation error about px4-devguide

[julianoes]

@eric1221bday Thanks for the elaborate feedback. This is appreciated.

@vilhjalmur89 you also ran into this issue with the redundant gazebo copies. Can you comment?

@eric1221bday it would be great if you can file a pull request for the things which are obviously wrong and we should try to fix the confusion about the build of sitl_gazebo.

[oceanusxiv]

@julianoes so just as a warning large parts of that page is basically invalid, the "what's happening behind the scenes" section is pretty much completely obsolete. I guess what I'm trying to say that the page would probably need an overhaul, the only section that as it stand works is probably just the top section, launch mavros to interface with no gazebo plugins

this first pull request would probably just be focussed on correcting the versioning on installing gazebo, I'll leave modifying the launch procedure for gazebo wrappers up for further discussion

[oceanusxiv]

@julianoes pull request created

[vilhjalmur89]

@eric1221bday Thanks for the feedback.

You are right about having to run make posix_sitl_default gazebo.

The sitl_gazebo issue is because of the line
source Tools/setup_gazebo.bash $(pwd) build_posix_sitl_default,
which should be
source Tools/setup_gazebo.bash $(pwd) $(pwd)/build_posix_sitl_default

[julianoes]

@vilhjalmur89 can you please review the PR, thanks.

[oceanusxiv]

@vilhjalmur89 you are correct, I am able to run SITL now using the corrected version of the instructions, however I would also advise removing

source ~/catkin_ws/devel/setup.bash

since having a catkin workspace is unnecessary to running this SITL, only if the user has some external packages he or she needs to run, that would be up to them, this line just causes confusion.

[vilhjalmur89]

@eric1221bday You might need that line for the Mavros example, but it wouldn't hurt to make it explicit

[oceanusxiv]

@vilhjalmur89 actually since there are debian packages of mavros, if you didn't build mavros from source you actually wouldn't need that line at all

[vilhjalmur89]

@eric1221bday Yes, but in the guide it is recommended to build it from source.

But you're right, it is unnecessarily confusing

[oceanusxiv]

@vilhjalmur89 alright then on second pass I'll leave it in but leave a note on the fact that it's optional

[LorenzMeier]

@vilhjalmur89 Can you review and close or fix? Thanks!

[rahulkalliadan]

Hello,

I have tried working on the code you provided.
When I tried Catkin Build, only 1 of 3 packages got succeeded.

[julianoes]

@rahulkalliadan this is an older issues, please create a new issue for your problem and copy&paste the error message instead of a screenshot.