URL: https://github.com/freeipa/freeipa/pull/204
Author: mbasti-rh
 Title: #204: ipautil.run: Remove hardcoded environ PATH value
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/204/head:pr204
git checkout pr204
From 078c05f6e2bb52f38d0b68a6b9a9e8429d977374 Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Tue, 1 Nov 2016 17:32:13 +0100
Subject: [PATCH] ipautil.run: Remove hardcoded environ PATH value

This was introduced in commit d0ea0bb63891babd1c5778df2e291b527c8e927c
as F14 compatibility. We don't need to have F14 compatibility anymore.
PATH should be always inherited from os.environ and then amended
also this is platform specific and should not
be in core code
 ipapython/ipautil.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/ipapython/ipautil.py b/ipapython/ipautil.py
index f7d7537..c6e7c4a 100644
--- a/ipapython/ipautil.py
+++ b/ipapython/ipautil.py
@@ -409,7 +409,6 @@ def run(args, stdin=None, raiseonerr=True, nolog=(), env=None,
     if env is None:
         # copy default env
         env = copy.deepcopy(os.environ)
-        env["PATH"] = "/bin:/sbin:/usr/kerberos/bin:/usr/kerberos/sbin:/usr/bin:/usr/sbin"
     if stdin:
         p_in = subprocess.PIPE
     if skip_output:
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to