[GitHub] [incubator-mxnet] DickJC123 edited a comment on issue #14392: Debug bulking test on windows [WIP]

[GitBox <gi...@apache.org>]

DickJC123 edited a comment on issue #14392: Debug bulking test on windows [WIP]
URL: https://github.com/apache/incubator-mxnet/pull/14392#issuecomment-472124999
 
 
   If this latest commit passes CI, it will be ready for merging.  The problem is that when one performs in Python:
   ```
   os.environ[<key>] = <valu>
   ```
   this effects what the C++ backend sees only on Linux.  Under Windows, some additional (and possibly windows-version-specific) work would be required to push the environment change to the msvcrt, which otherwise gets a copy of the environment at process start-up.  My preferred fix here is to resort to the more bomb-proof approach of setting environment variables, then spawning a new process to perform the test (although admittedly this is bypassed on Python2 due to lack of spawn support).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services