On 08/16/2016 08:05 AM, Lukas Slebodnik wrote:
On (12/08/16 12:24), Justin Stephenson wrote:
Simple error message patch, resolves
https://fedorahosted.org/sssd/ticket/3130

Kind regards,

Justin Stephenson


From 080f9639e120329d069d4f0ba5edcc776e0179c2 Mon Sep 17 00:00:00 2001
From: Justin Stephenson <jstep...@redhat.com>
Date: Fri, 12 Aug 2016 12:12:57 -0400
Subject: [PATCH]     SSSCTL: More helpful error message when InfoPipe is
disabled

   Resolves:
   https://fedorahosted.org/sssd/ticket/3130
---
src/tools/sssctl/sssctl_sifp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/tools/sssctl/sssctl_sifp.c b/src/tools/sssctl/sssctl_sifp.c
index 
e541c4b27ba38e50b209b0957c8b38f03afc891a..d61754c095366d07bae812c38a24a88f07c197f5
 100644
--- a/src/tools/sssctl/sssctl_sifp.c
+++ b/src/tools/sssctl/sssctl_sifp.c
@@ -25,8 +25,8 @@
#include "util/util.h"
#include "tools/sssctl/sssctl.h"

-#define ERR_SSSD _("Check that SSSD is running and " \
-                   "the InfoPipe responder is enabled.\n")
+#define ERR_SSSD _("IFP Disabled: Please add the ifp service to the service" \
+                   "list in sssd.conf and restart the service.\n")
Here is a comment from the author of previous message.
05:26 < lslebodn> pbrezina: Do you have any coment to the patch
                   "SSSCTL: More helpful error message when
                   InfoPipe is disabled" ?
05:26 < lslebodn> becuase I plan to push it :-)
05:28 < pbrezina> lslebodn, I still think that the original message
                  is more accurate, but I don't oppose.

I tent to agree.
I would reserve the original message and wrote a "HINT" in next line.
+1

Any objection for such compromise?

LS

--
Petr^4 Čech
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to