<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
<br><br><div><div id="SkyDrivePlaceholder"></div>&gt; Date: Wed, 28 Nov 2012 17:50:09 +0100<br>&gt; From: okos@redhat.com<br>&gt; To: sssd-devel@lists.fedorahosted.org<br>&gt; Subject: Re: [SSSD] [PATHC] 1625-Confusing error messages for invalid        sssd.conf<br>&gt; <br>&gt; On 28/11/12 17:12, Ariel Barria wrote:<br>&gt; &gt; Thanks for your comments.<br>&gt; &gt;<br>&gt; &gt;  &gt; Date: Wed, 28 Nov 2012 14:38:29 +0100<br>&gt; &gt;  &gt; From: okos@redhat.com<br>&gt; &gt;  &gt; To: sssd-devel@lists.fedorahosted.org<br>&gt; &gt;  &gt; Subject: Re: [SSSD] [PATHC] 1625-Confusing error messages for invalid<br>&gt; &gt; sssd.conf<br>&gt; &gt;  &gt;<br>&gt; &gt;  &gt; On 28/11/12 01:31, Ariel Barria wrote:<br>&gt; &gt;  &gt; &gt; https://fedorahosted.org/sssd/ticket/1625<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt; &gt; /<br>&gt; &gt;  &gt; &gt; // Additional info:/<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt; &gt; ..../the class of configuration error messages could be<br>&gt; &gt;  &gt; &gt; changed to match the default debug level./<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt; &gt; Apparently this was covered with #1345 and in server.c<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt; &gt; / if (debug_level == SSSDBG_UNRESOLVED) {<br>&gt; &gt;  &gt; &gt; /* set debug level if any in conf_entry */<br>&gt; &gt;  &gt; &gt; ret = confdb_get_int(ctx-&gt;confdb_ctx, conf_entry,<br>&gt; &gt;  &gt; &gt; CONFDB_SERVICE_DEBUG_LEVEL,<br>&gt; &gt;  &gt; &gt; SSSDBG_DEFAULT,<br>&gt; &gt;  &gt; &gt; &amp;debug_level);/<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt;<br>&gt; &gt;  &gt; Hi Ariel,<br>&gt; &gt;  &gt;<br>&gt; &gt;  &gt; Could you add the path of database to the<br>&gt; &gt;  &gt; &gt; + DEBUG(SSSDBG_CRIT_FAILURE, ("Error loading configuration file: "<br>&gt; &gt;  &gt; &gt; "[%d]: %s\n", ret, strerror(ret)));<br>&gt; &gt;  &gt; part as well? And error messages from strerror are not much help in this<br>&gt; &gt;  &gt; case<br>&gt; &gt;  &gt; [22]: Invalid argument // for empty file<br>&gt; &gt;  &gt; [2]: No such file or directory // for invalid syntax in config file<br>&gt; &gt;  &gt; so could you change this accordingly to ticket proposal ?<br>&gt; &gt;  &gt;<br>&gt; &gt;  &gt; Thanks<br>&gt; &gt;  &gt;<br>&gt; &gt;  &gt; Ondra<br>&gt; &gt;  &gt; &gt;<br>&gt; &gt;  &gt;<br>&gt; &gt;<br>&gt; Thanks for the patch, but now, there's a bit wide if-else statement, <br>&gt; could you please rewrite it into swich? And append a commit message <br>&gt; (what's been done and why)<br><br>sure, new patch.<br><br><br></div>                                               </div></body>
</html>