[issue19205] Don't import re and sysconfig in site.py

2013-10-21 Thread Christian Heimes
Christian Heimes added the comment: Good point, #19325 -- stage: commit review -> committed/rejected status: open -> closed ___ Python tracker ___ ___

[issue19205] Don't import re and sysconfig in site.py

2013-10-21 Thread STINNER Victor
STINNER Victor added the comment: > The imports in _osx_support counteract the performance boost. I neither have > an OS X machine and nor do I understand the internals of _osx_support. > Perhaps somebody else likes to work on the module. Please open a separated issue for OS X, this issue can

[issue19205] Don't import re and sysconfig in site.py

2013-10-21 Thread Christian Heimes
Christian Heimes added the comment: The imports in _osx_support counteract the performance boost. I neither have an OS X machine and nor do I understand the internals of _osx_support. Perhaps somebody else likes to work on the module. -- assignee: christian.heimes -> components: +Mac

[issue19205] Don't import re and sysconfig in site.py

2013-10-12 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9f6ef09f6492 by Christian Heimes in branch 'default': Issue #19205: _osx_support uses the re module all over the place. Omit the test for nw. http://hg.python.org/cpython/rev/9f6ef09f6492 -- ___ Python t

[issue19205] Don't import re and sysconfig in site.py

2013-10-12 Thread STINNER Victor
STINNER Victor added the comment: > New changeset a57dfbba91f9 by Christian Heimes in branch 'default': > Issue #19205: add debugging output for failing test on Snow Leopard > http://hg.python.org/cpython/rev/a57dfbba91f9 So the "import re" comes from _osx_support, _osx_support is imported by s

[issue19205] Don't import re and sysconfig in site.py

2013-10-12 Thread Roundup Robot
Roundup Robot added the comment: New changeset a57dfbba91f9 by Christian Heimes in branch 'default': Issue #19205: add debugging output for failing test on Snow Leopard http://hg.python.org/cpython/rev/a57dfbba91f9 -- ___ Python tracker

[issue19205] Don't import re and sysconfig in site.py

2013-10-12 Thread Antoine Pitrou
Antoine Pitrou added the comment: Christian, the test is failing on Snow Leopard: == FAIL: test_startup_imports (test.test_site.StartupImportTests) -- Tracebac

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Roundup Robot
Roundup Robot added the comment: New changeset 2cd1b28d1666 by Christian Heimes in branch 'default': Issue #19205 fix 406529adf156 http://hg.python.org/cpython/rev/2cd1b28d1666 -- ___ Python tracker ___

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Christian Heimes
Christian Heimes added the comment: Thanks for your input! -- resolution: -> fixed stage: patch review -> commit review status: open -> closed ___ Python tracker ___ ___

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Roundup Robot
Roundup Robot added the comment: New changeset 406529adf156 by Christian Heimes in branch 'default': Issue #19205: Don't import the 're' module in site and sysconfig module to http://hg.python.org/cpython/rev/406529adf156 -- nosy: +python-dev ___ Pyth

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread STINNER Victor
STINNER Victor added the comment: I accept hacks to speedup Python is the site module, but it becomes more surprising in the locale module. The issue #9548 proposes to a more generic solution for the locale module at startup. -CONFIG_LINE = re.compile(r'^(?P(\w|[-_])+)\s*=\s*(?P.*)\s*$') +CON

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Antoine Pitrou
Antoine Pitrou added the comment: Review posted on Rietveld. -- nosy: +pitrou stage: needs patch -> patch review ___ Python tracker ___ __

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Christian Heimes
Christian Heimes added the comment: Here is a new patch with unit test and patch for the locale module. The locale modules is loaded by the _io module when any standard stream is not connected to a terminal (or so). -- Added file: http://bugs.python.org/file32048/startup_no_re.patch _

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Christian Heimes
Changes by Christian Heimes : Removed file: http://bugs.python.org/file32017/site_no_re.patch ___ Python tracker ___ ___ Python-bugs-list mail

[issue19205] Don't import re and sysconfig in site.py

2013-10-11 Thread Christian Heimes
Changes by Christian Heimes : Removed file: http://bugs.python.org/file32023/sysconfig_delay_re.patch ___ Python tracker ___ ___ Python-bugs-l

[issue19205] Don't import re and sysconfig in site.py

2013-10-09 Thread Christian Heimes
Christian Heimes added the comment: The site and sysconfig module are too intermingled to remove the import of sysconfig from the site module. However the sysconfig module doesn't use the re module in most cases anymore. The parsing functions for Makefile and pyconfig.h are only used by distut

[issue19205] Don't import re and sysconfig in site.py

2013-10-09 Thread Barry A. Warsaw
Changes by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue19205] Don't import re and sysconfig in site.py

2013-10-09 Thread Christian Heimes
Changes by Christian Heimes : -- keywords: +patch Added file: http://bugs.python.org/file32017/site_no_re.patch ___ Python tracker ___ ___

[issue19205] Don't import re and sysconfig in site.py

2013-10-09 Thread Christian Heimes
New submission from Christian Heimes: The site module is loaded at every startup. Nowadays it imports the re and the sysconfig modules. The re module is used for venv config parsing and inside sysconfig. sysconfig is loaded to find the location of the user's site-packages directory. Suggestio