search

thelsing / knx

knx stack (TP, IP and RF) for arduino and linux, Can be configured with ETS
GNU General Public License v3.0
257 stars 91 forks source link

Fix: build for cemi_server on platform without RF support will failed #283

Closed tonylu00 closed 3 weeks ago

tonylu00 commented 3 weeks ago

I want to use "knx-usb" example on ESP32 platform. And I noticed a problem. Build for example "knx-usb" will failed if define 'USE_CEMI_SERVER' without 'USE_RF', such as ESP32. This commit only solve the build error, by add preprocessor to check if USE_RF is defined. Tested on following env:

[env:adafruit_qtpy_esp32s3_n4r2] platform = espressif32 board = adafruit_qtpy_esp32s3_n4r2 framework = arduino lib_extra_dirs = ../../../ extra_scripts = pre:custom_hwids.py board_build.usb_product = "KNX RF - USB Interface" lib_deps = knx adafruit/Adafruit TinyUSB Library@^3.2.0 build_flags = -DUSE_USB -DUSE_TINYUSB -Wno-unknown-pragmas -DMASK_VERSION=0x07B0

thelsing commented 3 weeks ago

Thank you for your contribution.