[SSSD] [PATCH] SYSDB: Add a sysdb utility call sysdb_attrs_add_lower_case_string

Pavel Reichl preichl at redhat.com
Thu Jun 5 14:44:34 UTC 2014


On Wed, 2014-06-04 at 17:34 +0200, Sumit Bose wrote:
> On Wed, Jun 04, 2014 at 04:40:41PM +0200, Pavel Reichl wrote:
> > Hello,
> > 
> > while working on
> > https://fedorahosted.org/sssd/ticket/2056
> > I noticed this already applied patch
> > http://fedorapeople.org/cgit/sbose/public_git/sssd.git/commit/?id=48eb2ca157f7cdc625d0eacdcdc085a3fe1a0fc8
> > 
> > What's the best course of action now?
> > 1) Simply close the ticket?
> > 
> > 2) Add sysdb_attrs_add_lower_case_string and make
> > sysdb_attrs_add_lc_name_alias to utilize it? (attached patch does so)
> 
> I would prefer 2.

Great, so I don't have to amend attached patch.
PR

> bye,
> Sumit
> 
> > 
> > 3) replace sysdb_attrs_add_lc_name_alias by 
> > sysdb_attrs_add_lower_case_string
> > 
> > Thanks,
> > 
> > PR
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel





More information about the sssd-devel mailing list