On Wed, 2012-10-31 at 13:18 -0400, Stephen Gallagher wrote:
fail:
- DEBUG(2, ("Failed to save group [%s]\n",
- name ? name : "Unknown"));
+ DEBUG(SSSDBG_MINOR_FAILURE,
+ ("Failed to save group [%s]: [%s]\n",
+ name ? name : "Unknown",
+ strerror(ret)));
Given you already print the strerror() in all previous debug messages,
does it make sense to repeat it here ?
(Otherwise ACK, patch looks good)
Simo.
--
Simo Sorce * Red Hat, Inc * New York