Changes by Terry J. Reedy tjre...@udel.edu:
--
versions: +Python 3.3 -Python 3.1, Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
Changes by Brett Cannon br...@python.org:
--
nosy: -brett.cannon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
___
Python-bugs-list
Éric Araujo added the comment:
Eric, as you know importlib, can you comment about this? If zipimport now
works with the invalidate_caches system, then we do have a documented way of
clearing the cache, and I would recommend closing this bug (the existing
attribute should stay for compat, but
Eric Snow added the comment:
I'll take a look but can't say when. My wife's is having a baby in the next
couple days so I'm a bit swamped. :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Éric Araujo added the comment:
No problem (and congratulations :)
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
___
Changes by Éric Araujo mer...@netwok.org:
Removed file: http://bugs.python.org/file13625/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
Changes by Éric Araujo mer...@netwok.org:
Removed file: http://bugs.python.org/file13638/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
Éric Araujo mer...@netwok.org added the comment:
Could the recently-added functools.lru_cache help here?
--
components: -Distutils
nosy: +eric.araujo
versions: +Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Brett Cannon br...@python.org added the comment:
2009/4/5 Tarek Ziadé rep...@bugs.python.org@psf.upfronthosting.co.za
Tarek Ziadé ziade.ta...@gmail.com added the comment:
Does distutils use _zip_directory_cache in any way?
Not at all. But pkgutil does,
Ah, OK.
and uses to get a
Tarek Ziadé ziade.ta...@gmail.com added the comment:
_zip_directory_cache is part of the zipimport module (C module) you can
find in Modules/zipimport.c.
The tests are located here: ./Lib/test/test_zipimport.py
_zip_directory_cache is a dict used by zipimporter_init to speed up the
lookup.
Maciek Fijalkowski fi...@genesilico.pl added the comment:
The bug with this is that setuptools uses it (and even relies on what is
there and in what order), while it has no single tests and no documentation.
--
___
Python tracker
Tarek Ziadé ziade.ta...@gmail.com added the comment:
while it has no single tests and no documentation.
what it refers to ? zipimport, distutils or setuptools ?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Maciek Fijalkowski fi...@genesilico.pl added the comment:
it == _zip_directory_cache
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
___
Tarek Ziadé ziade.ta...@gmail.com added the comment:
I am not sure what you would like to see added.
in zipimport.c:
- _zip_directory_cache: a dict, mapping archive paths to zip directory
info dicts, as used in zipimporter._files.
while there are no direct test, Lib/test/test_zipimport.py
Tarek Ziadé ziade.ta...@gmail.com added the comment:
maybe what's missing is a public API for the clear() method since
setuptools uses it by calling it directly
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Maciek Fijalkowski fi...@genesilico.pl added the comment:
setuptools at least deletes stuff from _zip_directory_cache (via _uncache
function in easy_install.py). it also iterates over it and checks for
existance of elements.
What about documenting details about _zip_directory_cache:
* does it
Brett Cannon br...@python.org added the comment:
Does distutils use _zip_directory_cache in any way? If not then setuptools
is in the wrong here for using a private attribute and it is on them to make
it work if the attribute does not exist, period.
Otherwise someone will need to come up with a
Tarek Ziadé ziade.ta...@gmail.com added the comment:
Does distutils use _zip_directory_cache in any way?
Not at all. But pkgutil does, and uses to get a dirlist,
so I guess a new API should be created in zipimport to handle this case
as well. (I think it's some setuptools code originallyt,
Changes by Tarek Ziadé ziade.ta...@gmail.com:
--
priority: - normal
versions: +Python 2.7, Python 3.1 -Python 2.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
Brett Cannon br...@python.org added the comment:
Actually importlib does not test this; that is an old thing from an old
sandbox version of importlib that I no longer use.
I am assigning to Tarek as the better solution is for distutils to stop
using this private attribute.
--
assignee:
Georg Brandl ge...@python.org added the comment:
That's fortunate, because I can just reassign this to him :)
--
assignee: georg.brandl - brett.cannon
nosy: +brett.cannon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Georg Brandl ge...@python.org added the comment:
I can't find a reference to _zip_directory_cache in distutils. Do you
mean setuptools?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Maciek Fijalkowski fi...@genesilico.pl added the comment:
Seems I meant setuptools indeed. Note that brett's importlib contains
some tests for _zip_directory_cache.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
Changes by Daniel Diniz aja...@gmail.com:
--
nosy: +tarek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2953
___
___
Python-bugs-list mailing list
New submission from Maciek Fijalkowski [EMAIL PROTECTED]:
_zip_directory_cache has no single test nor piece of documentation. The
only place where it's mentioned is a call to method .clear() on it in
tests. Besides, it can be whatever. Despite _ prefix, which I would
consider to be private,
25 matches
Mail list logo
