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

Update README.md #80

Open
wants to merge 1 commit into
base: spot_arm
Choose a base branch
from
Open

Conversation

a-ichikura
Copy link

@a-ichikura a-ichikura commented Nov 1, 2022

I tried to make spot_ws on my laptop. Then found an error.
wstool merge -t . jsk-ros-pkg/jsk_robot/jsk_spot_robot/jsk_spot_user.rosinstall to
wstool merge -t . jsk-ros-pkg/jsk_robot/jsk_spot_robot/jsk_spot_driver.rosinstall

I tried to make spot_ws on my laptop. Then found an error.
`wstool merge -t . jsk-ros-pkg/jsk_robot/jsk_spot_robot/jsk_spot_user.rosinstall`
to
`wstool merge -t . jsk-ros-pkg/jsk_robot/jsk_spot_robot/jsk_spot_driver.rosinstall`
@tkmtnt7000
Copy link
Collaborator

I think jsk_spot_driver.rosinstall may be so large that we make workspace in our remote PC.
I try to search jsk_spot_user.rosinstall from develop/spot branch.

@tkmtnt7000
Copy link
Collaborator

Because it is very hard to cherry-pick commits related to jsk_spot_user.rosinstall, I will create new commit ......

@tkmtnt7000
Copy link
Collaborator

40c15b5

@k-okada
Copy link
Owner

k-okada commented Nov 1, 2022

@a-ichikura sorry,
please git pull k-okada spot_arm and you can find jsk_spot_user.rosinstall. I have not tested this file on real robot yet, so if you have any problem, please let me know.
9de0873

@a-ichikura
Copy link
Author

I did build after new jsk_spor_user.rosinstall, then I had no problem.
It seems to work well.

@k-okada k-okada force-pushed the spot_arm branch 4 times, most recently from 4d64e9f to 7908647 Compare November 20, 2024 01:38
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.

3 participants