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

Incorrect __init__.py script installed when not using catkin_make #47

Closed
jbrindza opened this issue Apr 15, 2014 · 2 comments
Closed

Incorrect __init__.py script installed when not using catkin_make #47

jbrindza opened this issue Apr 15, 2014 · 2 comments

Comments

@jbrindza
Copy link

If you compile and install a package that contains python source code and messages the <package_name>/__init__.py file from the devel space gets installed into the install space overwriting the previously installed __init__.py from the source.

You can recreate this problem by building dynamic_reconfigure with catkin_make_isolated. The install_isolated/lib/python2.7/dist-packages/dynamic_reconfigure/__init__.py file is the one from devel_isolated/dynamic_reconfigure/lib/python2.7/dist-packages/dynamic_reconfigure/__init__.py.

I believe it is due to the install regex rule for installing __init__.py files from subdirectories. I assume this rule is matching <pkg_name>/./__init__.py

 install(
 DIRECTORY ${CATKIN_DEVEL_PREFIX}/${@(l)_INSTALL_DIR}/@pkg_name
 DESTINATION ${@(l)_INSTALL_DIR}
 FILES_MATCHING
 REGEX "${CATKIN_DEVEL_PREFIX}/${@(l)_INSTALL_DIR}/@(pkg_name)/.+/__init__.pyc?$"
 )

Strangely, this does not happen with catkin_make, I don't understand why its different.

Same behaviour using cmake version 2.8.12.2 and 2.8.7

@dirk-thomas
Copy link
Member

The problem should have been addressed some time ago with #38. Can you please mention with which genmsg version on what ROS distribution you are having the problem?

@jbrindza
Copy link
Author

Sorry about that, looks like I was using an old version. Updating to the latest version fixed the issue. Thanks.

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

No branches or pull requests

2 participants