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

fix problem on importing submodule #8

Closed
wants to merge 1 commit into from
Closed

fix problem on importing submodule #8

wants to merge 1 commit into from

Conversation

yocchi
Copy link

@yocchi yocchi commented Aug 15, 2011

problem:
rtprint cannot import submodule like A.B.
$ rtprint -m A.B rtc:port
rtprint: No module named A.B

solution:
use __import__ instead of imp.find_module & imp.load_module.
"__POA" module importing is also fixed.

@gbiggs gbiggs closed this in 5b2a917 Apr 21, 2014
@gbiggs
Copy link
Owner

gbiggs commented Apr 21, 2014

Importing of sub-modules is fixed.

The provided solution for the POAs was not a fix. the POA module is not guaranteed to be the top-level module (just as it is not guaranteed to be the bottom-level module). Until a better solution that actively searches for the POA module is implemented, I recommend that you specify both the top-level module and the complete module path to be loaded on the command line. The command will fail to load the POA module for one of them, but it will still load all the rest correctly.

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