diff options
author | Jan Cholasta <jcholast@redhat.com> | 2016-11-24 09:46:42 +0100 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-11-29 14:50:51 +0100 |
commit | 9117a5d5a6ae7b3b97407e46f81a06c387974d7f (patch) | |
tree | 2459cb75a507cbdfd928c5881d1009c471755a27 | |
parent | a291c6ded91611ea2bd1a1fdb96314721d73a75f (diff) | |
download | freeipa-9117a5d5a6ae7b3b97407e46f81a06c387974d7f.tar.gz freeipa-9117a5d5a6ae7b3b97407e46f81a06c387974d7f.tar.xz freeipa-9117a5d5a6ae7b3b97407e46f81a06c387974d7f.zip |
paths: remove DEV_NULL
The platform-specific path to /dev/null is provided by the Python standard
library in os.devnull.
Replace all uses of paths.DEV_NULL with os.devnull and remove DEV_NULL.
https://fedorahosted.org/freeipa/ticket/6474
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
-rw-r--r-- | ipaclient/install/client.py | 2 | ||||
-rw-r--r-- | ipaplatform/base/paths.py | 1 | ||||
-rw-r--r-- | ipapython/ipautil.py | 2 |
3 files changed, 2 insertions, 3 deletions
diff --git a/ipaclient/install/client.py b/ipaclient/install/client.py index d18d8bb1f..410201606 100644 --- a/ipaclient/install/client.py +++ b/ipaclient/install/client.py @@ -1083,7 +1083,7 @@ def configure_sshd_config(fstore, options): ) for candidate in candidates: - args = ['sshd', '-t', '-f', paths.DEV_NULL] + args = ['sshd', '-t', '-f', os.devnull] for item in candidate.items(): args.append('-o') args.append('%s=%s' % item) diff --git a/ipaplatform/base/paths.py b/ipaplatform/base/paths.py index a30e6b33b..d4f7e8a66 100644 --- a/ipaplatform/base/paths.py +++ b/ipaplatform/base/paths.py @@ -33,7 +33,6 @@ class BasePathNamespace(object): SYSTEMCTL = "/bin/systemctl" TAR = "/bin/tar" BIN_TRUE = "/bin/true" - DEV_NULL = "/dev/null" DEV_STDIN = "/dev/stdin" AUTOFS_LDAP_AUTH_CONF = "/etc/autofs_ldap_auth.conf" ETC_DIRSRV = "/etc/dirsrv" diff --git a/ipapython/ipautil.py b/ipapython/ipautil.py index 472ba3596..fcb2e3500 100644 --- a/ipapython/ipautil.py +++ b/ipapython/ipautil.py @@ -413,7 +413,7 @@ def run(args, stdin=None, raiseonerr=True, nolog=(), env=None, if stdin: p_in = subprocess.PIPE if skip_output: - p_out = p_err = open(paths.DEV_NULL, 'w') + p_out = p_err = open(os.devnull, 'w') elif redirect_output: p_out = sys.stdout p_err = sys.stderr |