On Wed, Jul 08, 2015 at 03:26:52PM +0200, Sumit Bose wrote: > I would suggest that you put sss_cli_command_2string() in a file on > its own similar like atomic_io.c or authtok-utils.c. And add this file > to pam_sss_la_SOURCES and libsss_debug_la_SOURCES in Makefile.am. I > leave it up to you to decide what would be a good place for this file. > The sss_client directory because the enum sss_cli_command is defined > here as well or the util directory because the main usage for it is in > the SSSD code and not in the pam_sss module.
This is really important, so much that I wonder if we should move all the files that are used by both client code and daemon code to some new directory in the SSSD tree (src/shared/ maybe) and use a different comment header in these files. _______________________________________________ sssd-devel mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
