[ENBD] [PATCH 2.4.35] bug fixes for configure.in
Peter T. Breuer
ptb at inv.it.uc3m.es
Sat Apr 19 00:02:07 MDT 2008
"Also sprach Bart Van Assche:"
> There are some bugs in the configure script of enbd version 2.4.35.
> Can the patch below please be reviewed and applied ?
Sure. Great. Thanks.
>
> Thanks,
>
> Bart.
>
> --- orig/enbd-2.4.35/nbd/configure.in 2008-04-17 17:08:54.000000000 +0200
> +++ enbd-2.4.35/nbd/configure.in 2008-04-18 14:28:34.000000000 +0200
> @@ -46,7 +46,7 @@
> #include <sys/time.h>
> #include <unistd.h>
> ],
> - [main() { static struct timeval a, b; timeradd(&a,&b,&a); }],
> + [static struct timeval a, b; timeradd(&a,&b,&a)],
You are correct. I know this!
> [ac_cv_func_timeradd=yes; AC_DEFINE(HAVE_TIMERADD)],
> [ac_cv_func_timeradd=no; cat >> confdefs.h <<\EOF
> #define timeradd(a, b, result) \
> @@ -152,7 +152,7 @@
> #include <unistd.h>
> #include <time.h>
> ],
> - [main() {
> + [
Ditto. Thanks. I've done it correctly elsewhere, I'm sure!
> static struct timespec a;
> switch(1) {
> case 1:
> @@ -167,7 +167,6 @@
> #endif
> return 0;
> }
> - }
> ],
> [ac_cv_func_clock_monotonic=yes; AC_DEFINE(HAVE_CLOCK_MONOTONIC)],
> [ac_cv_func_clock_monotonic=no]
> @@ -185,6 +184,7 @@
> # if HAVE_STDLIB_H
> # include <stdlib.h>
> # endif
> +#endif
Argh. Even when I indent correctly, I don't get it right!
Thanks.
Sorry for minor vibrations in 2.4.35 at the moment even though
2.4.36pre is out. I am preparing a release of some software based on
enbd next month, and am using 2.4.35 as the "stable" platform for it,
which means that I have to fix such lacks as I find in 2.4.35 to satisfy
everyone. At the meoment the only functional change is that one has to
echo rioctl=1 to /proc/nbdinfo in order to turn on remote ioctls ...
well the nontrivial ones. That makes cross-platform server/client
combinations safe by default.
> #if HAVE_SYS_SOCKET_H
> # include <sys/socket.h>
> #endif
> _______________________________________________
> ENBD mailing list
> ENBD at lists.community.tummy.com
> http://lists.community.tummy.com/mailman/listinfo/enbd
Many thanks.
Peter
>
More information about the ENBD
mailing list