Rob Crittenden wrote:
There are probably occasions where a caller will want more control over what happens when running a command fails. I've added an optional argument to run where it will not raise an exception on errors.

I've also added returncode to the tuple of things returned.


I forgot to include this additional change in the patch. When acked I'll add this bit too and commit it.

--- a/ipaserver/install/
+++ b/ipaserver/install/
@@ -100,7 +100,7 @@ class HTTPInstance(service.Service):
         if selinux:
                 # returns e.g. "httpd_can_network_connect --> off"
-                (stdout, stderr) =["/usr/sbin/getsebool",
+ (stdout, stderr, returncode) =["/usr/sbin/getsebool",

self.backup_state("httpd_can_network_connect", stdout.split()[2])

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Freeipa-devel mailing list

Reply via email to