[SSSD] [PATCH]-Bad debug message when adding services without explicit dns_discovery_domain

Ariel Barria olivares73 at hotmail.com
Thu Jun 14 04:58:06 UTC 2012


thanks for comments.

From: sgallagh at redhat.com
To: sssd-devel at lists.fedorahosted.org
Date: Wed, 13 Jun 2012 07:27:54 -0400
Subject: Re: [SSSD] [PATCH]-Bad debug message when adding services without explicit dns_discovery_domain

On Tue, 2012-06-12 at 23:20 -0500, Ariel Barria wrote:
> thanks for comments
> 
> 
> > From: simo at redhat.com
> > To: sssd-devel at lists.fedorahosted.org
> > Date: Tue, 12 Jun 2012 08:30:07 -0400
> > Subject: Re: [SSSD] [PATCH]-Bad debug message when adding services
> without explicit dns_discovery_domain
> > 
> > On Tue, 2012-06-12 at 11:11 +0200, Jakub Hrozek wrote:
> > > On Tue, Jun 12, 2012 at 11:04:28AM +0200, Jan Zelený wrote:
> > > > > https://fedorahosted.org/sssd/ticket/920
> > > > 
> > > > Nack,
> > > > please follow our coding guidelines:
> > > > 
> > > > http://www.freeipa.org/page/Coding_Style
> > > > 
> > > > In particular, I'm referring to the line length.
> > > > 
> > > > Thanks
> > > > Jan
> > > 
> > > Please also change the debug message to appropriate symbolic
> constant.
> > 
> > s/message/level/ I guess :)
> > 
 
 
Nack
 
First of all, you're still not following our style guidelines. Please do
not use tabs anywhere, keep your lines to 79 charcacters and do not have
trailing spaces in the lines.
 
Now, for the meat of the problem. The message that you're printing is
incorrect. We don't want to print the literal strings "example.com" and
"DOMAINNAME".
 
"DOMAINNAME" should be replaced by the input  string sssd_domain. 
 
Unfortunately, at this stage of evaluation we do not have the actual DNS
domain to print, so we'll have to replace "example.com" with "the
machine's DNS domain name".
 

_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120613/88267114/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Bad-debug-message-when-adding-services-without-expli.patch
Type: text/x-patch
Size: 1318 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120613/88267114/attachment.bin>


More information about the sssd-devel mailing list