build.sh ... distribution failed for amd64

["K. Schreiner" <ks%ub.uni-mainz.de@localhost>]

Hi,

with current source cvsup'ed some minutes ago::

/u/NetBSD/ks-src/sbin/raidctl/raidctl.c: In function 'rf_get_device_status':
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c:446:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
   for(i=0; i < device_config.nspares; i++) {
              ^
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c:465:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
   for(i=0; i < device_config.nspares; i++) {
              ^
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c: In function 'rf_output_configuration':
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c:636:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
   for(i=0; i < device_config.nspares; i++)
              ^
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c: In function 'get_component_number':
/u/NetBSD/ks-src/sbin/raidctl/raidctl.c:699:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
   for(i=0; i < device_config.nspares; i++) {
              ^


A possible (somewhat ugly) patch:

>-1545: cvs diff -u sbin/raidctl/raidctl.c
Index: sbin/raidctl/raidctl.c
===================================================================
RCS file: /cvsroot/src/sbin/raidctl/raidctl.c,v
retrieving revision 1.69
diff -u -r1.69 raidctl.c
--- sbin/raidctl/raidctl.c      6 Feb 2019 22:38:10 -0000       1.69
+++ sbin/raidctl/raidctl.c      26 Sep 2019 10:04:37 -0000
@@ -443,7 +443,7 @@
        }
        if (device_config.nspares > 0) {
                printf("Spares:\n");
-               for(i=0; i < device_config.nspares; i++) {
+               for(i=0; (u_int)i < device_config.nspares; i++) {
                        printf("%20s: %s\n",
                               device_config.spares[i].devname,
                               device_status(device_config.spares[i].status));
@@ -462,7 +462,7 @@
        }

        if (device_config.nspares > 0) {
-               for(i=0; i < device_config.nspares; i++) {
+               for(i=0; (u_int)i < device_config.nspares; i++) {
                        if ((device_config.spares[i].status ==
                             rf_ds_optimal) ||
                            (device_config.spares[i].status ==
@@ -633,7 +633,7 @@

        if (device_config.nspares > 0) {
                printf("START spare\n");
-               for(i=0; i < device_config.nspares; i++)
+               for(i=0; (u_int)i < device_config.nspares; i++)
                        printf("%s\n", device_config.spares[i].devname);
                printf("\n");
        }
@@ -696,7 +696,7 @@
                }
        }
        if (!found) { /* maybe it's a spare? */
-               for(i=0; i < device_config.nspares; i++) {
+               for(i=0; (u_int)i < device_config.nspares; i++) {
                        if (strncmp(component_name,
                                    device_config.spares[i].devname,
                                    PATH_MAX)==0) {

Kurt