Head/amd64 @ r281689 built OK, but my i386 build failed:

...
--- usr.sbin.all__D ---
--- all_subdir_bluetooth ---
/usr/src/usr.sbin/bluetooth/hccontrol/le.c:236:4: error: format specifies type 
'unsigned long' but the argument has type 'u_int64_t' (aka 'unsigned long 
long') [-Werror,-Wformat]
                        rp.le_features);
                        ^~~~~~~~~~~~~~
/usr/src/usr.sbin/bluetooth/hccontrol/le.c:253:49: error: format specifies type 
'unsigned long' but the argument has type 'u_int64_t' (aka 'unsigned long 
long') [-Werror,-Wformat]
        printf("LE_STATUS: %d %d %lx\n", e, rp.status, rp.le_status);
                                 ~~~                   ^~~~~~~~~~~~
                                 %llx
2 errors generated.
....


The attached patch allows i386 to build, and I then tested to verify
that amd64 still built.  (I don't have Bluetooth devices, so I
cannot test its operation.)

Peace,
david
-- 
David H. Wolfskill                              da...@catwhisker.org
Those who murder in the name of God or prophet are blasphemous cowards.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.
Index: usr.sbin/bluetooth/hccontrol/le.c
===================================================================
--- usr.sbin/bluetooth/hccontrol/le.c	(revision 281689)
+++ usr.sbin/bluetooth/hccontrol/le.c	(working copy)
@@ -232,7 +232,7 @@
 			NG_HCI_OCF_LE_READ_LOCAL_SUPPORTED_FEATURES), 
 			(void *)&rp, &n);
 
-	printf("LOCAL SUPPORTED: %d %d %lu\n", e, rp.status,
+	printf("LOCAL SUPPORTED: %d %d %ju\n", e, rp.status,
 			rp.le_features);
 
 	return 0;
@@ -250,7 +250,7 @@
 					NG_HCI_OCF_LE_READ_SUPPORTED_STATUS),
 			       		(void *)&rp, &n);
 
-	printf("LE_STATUS: %d %d %lx\n", e, rp.status, rp.le_status);
+	printf("LE_STATUS: %d %d %jx\n", e, rp.status, rp.le_status);
 
 	return 0;
 }

Attachment: pgpX67JRiy9Zl.pgp
Description: PGP signature

Reply via email to