When system_info.py is trying to find cblas as libblas, there can be a
different library called libblas that doesn't contain the referenced
function cblas_ddot. This raises a distutils.errors.LinkError, which
is currently not caught.

This patch fixes #15389.

The system_info in master was reworked for 1.17, and doesn't have this
issue.

* distutils/system_info.py fix missing subprocess import

* Update system_info.py

which used to fail for ifort

The patch was a workaround for our broken `exec_command`. It was removed
in gh-1217, but then added back in gh-1321.  Now that we've fixed
exec_command, I believe we can finally take it out for good.

Co-Authored-By: rainwoodman <rainwoodman@gmail.com>

This PR fixes #13074.

* BUG: Use subprocess.check_output to avoid leaking file handles

This emitted a ResourceWarning as previously written, as the streams
never got closed.

This change also detects errors in `svnversion` and `hg`, rather than
ignoring the error code and hoping stdout contains a revision anyway.

* BUG: Fix distutils.get_version on python 3

Previously, this would throw `TypeError: cannot use a string pattern on
a bytes-like object`, which would then be silenced and never seen.

The previous commit stopped silencing this error, so now everything is
visibly broken rather than silently broken

Changing the strings to raw strings fixes the problem

* BUG: Don't use `close_fds`, which fails on windows

This argument was introduced along with subprocess in
f5afa7836d56640f25ff3fbd2c59b8dcf016e26f to replace `popen4`, and
probably wasn't necessary in the first place.

BUG: parse shell escaping in extra_compile_args and extra_link_args