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

allow from .foo import * (relative) sf 3309660 #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JonnyJD
Copy link
Owner

@JonnyJD JonnyJD commented Mar 28, 2012

patch from
https://sourceforge.net/support/tracker.php?aid=3309668

Tested on 3.0.1

A relative import like:
from .foo import *

is not parsed correctly.

| Source code parsing failed (but introspection was successful).
| Error: Error during parsing: invalid syntax
| -- Bad dotted name

With the attached patch it does parse correctly.


Note that general support for relative imports was added with
d38a577 in 2007-03-10.
This is just a special case, that was ommited before.

patch from
https://sourceforge.net/support/tracker.php?aid=3309668

Tested on 3.0.1

A relative import like:
from .foo import *

is not parsed correctly.

| Source code parsing failed (but introspection was successful).
| Error: Error during parsing: invalid syntax
| -- Bad dotted name

With the attached patch it does parse correctly.

----

Note that general support for relative imports was added with
d38a577 in 2007-03-10.
This is just a special case, that was ommited before.

Signed-off-by: Johannes Dewender <[email protected]>
@sfinkens
Copy link

The patch works fine, thanks! Could you please merge it into master?

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