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

Possible bug in import.c (2.1.2) #36216

Closed
jeffsasmor mannequin opened this issue Mar 7, 2002 · 2 comments
Closed

Possible bug in import.c (2.1.2) #36216

jeffsasmor mannequin opened this issue Mar 7, 2002 · 2 comments
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs)

Comments

@jeffsasmor
Copy link
Mannequin

jeffsasmor mannequin commented Mar 7, 2002

BPO 526726

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2002-03-10.20:42:08.000>
created_at = <Date 2002-03-07.00:43:03.000>
labels = ['interpreter-core']
title = 'Possible bug in import.c (2.1.2)'
updated_at = <Date 2002-03-10.20:42:08.000>
user = 'https://bugs.python.org/jeffsasmor'

bugs.python.org fields:

activity = <Date 2002-03-10.20:42:08.000>
actor = 'jvr'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['Interpreter Core']
creation = <Date 2002-03-07.00:43:03.000>
creator = 'jeffsasmor'
dependencies = []
files = []
hgrepos = []
issue_num = 526726
keywords = []
message_count = 2.0
messages = ['9556', '9557']
nosy_count = 2.0
nosy_names = ['jvr', 'jeffsasmor']
pr_nums = []
priority = 'normal'
resolution = None
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue526726'
versions = []

@jeffsasmor
Copy link
Mannequin Author

jeffsasmor mannequin commented Mar 7, 2002

Referencing the source distribution; file
Python/import.c:

lines 1881 through 1884 read:

if (parent == NULL) {
 PyErr_Format(PyExc_ImportError,
 "reload(): parent %.200s not in sys.modules",name); 

It might be more correct to subsitute the variable
parentname here (rather than 'name') since it is
the parentname'd module that wasn't found. Perhaps
there was some other intent in mind.....

@jeffsasmor jeffsasmor mannequin closed this as completed Mar 7, 2002
@jeffsasmor jeffsasmor mannequin added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Mar 7, 2002
@jeffsasmor jeffsasmor mannequin closed this as completed Mar 7, 2002
@jeffsasmor jeffsasmor mannequin added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Mar 7, 2002
@jvr
Copy link
Mannequin

jvr mannequin commented Mar 10, 2002

Logged In: YES
user_id=92689

No:

  1. I read the error string as "the parent of 'name' is not
    in sys.modules", which is indeed what's intended.
  2. parentname is a PyObject* and name is a char*
    (I assume it is correct to close this bug)

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
nedko pushed a commit to LADI/cpython that referenced this issue Jul 5, 2023
… netloc that

    normalize to separators. Closes: #924073.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs)
Projects
None yet
Development

No branches or pull requests

0 participants