[issue13670] Increase test coverage for pstats.py

2019-03-16 Thread andrea crotti
andrea crotti added the comment: It has been a long time but if it's still useful sure. I can see some tests have been added in commit 863b1e4d0e95036bca4e97c1b8b2ca72c19790fb but if these are still relevant I'm happy to go ahead. -- ___ Python

[issue13991] namespace packages depending on order

2012-02-13 Thread andrea crotti
andrea crotti andrea.crott...@gmail.com added the comment: I reopen the ticket because I'm still not convinced.. I tried to substitute the setuptools namespace declaration with the more standard python: from pkgutil import extend_path __path__ = extend_path(__path__, __name__) It behaves

[issue13991] namespace packages depending on order

2012-02-13 Thread andrea crotti
Changes by andrea crotti andrea.crott...@gmail.com: -- resolution: invalid - ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13991 ___ ___ Python

[issue13991] namespace packages depending on order

2012-02-12 Thread andrea crotti
andrea crotti andrea.crott...@gmail.com added the comment: About the binary file, in theory I agree with you, but that archive contains 5 or 6 subdirectories with a few almost empty files. Of course I can write a script that recreates that situation, but does it make sense when I can just tar

[issue13991] namespace packages depending on order

2012-02-11 Thread andrea crotti
andrea crotti andrea.crott...@gmail.com added the comment: There is nothing binary in the archive, just a simple example of namespace packages, which was the minimal example that I could create to make things fail. I use the standard pkg_resources way to do things: __import__('pkg_resources

[issue13991] namespace packages depending on order

2012-02-10 Thread andrea crotti
New submission from andrea crotti andrea.crott...@gmail.com: I am not really sure that it is a bug, but for me it's at least not the expected behaviour. In short supposing I have two namespace packages ab and ac (as seen in the tar file), this works perfectly: import sys from os import path

[issue13670] Increase test coverage for pstats.py

2011-12-28 Thread andrea crotti
New submission from andrea crotti andrea.crott...@gmail.com: This patch increases test coverage for pstats.py from 25 to 36%. It's my first proposed patch so sorry in advance if there are problems. Much more can be done for pstats.py (which is also not much commented) but I want to get some

[issue13670] Increase test coverage for pstats.py

2011-12-28 Thread andrea crotti
andrea crotti andrea.crott...@gmail.com added the comment: It's really hard to understand true, and if should not go in the patch in general of course. The sense was that the only test I added is trivial, but I haven't produced something better yet. And ok I will remove the docstrings, I