]> git.proxmox.com Git - mirror_qemu.git/commitdiff
configure: use valid args testing sem_timedwait
authorDaniel P. Berrangé <berrange@redhat.com>
Mon, 17 Jun 2019 11:41:14 +0000 (12:41 +0100)
committerLaurent Vivier <laurent@vivier.eu>
Wed, 26 Jun 2019 16:27:07 +0000 (18:27 +0200)
The sem_timedwait function has been annotated as requiring
non-null args in latest header files from GCC snapshot
representing the future 2.30 release.

This causes configure to fail when -Werror is used:

config-temp/qemu-conf.c: In function ‘main’:
config-temp/qemu-conf.c:2:25: error: null argument where non-null required (argument 1) [-Werror=nonnull]
    2 | int main(void) { return sem_timedwait(0, 0); }
      |                         ^~~~~~~~~~~~~
config-temp/qemu-conf.c:2:25: error: null argument where non-null required (argument 2) [-Werror=nonnull]

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20190617114114.24897-1-berrange@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
configure

index cf3d9d30bfce571ee9c9d1f39552d3f294e1531c..f9463310109481fda964c0ce1c3a958fb75a9d1d 100755 (executable)
--- a/configure
+++ b/configure
@@ -5144,7 +5144,7 @@ fi
 sem_timedwait=no
 cat > $TMPC << EOF
 #include <semaphore.h>
-int main(void) { return sem_timedwait(0, 0); }
+int main(void) { sem_t s; struct timespec t = {0}; return sem_timedwait(&s, &t); }
 EOF
 if compile_prog "" "" ; then
     sem_timedwait=yes