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

ionic/install_windows_src: More bulletproof terminal setup instructions #515

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

peci1
Copy link
Contributor

@peci1 peci1 commented Sep 15, 2024

🦟 Bug fix

Summary

I've added a few tips for Windows usage based on my experience.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@peci1
Copy link
Contributor Author

peci1 commented Sep 15, 2024

@peci1
Copy link
Contributor Author

peci1 commented Sep 15, 2024

I actually don't know if it's the intended behavior that Gazebo requires to be run inside a VS shell. However, that's what was needed on my system. Without it (and with conda activated and workspace sourced), I get:

λ gz sim -s
Library error for [D:/programming/gz9-ws/gz-ws/install/bin/gz-sim9-gz.dll]: can't load D:/programming/gz9-ws/gz-ws/install/bin/gz-sim9-gz.dll

@traversaro
Copy link
Contributor

I actually don't know if it's the intended behavior that Gazebo requires to be run inside a VS shell. However, that's what was needed on my system. Without it (and with conda activated and workspace sourced), I get:

λ gz sim -s
Library error for [D:/programming/gz9-ws/gz-ws/install/bin/gz-sim9-gz.dll]: can't load D:/programming/gz9-ws/gz-ws/install/bin/gz-sim9-gz.dll

That is indeed strange. Are you using Command Prompt or Windows Powershell?

@peci1
Copy link
Contributor Author

peci1 commented Sep 15, 2024

I'm using the command prompt.

@traversaro
Copy link
Contributor

Then something is indeed strange. If you want to debug more, you could try to load D:/programming/gz9-ws/gz-ws/install/bin/gz-sim9-gz.dll in https://github.com/lucasg/Dependencies in the same environment you are trying to run, and check if some dependent .dll is missing.

Signed-off-by: Martin Pecka <peci1@seznam.cz>
@peci1
Copy link
Contributor Author

peci1 commented Sep 15, 2024

Weird. I just rebooted the computer and now it works in a normal CMD shell.

One possibility is that I'm using "conemu" as the command prompt, which normally just spawns a CMD subprocess and works the same as CMD. However, I've noticed that conemu is unable to reload system env variables if you change them - it always needs to reboot the computer. I don't know exactly which system variable it needed, but my guess is that this requirement to reboot can be attributed to conemu and that it should not affect other users.

@peci1
Copy link
Contributor Author

peci1 commented Sep 15, 2024

I've removed the wording about VS command prompt from this PR.

Signed-off-by: Martin Pecka <peci1@seznam.cz>
@azeey
Copy link
Contributor

azeey commented Sep 16, 2024

We've noticed that versions of tinyxml or Bullet installed via chocolatey (as used in the ROS 2 install instructions) might cause weird issues like this.

@peci1
Copy link
Contributor Author

peci1 commented Sep 16, 2024

Oh, I haven't dived into any ROS 2 + GZ tutorials. Combining Conda + Choco packages, that must be a very "interesting"... hell :-D

Signed-off-by: Martin Pecka <peci1@seznam.cz>
@azeey azeey merged commit 1eb1c1a into gazebosim:master Sep 16, 2024
5 checks passed
@Apurborj
Copy link

Hi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants