Message265242
I merged Victor’s patch with the current code and addressed most of the comments:
* restore re.escape()
* single "-l" + name argument
* copy with dict(os.environ)
* redirect GCC stderr=STDOUT
* changed tempfile cleanup to try / finally
I also added a test case.
I kept Victor’s behaviour of not raising OSError when the command is missing. I think this should be considered separately, and only changed for 3.6+, if at all. The buggy code was added in Issue 4861.
I only have Linux and GCC, but I briefly tested each platform-specific branch by hacking the “if” statements and creating mock crle, ldconfig, etc commands, so I am somewhat confident that everything is still working. |
|
| Date |
User |
Action |
Args |
| 2016-05-10 12:01:49 | martin.panter | set | recipients:
+ martin.panter, barry, vstinner, Arfrever, serhiy.storchaka, koobs |
| 2016-05-10 12:01:49 | martin.panter | set | messageid: <[email protected]> |
| 2016-05-10 12:01:49 | martin.panter | link | issue22636 messages |
| 2016-05-10 12:01:49 | martin.panter | create | |
|