From bd14da4804babdc79ed6255dcb838d2227e69183 Mon Sep 17 00:00:00 2001
From: Robert Clipsham <robert@octarineparrot.com>
Date: Fri, 8 Apr 2022 16:17:31 +0100
Subject: [PATCH] Ensure warnings cause build failures in CI

---
 .github/workflows/ci.yml | 1 +
 build.sh                 | 8 ++++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 0b7bf4df..684c8e4f 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -10,6 +10,7 @@ env:
   CARGO_TERM_COLOR: always
   PNET_FEATURES: travis pcap serde
   PNET_MACROS_FEATURES: travis
+  VERBOSE: 1
 
 jobs:
   build-ubuntu:
diff --git a/build.sh b/build.sh
index 0332d2c2..a61cc893 100644
--- a/build.sh
+++ b/build.sh
@@ -23,13 +23,13 @@ MACROS_WITH_SYNTEX=0
 if [[ -n "$PNET_FEATURES" ]]; then
     PNET_CARGO_FLAGS="--no-default-features --features \"$PNET_FEATURES\""
 else
-    PNET_CARGO_FLAGS=
+    PNET_CARGO_FLAGS=""
 fi
 
 if [[ -n "$PNET_MACROS_FEATURES" ]]; then
     PNET_MACROS_CARGO_FLAGS="--no-default-features --features \"$PNET_MACROS_FEATURES\""
 else
-    PNET_MACROS_CARGO_FLAGS=
+    PNET_MACROS_CARGO_FLAGS=""
 fi
 
 # FIXME Need to get interface differently on Windows
@@ -38,6 +38,8 @@ IPROUTE2=$(which ip)
 
 echo $PNET_MACROS_FEATURES | grep -q with-syntex && MACROS_WITH_SYNTEX=1
 
+PNET_TEST_IFACE=""
+
 if [[ -x "$IFCONFIG" ]]; then
     PNET_TEST_IFACE=$($IFCONFIG | egrep 'UP| active' | \
                       perl -pe '/^[A-z0-9]+:([^\n]|\n\t)*status: active/' | \
@@ -60,6 +62,8 @@ if [[ -z "$PNET_TEST_IFACE" && "$SYSTEM" = "Linux" ]]; then
     done
 fi
 
+set -euo pipefail
+
 # FIXME Need to link libraries properly on Windows
 build() {
     if [[ -x "$CARGO" ]]; then