Fix compliation issues (#121)

* Fix errors and warnings connected to lwip2, so that project compiles for NodeMCU v1.0
* Adapt code to change in Neopixelbus
* Add ESPAsyncUDP dependency to travis.
* Add aJson patch and commit hashes to buildscript
* Change paths in buildscript
* Fix build failure with another path using environment variable.
* Unbreak ArduinoJson
* Change WebServer dependencies and NeoPixel method as suggested by Makuna in issue #120

Co-authored-by: probonopd <[email protected]>

.travis.yml

@@ -6,12 +6,17 @@ before_install:
 install:
  - mkdir -p $HOME/arduino_ide/libraries/
  - cd $HOME/arduino_ide/libraries/
- - git clone --branch 2.1.4 https://github.com/Makuna/NeoPixelBus.git
- - git clone https://github.com/interactive-matter/aJson.git
- - git clone https://github.com/PaulStoffregen/Time.git
- - git clone https://github.com/gmag11/NtpClient.git
+ - git clone -o cf7b4a6 https://github.com/Makuna/NeoPixelBus
+ - git clone -o b05d455 https://github.com/PaulStoffregen/Time
+ - git clone -o 3c22f2f https://github.com/gmag11/NtpClient
+ - git clone -o b592ac6 https://github.com/me-no-dev/ESPAsyncUDP
+ - git clone -o ec56354 https://github.com/interactive-matter/aJson
  - sed -i -e 's|#define PRINT_BUFFER_LEN 256|#define PRINT_BUFFER_LEN 4096|g' aJson/aJSON.h
- - cd -
+ - cd aJson
+ - curl -OSLs https://gitlab.com/xarduino/lightsw/raw/2424e3a9348e7144373311af132ced38f14489ba/patch/ajson-void-flush.patch
+ - git apply ajson-void-flush.patch
+ - rm ajson-void-flush.patch
+ - cd $TRAVIS_BUILD_DIR
 script:
  - sed -i -e 's|#include "/secrets.h"|//#include "/secrets.h"|g' ESP8266HueEmulator/ESP8266HueEmulator.ino
  - sed -i -e 's|//const char|const char|g' ESP8266HueEmulator/ESP8266HueEmulator.ino

ESP8266HueEmulator/ESP8266HueEmulator.ino

@@ -14,7 +14,6 @@
 #include "LightService.h"
 
 // these are only used in LightHandler.cpp, but it seems that the IDE only scans the .ino and real libraries for dependencies
-#include <ESP8266WebServer.h>
 #include "SSDP.h"
 #include <aJSON.h> // Replace avm/pgmspace.h with pgmspace.h there and set #define PRINT_BUFFER_LEN 4096 ################# IMPORTANT
 
@@ -32,7 +31,7 @@ RgbColor black = RgbColor(0);
 
 #define pixelCount 30
 #define pixelPin 2 // Strip is attached to GPIO2 on ESP-01
-NeoPixelBus<NeoGrbFeature, NeoEsp8266Uart800KbpsMethod> strip(MAX_LIGHT_HANDLERS * NUM_PIXELS_PER_LIGHT, pixelPin);
+NeoPixelBus<NeoGrbFeature, NeoEsp8266Uart1Ws2812xMethod> strip(MAX_LIGHT_HANDLERS * NUM_PIXELS_PER_LIGHT, pixelPin);
 NeoPixelAnimator animator(MAX_LIGHT_HANDLERS * NUM_PIXELS_PER_LIGHT, NEO_MILLISECONDS); // NeoPixel animation management object
 LightServiceClass LightService;
 

ESP8266HueEmulator/LightService.cpp

@@ -134,7 +134,7 @@ class WcFnRequestHandler : public RequestHandler {
  {
  assert(_wildcard != '/');
  // verify that the URI is reasonable (only contains wildcard at the beginning/end/whole path segments
- for(int i = 0; i < _uri.length(); i++) {
+ for(size_t i = 0; i < _uri.length(); i++) {
  if (_uri[i] == _wildcard) {
  if (i != 0 && i != _uri.length() - 1 && (_uri[i-1] != '/' || _uri[i+1] != '/')) {
  assert(false);

ESP8266HueEmulator/LightService.h

@@ -1,4 +1,5 @@
 #include <Arduino.h>
+#include <ESP8266WebServer.h>
 
 enum HueColorType {
  TYPE_HUE_SAT, TYPE_CT, TYPE_XY
@@ -49,7 +50,6 @@ class LightHandler {
 #define MAX_LIGHT_HANDLERS 2
 #define COLOR_SATURATION 255.0f
 
-class ESP8266WebServer;
 class LightServiceClass {
  const char* _friendlyName;
 

ESP8266HueEmulator/SSDP.cpp

@@ -154,7 +154,6 @@ SSDPClass::~SSDPClass(){
 bool SSDPClass::begin(){
  _pending = false;
 
- uint32_t chipId = ESP.getChipId();
  String mac = WiFi.macAddress();
  mac.replace(":", "");
  mac.toLowerCase();
@@ -181,14 +180,14 @@ bool SSDPClass::begin(){
  return false;
  }
 
- if (!_server->listen(*IP_ADDR_ANY, SSDP_PORT)) {
+ if (!_server->listen(IP_ADDR_ANY, SSDP_PORT)) {
  return false;
  }
 
  _server->setMulticastInterface(ifaddr);
  _server->setMulticastTTL(_ttl);
  _server->onRx(std::bind(&SSDPClass::_update, this));
- if (!_server->connect(multicast_addr, SSDP_PORT)) {
+ if (!_server->connect(&multicast_addr, SSDP_PORT)) {
  return false;
  }
 
@@ -199,11 +198,11 @@ bool SSDPClass::begin(){
 
 void SSDPClass::_send(ssdp_method_t method){
  char buffer[1460];
- uint32_t ip = WiFi.localIP();
+ ip_addr_t ip = WiFi.localIP();
 
  int len;
  if (_messageFormatCallback) {
- len = _messageFormatCallback(this, buffer, sizeof(buffer), method != NONE, SSDP_INTERVAL, _modelName, _modelNumber, _uuid, _deviceType, ip, _port, _schemaURL);
+ len = _messageFormatCallback(this, buffer, sizeof(buffer), method != NONE, SSDP_INTERVAL, _modelName, _modelNumber, _uuid, _deviceType, ip.addr, _port, _schemaURL);
  } else {
  len = snprintf(buffer, sizeof(buffer),
  _ssdp_packet_template,
@@ -244,7 +243,7 @@ void SSDPClass::_send(ssdp_method_t method){
 }
 
 void SSDPClass::schema(WiFiClient client){
- uint32_t ip = WiFi.localIP();
+ ip_addr_t ip = WiFi.localIP();
  client.printf(_ssdp_schema_template,
  IP2STR(&ip), _port,
  _deviceType,