On (24/06/16 10:16), Jakub Hrozek wrote:
On Thu, Jun 23, 2016 at 04:23:12PM -0400, Simo Sorce wrote:
> On Thu, 2016-06-23 at 21:36 +0200, Jakub Hrozek wrote:
> > On Thu, Apr 21, 2016 at 09:21:17AM +0200, Jakub Hrozek wrote:
> > > Given that Lukas says "http_parser can provide pkgconfig in
future", I
> > > read his mail as a preference to keep the pkg-check test. And actually I
> > > agree, it doesn't hurt, let's keep it in.
> >
> > I wanted to push these patches:
> >
https://github.com/jhrozek/sssd/tree/secrets-review
> > but..I can't find http_parser_strict on Debian, only http-parser and the
> > patches seem to require the _strict version. I really don't know the
> > difference between the two, can we fallback to the non-strict?
>
> If it not too hard to detect if strict is present I would try to use it
> and fallback to not strict only of not available.
Thanks, I will prepare a fallback patch. I agree about the preference,
but was wondering if we should require strict or not.
I did a very brief test of
latest patches from github and I can see
few errors/warnings.
src/responder/common/responder_common.c: In function ‘activate_unix_sockets’:
src/responder/common/responder_common.c:795:13: error: too many arguments for format
[-Werror=format-extra-args]
DEBUG(SSSDBG_CRIT_FAILURE,
^~~~~
src/responder/common/responder_common.c:808:17: error: too many arguments for format
[-Werror=format-extra-args]
DEBUG(SSSDBG_CRIT_FAILURE,
^~~~~
src/responder/secrets/proxy.c: In function ‘ph_on_status’:
src/responder/secrets/proxy.c:648:29: error: passing argument 2 of ‘ph_append_string’ from
incompatible pointer type [-Werror=incompatible-pointer-types]
ph_append_string(reply, &reply->reason_phrase, at, length);
^
src/responder/secrets/proxy.c:624:13: note: expected ‘char **’ but argument is of type
‘const char **’
static void ph_append_string(TALLOC_CTX *memctx, char **dest,
^~~~~~~~~~~~~~~~
CCLD test_cert_utils
/usr/bin/ld: src/util/test_cert_utils-util.o: undefined reference to symbol
'hash_enter@(a)DHASH_0.4.3'
/usr/lib64/libdhash.so.1: error adding symbols: DSO missing from command line
Previous linking error is fixed by following diff
diff --git a/Makefile.am b/Makefile.am
index 622b0d2..7416141 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -2895,7 +2895,6 @@ test_krb5_wait_queue_LDADD = \
test_cert_utils_SOURCES = \
src/tests/cmocka/test_cert_utils.c \
- src/util/util.c \
$(NULL)
test_cert_utils_CFLAGS = \
$(AM_CFLAGS) \
CCLD responder_common-tests
/usr/bin/ld: ../../../src/responder/common/responder_common_tests-responder_common.o:
undefined reference to symbol 'sd_listen_fds@@LIBSYSTEMD_209'
/usr/lib64/libsystemd.so.0: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
CCLD negcache-tests
/usr/bin/ld: ../../../src/responder/common/negcache_tests-responder_common.o: undefined
reference to symbol 'sd_listen_fds@@LIBSYSTEMD_209'
/usr/lib64/libsystemd.so.0: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
and previous two linking error with following diff
diff --git a/src/tests/cwrap/Makefile.am b/src/tests/cwrap/Makefile.am
index 8005d99..d8a49f1 100644
--- a/src/tests/cwrap/Makefile.am
+++ b/src/tests/cwrap/Makefile.am
@@ -145,6 +145,7 @@ responder_common_tests_LDADD = \
$(CMOCKA_LIBS) \
$(SSSD_LIBS) \
$(SELINUX_LIBS) \
+ $(SYSTEMD_DAEMON_LIBS) \
$(abs_top_builddir)/libsss_util.la \
$(abs_top_builddir)/libsss_debug.la \
$(abs_top_builddir)/libsss_test_common.la \
@@ -162,6 +163,7 @@ negcache_tests_LDADD = \
$(CMOCKA_LIBS) \
$(SSSD_LIBS) \
$(SELINUX_LIBS) \
+ $(SYSTEMD_DAEMON_LIBS) \
$(abs_top_builddir)/libsss_util.la \
$(abs_top_builddir)/libsss_debug.la \
$(abs_top_builddir)/libsss_test_common.la \
LS