[Freeipa-devel] [PATCH] Minor fix in ipa-adtrust-install

2012-02-22 Thread Jan Zelený
If not set, it causes the script to fail with traceback on some machines (for 
example when resolve_host() couple lines below threw an exception).

Jan
From b312e4210866a11266d12b56e2be4ca08dc94379 Mon Sep 17 00:00:00 2001
From: Jan Zeleny jzel...@redhat.com
Date: Wed, 22 Feb 2012 16:01:37 +0100
Subject: [PATCH] Initialize IP when installing AD trust in case some error
 occurs

---
 install/tools/ipa-adtrust-install |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/install/tools/ipa-adtrust-install b/install/tools/ipa-adtrust-install
index 248ea35eaa86dd59ebbc871b86df780cfd71ccf6..0f3e473f839a0e8e0b387108fc4a6b87b36ba2c1 100755
--- a/install/tools/ipa-adtrust-install
+++ b/install/tools/ipa-adtrust-install
@@ -131,6 +131,7 @@ def main():
 break
 
 # Check we have a public IP that is associated with the hostname
+ip = None
 try:
 if options.ip_address:
 ip = ipautil.CheckedIPAddress(options.ip_address, match_local=True)
-- 
1.7.6.5



signature.asc
Description: This is a digitally signed message part.
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] Minor fix in ipa-adtrust-install

2012-02-22 Thread Alexander Bokovoy
On Wed, 22 Feb 2012, Jan Zelený wrote:

 If not set, it causes the script to fail with traceback on some machines (for 
 example when resolve_host() couple lines below threw an exception).
ACK.

Please commit as oneliner, I had same in my local copy for long time, 
slated to come with trusts. 


-- 
/ Alexander Bokovoy

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel