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.

Sign up for GitHub

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

Jump to bottom

SystemPaths::GetOgrePaths(), SystemPaths::GetGazeboPaths(), SystemPaths::GetPluginPaths(), should not call Update*Paths() #15

Closed
osrf-migration opened this issue Aug 27, 2012 · 4 comments
Closed

SystemPaths::GetOgrePaths(), SystemPaths::GetGazeboPaths(), SystemPaths::GetPluginPaths(), should not call Update*Paths() #15

osrf-migration opened this issue Aug 27, 2012 · 4 comments
Labels
all bug Something isn't working major

Comments

@osrf-migration
Copy link

Original report (archived issue) by John Hsu (Bitbucket: hsu, GitHub: hsu).

SystemPaths::GetOgrePaths(), SystemPaths::GetGazeboPaths(), SystemPaths::GetPluginPaths() calls corresponding Update*Paths() before returning the list of stored paths, this will always use the paths set by environmental variables even if the user use SystemPaths::Set*Paths(...) call to specify custom paths. Instead SystemPaths::Update*Paths() should be called only at start-up.
@osrf-migration
Copy link
Author

Original comment by John Hsu (Bitbucket: hsu, GitHub: hsu).

[[https://osrf-migration.github.io/gazebo-gh-pages/#!/osrf/gazebo/diff/gazebo/common/SystemPaths.cc?diff2=01173c5767683d4c9085f91ac78fac0333a86357&diff1=74cfc49f94fad18c73cf3cd22329b43a1f364743|patched]]

@osrf-migration
Copy link
Author

Original comment by John Hsu (Bitbucket: hsu, GitHub: hsu).

  • changed state from "new" to "resolved"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

  • set version to "all"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

  • changed state from "resolved" to "closed"

@osrf-migration osrf-migration added major bug Something isn't working all labels Apr 19, 2020
@farzingkh farzingkh mentioned this issue Jul 24, 2020
Closed
@nlamprian nlamprian mentioned this issue Nov 16, 2021
Closed
qaz6906 pushed a commit to qaz6906/gazebo-classic_Hexa_vtol that referenced this issue Feb 10, 2023
@LorenzMeier
Merge pull request gazebosim#15 from ChristophTobler/lidar_opt_flow
1b06d4c 
Lidar and optical flow in sitl gazebo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
all bug Something isn't working major
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@osrf-migration