URL: https://github.com/freeipa/freeipa/pull/243
Author: tiran
 Title: #243: Don't modify redhat_system_units
Action: opened

PR body:
"""
ipaplatform.fedora.services used to modify the redhat_system_units dict.
It now creates a proper shallow copy.

Signed-off-by: Christian Heimes <chei...@redhat.com>
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/243/head:pr243
git checkout pr243
From 42e856dcdbc54508e6445d81d613c6cdab311294 Mon Sep 17 00:00:00 2001
From: Christian Heimes <chei...@redhat.com>
Date: Wed, 16 Nov 2016 10:39:05 +0100
Subject: [PATCH] Don't modify redhat_system_units

ipaplatform.fedora.services used to modify the redhat_system_units dict.
It now creates a proper shallow copy.

Signed-off-by: Christian Heimes <chei...@redhat.com>
---
 ipaplatform/fedora/services.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipaplatform/fedora/services.py b/ipaplatform/fedora/services.py
index 5b1dfc8..a3b951a 100644
--- a/ipaplatform/fedora/services.py
+++ b/ipaplatform/fedora/services.py
@@ -26,7 +26,7 @@
 
 # Mappings from service names as FreeIPA code references to these services
 # to their actual systemd service names
-fedora_system_units = redhat_services.redhat_system_units
+fedora_system_units = redhat_services.redhat_system_units.copy()
 
 # Service that sets domainname on Fedora is called fedora-domainname.service
 fedora_system_units['domainname'] = 'fedora-domainname.service'
-- 
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