- move source file according with platformio standard directory structure

- remove lib_deps from platformio.ini since they are already in libs folder

.gitignore

@@ -3,3 +3,6 @@ build
 cmake-build-debug
 cmake-build-release
 cmake-build-release-s3
+.pio
+.vscode
+.project

CMakeLists.txt

@@ -42,34 +42,34 @@ include_directories(${PROJECT_NAME}
 )
 
 set(SRCFILES
-        Config.h
+        src/Config.h
-        NukiDeviceId.cpp
+        src/NukiDeviceId.cpp
-        CharBuffer.cpp
+        src/CharBuffer.cpp
-        Network.cpp
+        src/Network.cpp
-        MqttReceiver.h
+        src/MqttReceiver.h
-        NetworkLock.cpp
+        src/NetworkLock.cpp
-        NetworkOpener.cpp
+        src/NetworkOpener.cpp
-        networkDevices/NetworkDevice.h
+        src/networkDevices/NetworkDevice.h
-        networkDevices/NetworkDevice.cpp
+        src/networkDevices/NetworkDevice.cpp
-        networkDevices/WifiDevice.cpp
+        src/networkDevices/WifiDevice.cpp
-        networkDevices/W5500Device.cpp
+        src/networkDevices/W5500Device.cpp
-        networkDevices/EthLan8720Device.cpp
+        src/networkDevices/EthLan8720Device.cpp
-        networkDevices/ClientSyncW5500.cpp
+        src/networkDevices/ClientSyncW5500.cpp
-        networkDevices/espMqttClientW5500.cpp
+        src/networkDevices/espMqttClientW5500.cpp
-        networkDevices/IPConfiguration.cpp
+        src/networkDevices/IPConfiguration.cpp
-        LockActionResult.h
+        src/LockActionResult.h
-        QueryCommand.h
+        src/QueryCommand.h
-        NukiWrapper.cpp
+        src/NukiWrapper.cpp
-        NukiOpenerWrapper.cpp
+        src/NukiOpenerWrapper.cpp
-        MqttTopics.h
+        src/MqttTopics.h
-        Ota.cpp
+        src/Ota.cpp
-        WebCfgServerConstants.h
+        src/WebCfgServerConstants.h
-        WebCfgServer.cpp
+        src/WebCfgServer.cpp
-        PresenceDetection.cpp
+        src/PresenceDetection.cpp
-        PreferencesKeys.h
+        src/PreferencesKeys.h
-        Gpio.cpp
+        src/Gpio.cpp
-        Logger.cpp
+        src/Logger.cpp
-        RestartReason.h
+        src/RestartReason.h
 #        include/RTOS.h
         lib/WiFiManager/WiFiManager.cpp
         lib/WiFiManager/wm_consts_en.h
@@ -114,7 +114,7 @@ file(GLOB_RECURSE SRCFILESREC
 )
 
 add_executable(${PROJECT_NAME}
-        main.cpp
+        src/main.cpp
         ${SRCFILES}
         ${SRCFILESREC}
         )

Docker/Dockerfile

@@ -14,14 +14,10 @@ RUN cd ~/arduino* && \
 
 RUN git clone --recurse-submodules https://github.com/technyon/Arduino-CMake-Toolchain.git ~/Arduino-CMake-Toolchain
 
-COPY icon /usr/src/nuki_hub/icon
+COPY src /usr/src/nuki_hub/src
-COPY include /usr/src/nuki_hub/include
 COPY lib /usr/src/nuki_hub/lib
-COPY networkDevices /usr/src/nuki_hub/networkDevices
+COPY include /usr/src/nuki_hub/include
 COPY CMakeLists.txt /usr/src/nuki_hub
-COPY index.html /usr/src/nuki_hub
-COPY *.h /usr/src/nuki_hub/
-COPY *.cpp /usr/src/nuki_hub/
 
 RUN mkdir -p /usr/src/nuki_hub/build
 

include/README

@@ -0,0 +1,39 @@
+
+This directory is intended for project header files.
+
+A header file is a file containing C declarations and macro definitions
+to be shared between several project source files. You request the use of a
+header file in your project source file (C, C++, etc) located in `src` folder
+by including it, with the C preprocessing directive `#include'.
+
+```src/main.c
+
+#include "header.h"
+
+int main (void)
+{
+ ...
+}
+```
+
+Including a header file produces the same results as copying the header file
+into each source file that needs it. Such copying would be time-consuming
+and error-prone. With a header file, the related declarations appear
+in only one place. If they need to be changed, they can be changed in one
+place, and programs that include the header file will automatically use the
+new version when next recompiled. The header file eliminates the labor of
+finding and changing all the copies as well as the risk that a failure to
+find one copy will result in inconsistencies within a program.
+
+In C, the usual convention is to give header files names that end with `.h'.
+It is most portable to use only letters, digits, dashes, and underscores in
+header file names, and at most one dot.
+
+Read more about using header files in official GCC documentation:
+
+* Include Syntax
+* Include Operation
+* Once-Only Headers
+* Computed Includes
+
+https://gcc.gnu.org/onlinedocs/cpp/Header-Files.html

lib/README

@@ -0,0 +1,46 @@
+
+This directory is intended for project specific (private) libraries.
+PlatformIO will compile them to static libraries and link into executable file.
+
+The source code of each library should be placed in an own separate directory
+("lib/your_library_name/[here are source files]").
+
+For example, see a structure of the following two libraries `Foo` and `Bar`:
+
+|--lib
+|  |
+|  |--Bar
+|  |  |--docs
+|  |  |--examples
+|  |  |--src
+|  |     |- Bar.c
+|  |     |- Bar.h
+|  |  |- library.json (optional, custom build options, etc) https://docs.platformio.org/page/librarymanager/config.html
+|  |
+|  |--Foo
+|  |  |- Foo.c
+|  |  |- Foo.h
+|  |
+|  |- README --> THIS FILE
+|
+|- platformio.ini
+|--src
+   |- main.c
+
+and a contents of `src/main.c`:
+```
+#include <Foo.h>
+#include <Bar.h>
+
+int main (void)
+{
+  ...
+}
+
+```
+
+PlatformIO Library Dependency Finder will find automatically dependent
+libraries scanning project source files.
+
+More information about PlatformIO Library Dependency Finder
+- https://docs.platformio.org/page/librarymanager/ldf.html

platformio.ini

@@ -10,7 +10,6 @@
 
 [platformio]
 default_envs = esp32dev
-src_dir = .
 
 [env]
 platform = espressif32
@@ -34,18 +33,11 @@ build_flags =
 ;	-DDEBUG_NUKI_HEX_DATA
 ;	-DDEBUG_NUKI_READABLE_DATA
 lib_deps = 
-	https://github.com/technyon/nuki_ble.git
+
-	bertmelis/espMqttClient@1.6.0
-	bblanchon/ArduinoJson@7.0.4
 monitor_speed = 115200
 monitor_filters = 
 	esp32_exception_decoder
 	time
-build_src_filter = 
-	-<*>
-	+<*.cpp>
-	+<*.h>
-	+<networkDevices>
 
 [env:esp32dev]
 board = esp32dev

src/Gpio.cpp

@@ -5,7 +5,7 @@
 #include "Logger.h"
 #include "PreferencesKeys.h"
 #include "RestartReason.h"
-#include "lib/gpio2go/src/Gpio2Go.h"
+#include "../lib/gpio2go/src/Gpio2Go.h"
 
 Gpio* Gpio::_inst = nullptr;
 unsigned long Gpio::_debounceTs = 0;

test/README

@@ -0,0 +1,11 @@
+
+This directory is intended for PlatformIO Test Runner and project tests.
+
+Unit Testing is a software testing method by which individual units of
+source code, sets of one or more MCU program modules together with associated
+control data, usage procedures, and operating procedures, are tested to
+determine whether they are fit for use. Unit testing finds problems early
+in the development cycle.
+
+More information about PlatformIO Unit Testing:
+- https://docs.platformio.org/en/latest/advanced/unit-testing/index.html