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

catkin should support the python setup.py argument py_modules #399

Closed
wjwwood opened this issue Mar 25, 2013 · 0 comments
Closed

catkin should support the python setup.py argument py_modules #399

wjwwood opened this issue Mar 25, 2013 · 0 comments
Assignees

Comments

@wjwwood
Copy link
Member

wjwwood commented Mar 25, 2013

When using this argument to setup.py's setup() function catkin warns:

*** Arguments ['py_modules'] to setup() not supported in catkin devel space in setup.py of camera_info_manager_py

It ges installed into the install space, but it does not create a redirection placeholder for the devel space.

Since this is a core functionality of python's distribution system we should support it.

Here is an example setup.py:

from distutils.core import setup
from catkin_pkg.python_setup import generate_distutils_setup

d = generate_distutils_setup(
    py_modules=['camera_info_manager'],
    package_dir={'': 'src'}
)

setup(**d)
@ghost ghost assigned dirk-thomas Dec 17, 2013
dirk-thomas added a commit that referenced this issue Dec 23, 2013
add support for py_modules in setup.py (fix #399)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants