On Thu, 2013-02-14 at 16:19 -0800, Brian C. Lane wrote:
From: "Brian C. Lane" <bcl(a)redhat.com>
Some package scripts may call utilities using dbus. Since this is just a
chroot that will fail. This unsets DESKTOP and DBUS_SESSION_BUS_ADDRESS
to keep them from crashing.
---
src/pylorax/__init__.py | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/pylorax/__init__.py b/src/pylorax/__init__.py
index e1c0069..3eff6e8 100644
--- a/src/pylorax/__init__.py
+++ b/src/pylorax/__init__.py
@@ -121,6 +121,10 @@ class Lorax(BaseLoraxClass):
# so we have to add it ourselves
os.environ["PATH"] =
"{0}:/sbin:/usr/sbin".format(os.environ["PATH"])
+ # remove some environmental variables that can cause problems with package
scripts
+ env_remove = ('DISPLAY', 'DBUS_SESSION_BUS_ADDRESS')
+ [os.environ.pop(k) for k in env_remove if k in os.environ]
+
Looks good to me. Interesting list comprehension usage. :)
--
Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic