> -----Original Message----- > From: Daniel P. Berrange [mailto:berra...@redhat.com] > Sent: Thursday, September 22, 2016 4:12 PM > To: Gonglei (Arei) > Cc: qemu-devel@nongnu.org; Wubin (H) > Subject: Re: [PATCH] qtest: fix make check complaint in crypto module > > On Thu, Sep 22, 2016 at 02:35:49PM +0800, Gonglei wrote: > > CC tests/test-crypto-tlscredsx509.o > > CC tests/crypto-tls-x509-helpers.o > > CC tests/pkix_asn1_tab.o > > tests/pkix_asn1_tab.c:7:22: warning: libtasn1.h: No such file or directory > > tests/pkix_asn1_tab.c:9: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or > ‘__attribute__’ before ‘pkix_asn1_tab’ > > make: *** [tests/pkix_asn1_tab.o] Error 1 > > > > Signed-off-by: Gonglei <arei.gong...@huawei.com> > > --- > > tests/Makefile.include | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/tests/Makefile.include b/tests/Makefile.include > > index e560ecb..ad6950f 100644 > > --- a/tests/Makefile.include > > +++ b/tests/Makefile.include > > @@ -92,8 +92,10 @@ gcov-files-test-write-threshold-y = > block/write-threshold.c > > check-unit-y += tests/test-crypto-hash$(EXESUF) > > check-unit-y += tests/test-crypto-cipher$(EXESUF) > > check-unit-y += tests/test-crypto-secret$(EXESUF) > > +ifeq ($(CONFIG_TASN1),y) > > check-unit-$(CONFIG_GNUTLS) += tests/test-crypto-tlscredsx509$(EXESUF) > > check-unit-$(CONFIG_GNUTLS) += tests/test-crypto-tlssession$(EXESUF) > > +endif > > We already check this condition in tests/crypto-tls-x509-helpers.h and > .c files should conditionalize themselves with > > #ifdef QCRYPTO_HAVE_TLS_TEST_SUPPORT > > the tests/pkix_asn1_tab.c file is just missing that check >
Yes, I see it. Regards, -Gonglei > Regards, > Daniel > -- > |: http://berrange.com -o- > http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- > http://virt-manager.org :| > |: http://autobuild.org -o- > http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- > http://live.gnome.org/gtk-vnc :|
