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

[pr2_calibration_launch] PYTHONPATH for hacked urdf_parser_py in capture_exec_pr2.launch #11

Open
wants to merge 1 commit into
base: hydro-devel
Choose a base branch
from

Conversation

wkentaro
Copy link
Member

No description provided.

@wkentaro
Copy link
Member Author

Do you mean we should change install() in CMakeLists.txt ?

@UltronDestroyer
Copy link
Contributor

Yes.

@UltronDestroyer
Copy link
Contributor

By installing the python script to the correct location it'll automatically get added to your PYTHONPATH since python path looks for ROS python scripts

@wkentaro
Copy link
Member Author

I see.
But I'm afraid that it may affect other repositories executables, because of using hacked version urdf_parser_py in this repository. Other pkgs expect standard urdf_parser_py which is installed via pip or apt.
I think the hacked version urdf_parser_py should be used only in pr2_calibration_launch pkg.

@UltronDestroyer
Copy link
Contributor

We can just install it under pr2_calibration_launch directory.

@wkentaro
Copy link
Member Author

By doing so, is it possible for only scripts in pr2_calibration_launch see hacked version urdf_parser_py without setting PYTHONPATH ?
Actually I don't have idea how to do it, with my PYTHONPATH environment variable.
PYTHONPATH=/home/applications/ros/hydro/devel/lib/python2.7/dist-packages:/opt/ros/hydro/lib/python2.7/dist-packages

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.

2 participants