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

PR body:
"""
This was introduced in commit d0ea0bb63891babd1c5778df2e291b527c8e927c
as F14 compatibility. PATH should be always inherited from from
os.environ and then amended also this is platform specific and should not
be in core code
"""

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 1e5e694273751593e6fa4cb53556c32d17cb7f82 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. PATH should be always inherited from 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:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to