On Tue, Nov 19, 2013 at 09:44:28PM +0100, Pavel Reichl wrote: > On Tue, 2013-11-19 at 18:08 +0100, Lukas Slebodnik wrote: > > On (19/11/13 11:16), Dmitri Pal wrote: > > >On 11/19/2013 10:16 AM, Michal Židek wrote: > > >> On 11/19/2013 03:46 PM, Jakub Hrozek wrote: > > >>> On Tue, Nov 19, 2013 at 03:41:04PM +0100, Lukas Slebodnik wrote: > > >>>> ENOENT means "No such file or directory", Please add more specific > > >>>> error > > >>>> message and syslog message (sss_log) should also contain information > > >>>> about path > > >>>> to config_file > > >>> > > >>> Would it make sense to extend load_configuration with new error codes > > >>> instead of errno? Then we wouldn't have to guess in the monitor.. > > >> > > >> +1 > > >> > > >> We can safely mix errno with our specific error codes, so for now, > > >> probaly adding just one specific error for missing conf file will be > > >> enough. > > >> > > >> _______________________________________________ > > >> sssd-devel mailing list > > >> sssd-devel@lists.fedorahosted.org > > >> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel > > > > > >Do you expect this to trickle down to ding-libs? > > >Eventually? > > > > > IITC, Intention was to transform ENOENT returned from libini_config to sssd > > specific error code. Something like SSS_MISSING_CONF > > Would adding ERR_MISSING_CONF to sssd_errors (util_errors.h) be a good > idea?
That's what I proposed, yes. _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
