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

Add importModuleFromResources (#2) #4

Closed
wants to merge 8 commits into from
Closed

Conversation

brvier
Copy link

@brvier brvier commented Oct 14, 2013

Add an importModuleFromResources, which read the code from the resources
file, and didn't use the importlib so can be use with python < 3.1.
This doesn't enable import of module located in QtResources from the
python code, which will require a more complex implementation with hook
in python capi. This solve the #2 enhancement request.

Benoît HERVIER added 8 commits October 14, 2013 11:23
Add an importModuleFromResources, which read the code from the resources
file, and didn't use the importlib so can be use with python < 3.1.
This doesn't enable import of module located in QtResources from the
python code, which will require a more complex implementation with hook
in python capi. This solve the thp#2 enhancement request.
…can be imported from pycode with import. That s works in progress fullpath aren t managed, and bytecode py[c|o] file can t be used. The path finder should be written and not hardcoded with ":/"+module_name+".py"
@thp
Copy link
Owner

thp commented Jan 20, 2014

Hi, can you squash those changes, rebase them against the latest Git master revision and remove all the code that you commented out because it wasn't needed?

@thp
Copy link
Owner

thp commented Feb 16, 2014

Obsoleted by commit 5c8d9cf, which implements importing transparently using qrc:/ URLs and the normal addImportPath function.

@thp thp closed this Feb 16, 2014
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.

None yet

2 participants