From 983bebb6ea18965e04cfb4fc019a89c6d42b1976 Mon Sep 17 00:00:00 2001
From: Tim Huyeng <huyeng@iib.tu-darmstadt.de>
Date: Tue, 1 Nov 2022 11:20:17 +0100
Subject: [PATCH] working on board ci

---
 .github/workflows/build.yml | 11 ++++++++---
 build.py                    | 10 +++++++++-
 platformio_orig.ini         | 13 ++++++++++---
 src/main.cpp                |  2 +-
 4 files changed, 28 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 53185297..07ac8f98 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -5,7 +5,7 @@ on:
   push:
     branches:
         - master
-        - development
+  workflow_dispatch:
 
 jobs:
   build:
@@ -23,6 +23,7 @@ jobs:
         - os: windows-latest
           path: ~\AppData\Local\pip\Cache
           platformio-path: ~\AppData\Local\platformio\Cache
+        board: [heltecv2.h,heltecv21.h,ttgov21new.h]
     runs-on: ${{ matrix.os }}
 
     steps:
@@ -53,5 +54,9 @@ jobs:
           cp src/loraconf_sample.h src/loraconf.h
           cp src/ota_sample.conf src/ota.conf
           cp src/paxcounter_orig.conf src/paxcounter.conf
-      - name: Run PlatformIO
-        run: pio run
+      - name: Clean
+        run: pio run -t clean -e ci
+      - name: Run PlatformIO Test
+        env:
+          CI_HALFILE: ${{ matrix.board }}
+        run: pio run -e ci
diff --git a/build.py b/build.py
index 94b53ff9..1ab79cbd 100644
--- a/build.py
+++ b/build.py
@@ -27,7 +27,15 @@ haldir = os.path.join (srcdir, "hal")
 
 # check if hal file is present in source directory
 halconfig = config.get("board", "halfile")
-halconfigfile = os.path.join (haldir, halconfig)
+
+# check if hal file is not set by ENV CI_HALFILE
+if not os.getenv("CI_HALFILE"):
+    halconfigfile = os.path.join (haldir, halconfig)
+else:
+    # if set, use ENV CI_HALFILE
+    print("CI_HALFILE ENV FOUND")
+    halconfigfile = os.path.join (haldir,os.getenv("CI_HALFILE"))
+
 if os.path.isfile(halconfigfile) and os.access(halconfigfile, os.R_OK):
     print("Parsing hardware configuration from " + halconfigfile)
 else:
diff --git a/platformio_orig.ini b/platformio_orig.ini
index 3eb41b62..e2005bc5 100644
--- a/platformio_orig.ini
+++ b/platformio_orig.ini
@@ -82,7 +82,7 @@ lib_deps_basic =
     https://github.com/SukkoPera/Arduino-Rokkit-Hash.git
     bblanchon/ArduinoJson @ ^6
     makuna/RTC @ ^2.3.5
-    spacehuhn/SimpleButton 
+    spacehuhn/SimpleButton
     lewisxhe/XPowersLib @ ^0.1.4
     256dpi/MQTT @ ^2.5.0
 lib_deps_all =
@@ -94,7 +94,6 @@ lib_deps_all =
     ${common.lib_deps_sensors}
     ${common.lib_deps_ledmatrix}
 build_flags_basic =
-    -include "src/hal/${board.halfile}"
     -include "src/paxcounter.conf"
     '-DCORE_DEBUG_LEVEL=${common.debug_level}'
     '-DLOG_LOCAL_LEVEL=${common.debug_level}'
@@ -115,7 +114,9 @@ upload_speed = ${common.upload_speed}
 ;upload_port = COM12
 platform = ${common.platform_espressif32}
 lib_deps = ${common.lib_deps_all}
-build_flags = ${common.build_flags_all}
+build_flags =
+    -include "src/hal/${board.halfile}"
+    ${common.build_flags_all}
 upload_protocol = ${common.upload_protocol}
 extra_scripts = ${common.extra_scripts}
 monitor_speed = ${common.monitor_speed}
@@ -131,3 +132,9 @@ upload_protocol = esptool
 upload_protocol = esptool
 platform = https://github.com/platformio/platform-espressif32.git#feature/arduino-upstream
 platform_packages = framework-arduinoespressif32 @ https://github.com/espressif/arduino-esp32.git
+
+[env:ci]
+build_flags =
+    -include "src/hal/${sysenv.CI_HALFILE}" ; set by CI
+    ${common.build_flags_all}
+upload_protocol = esptool
\ No newline at end of file
diff --git a/src/main.cpp b/src/main.cpp
index 380120ee..00a8bf2e 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -276,7 +276,7 @@ void setup() {
   libpax_default_config(&configuration);
 
   // configure WIFI sniffing
-  strcpy(configuration.wifi_my_country, WIFI_MY_COUNTRY);
+  strcpy(configuration.wifi_my_country_str, WIFI_MY_COUNTRY);
   configuration.wificounter = cfg.wifiscan;
   configuration.wifi_channel_map = WIFI_CHANNEL_ALL;
   configuration.wifi_channel_switch_interval = cfg.wifichancycle;