Re: [pypy-dev] [pypy-svn] r59181 - pypy/branch/cbuild-refactor/pypy/translator/platform

On Fri, Oct 17, 2008 at 3:19 PM, <afa@codespeak.net> wrote:
Author: afa Date: Fri Oct 17 15:19:27 2008 New Revision: 59181
Modified: pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py Log: Log message about updated environment
Modified: pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py ============================================================================== --- pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py (original) +++ pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py Fri Oct 17 15:19:27 2008 @@ -38,6 +38,7 @@ key, value = line.split('=', 1) if key.upper() in ['PATH', 'INCLUDE', 'LIB']: os.environ[key] = value + log.msg("Updated environment with %s" % (vcvars,))
try: _install_msvc_env() _______________________________________________ pypy-svn mailing list pypy-svn@codespeak.net http://codespeak.net/mailman/listinfo/pypy-svn
Can you please make sure that os.environ is cleaned afterwards? thanks

On Sat, Oct 18, 2008 at 6:07 PM, Maciej Fijalkowski <fijall@gmail.com> wrote:
On Fri, Oct 17, 2008 at 3:19 PM, <afa@codespeak.net> wrote:
Author: afa Date: Fri Oct 17 15:19:27 2008 New Revision: 59181
Modified: pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py Log: Log message about updated environment
Modified: pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py ============================================================================== --- pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py (original) +++ pypy/branch/cbuild-refactor/pypy/translator/platform/windows.py Fri Oct 17 15:19:27 2008 @@ -38,6 +38,7 @@ key, value = line.split('=', 1) if key.upper() in ['PATH', 'INCLUDE', 'LIB']: os.environ[key] = value + log.msg("Updated environment with %s" % (vcvars,))
try: _install_msvc_env() _______________________________________________ pypy-svn mailing list pypy-svn@codespeak.net http://codespeak.net/mailman/listinfo/pypy-svn
Can you please make sure that os.environ is cleaned afterwards? thanks
Ah, xoraxax just pointed out that this is once-per-import, so it's completely fine.
participants (1)
-
Maciej Fijalkowski