Switch to a preconfigured Clang for k8 and armv7
This one reuses more of Bazel's builtin features to remain up to date
more easily, and it's easier to expand support for more platforms. This
also takes care of upgrading to a newer Clang.
This will require updating to raspios bullseye on the Raspberry Pis.
This also renames armhf-debian to armv7 to match the broader Bazel
ecosystem's name for this architecture.
Change-Id: I3e2a4f4efea43e76314ca34a9293c7b4b45edd2c
Signed-off-by: Brian Silverman <bsilver16834@gmail.com>
diff --git a/y2019/image_streamer/deploy.sh b/y2019/image_streamer/deploy.sh
index 9cbf8ab..5ac9eb7 100755
--- a/y2019/image_streamer/deploy.sh
+++ b/y2019/image_streamer/deploy.sh
@@ -40,7 +40,7 @@
echo -e "\n# Building image_streamer"
(
set -x
-bazel build -c opt //y2019/image_streamer:image_streamer --config=armhf-debian
+bazel build -c opt //y2019/image_streamer:image_streamer --config=armv7
)
echo -e "\n# Copy files to ODROID"
diff --git a/y2019/image_streamer/flip_image.cc b/y2019/image_streamer/flip_image.cc
index 48a18c5..a882223 100644
--- a/y2019/image_streamer/flip_image.cc
+++ b/y2019/image_streamer/flip_image.cc
@@ -1,5 +1,15 @@
#include "flip_image.h"
+#ifdef __clang__
+// CImg has undefined behavior that Clang warns about. Just suppress the
+// warnings, somebody should evaluate these more carefully if this code is used
+// again.
+#pragma clang diagnostic ignored "-Wvarargs"
+#pragma clang diagnostic ignored "-Wnull-pointer-arithmetic"
+#pragma clang diagnostic ignored "-Wchar-subscripts"
+#pragma clang diagnostic ignored "-Wtautological-unsigned-char-zero-compare"
+#endif
+
#define cimg_display 0
#define cimg_use_jpeg
#define cimg_plugin "plugins/jpeg_buffer.h"