This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Title: Clean up use of packages_root/package_dir
Type: Stage: resolved
Components: Distutils2 Versions: Python 3.3
Status: closed Resolution: out of date
Dependencies: Superseder:
Assigned To: eric.araujo Nosy List: alexis, eric.araujo, erik.bray, tarek
Priority: normal Keywords:

Created on 2011-06-20 19:20 by erik.bray, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Messages (2)
msg138747 - (view) Author: Erik Bray (erik.bray) * (Python triager) Date: 2011-06-20 19:20
This issue was first mentioned in issue12375.  The Distribution.package_dir attribute (which also appears in a few commands) is not used in a consistent manner.  Some code expects it to be a string, while other code expects it to be a dict.

I believe the correct behavior now is for it to be a string, since only one root directory for Python packages is allowed in packaging.

Eric Araujo also points out that packages_root and the associated package_dir attribute are misnamed: They also point to the root for any Python modules.  I'm not sure what a better name would be though--I'm tempted to suggest "source_dir", but that seems a little broad since it doesn't necessarily include C extension source.
msg138749 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2011-06-20 19:23
I would rename it modules_root, but a related discussion last summer showed that using “modules” to refer to pure Python modules, extension modules and Python packages was not 100% ideal for 100% of users.
Date User Action Args
2022-04-11 14:57:18adminsetgithub: 56586
2014-03-12 10:28:08eric.araujosetstatus: open -> closed
resolution: out of date
stage: needs patch -> resolved
2011-06-20 19:23:51eric.araujosetassignee: tarek -> eric.araujo
title: Clean up packages_root option -> Clean up use of packages_root/package_dir
messages: + msg138749
stage: needs patch
2011-06-20 19:20:42erik.braycreate