Hi Chris,
I do not think this change is unrelated.
The pre-fix code used "strerror" the new one might handle sssd specific error codes and therefore used "sss_strerror" which is provided by util/util_errors[.h/.c] and therefore has to be linked into it.
So adding that is required and IMHO fine to be done in that patch.
util/signal.c on the other hand only provides BlockSignals & CatchSignal which isn't used in any of the gpo_child_* code so it was removed while touching the list of files to link.
Is that sufficient to let is pass on a second try, or did I miss some underlying problem?
Hi Chris,
I do not think this change is unrelated.
The pre-fix code used "strerror" the new one might handle sssd specific error codes and therefore used "sss_strerror" which is provided by util/util_ errors[ .h/.c] and therefore has to be linked into it.
So adding that is required and IMHO fine to be done in that patch.
util/signal.c on the other hand only provides BlockSignals & CatchSignal which isn't used in any of the gpo_child_* code so it was removed while touching the list of files to link.
Is that sufficient to let is pass on a second try, or did I miss some underlying problem?