Build isn't working here. git submodule update from the esp-idf directory just returned without making any apparent changes; did things get pushed to remote/master?
I'm getting a build error. This is after a make clean. What did I miss?
/home/greg/greg/ovms/Open-Vehicle-Monitoring-System-3-master/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/components/ovms_webserver/src/ovms_webserver.cpp: In member function 'user_session* OvmsWebServer::GetSession(http_message*)':
/home/greg/greg/ovms/Open-Vehicle-Monitoring-System-3-master/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/components/ovms_webserver/src/ovms_webserver.cpp:584:84: error: 'mg_http_parse_header2' was not declared in this scope
mg_http_parse_header2(cookie_header, SESSION_COOKIE_NAME, &ssid, sizeof(ssid_buf));
^
make[1]: *** [/home/greg/esp/esp-idf/make/component_wrapper.mk:274: src/ovms_webserver.o] Error 1
make: *** [/home/greg/esp/esp-idf/make/project.mk:450: component-ovms_webserver-build] Error 2
greg@linux-0rpb:~/greg/ovms/Open-Vehicle-Monitoring-System-3-master/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3>
Greg
Michael Balzer wrote:
Just pushed the update, so you'll need to do… git submodule update …after your next pull. I've excluded the dev branch changes for now as mg_broadcast() isn't usable yet for us anyway, at least not for my purpose. Everything OK in my environment, but please test yours. Regards, Michael Am 11.03.2018 um 03:34 schrieb Mark Webb-Johnson:Fine by me. Just let us know when done so we know to submodule update.On 11 Mar 2018, at 2:34 AM, Michael Balzer <dexter@expeedo.de> wrote: Mark, Steve, I'm about to update our mongoose submodule to the latest changes _including_ the latest additions to the "dev" branch. That's because I want to use the mg_broadcast() function, which doesn't work on the ESP32 until this fix from the dev branch: https://github.com/cesanta/mongoose/commit/780077d1187ff632bdcbf33d65207a084027aba9 Quickly found that in the mongoose issues once I knew what to look for… ;) My local checkout builds & runs. I've checked through the commits, there is no API change (except a minor one on HTTP header parsing). None of the commits looks like it could break something and there are some clear fixes included, so I think we can just merge the dev branch. I'll do some general tests before pushing the change, if you'd like me to wait with the update or do some specific tests, tell me. Regards, Michael -- Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal Fon 02333 / 833 5735 * Handy 0176 / 206 989 26 _______________________________________________ OvmsDev mailing list OvmsDev@lists.teslaclub.hk http://lists.teslaclub.hk/mailman/listinfo/ovmsdev_______________________________________________ OvmsDev mailing list OvmsDev@lists.teslaclub.hk http://lists.teslaclub.hk/mailman/listinfo/ovmsdev
_______________________________________________ OvmsDev mailing list OvmsDev@lists.teslaclub.hk http://lists.teslaclub.hk/mailman/listinfo/ovmsdev
-- Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal Fon 02333 / 833 5735 * Handy 0176 / 206 989 26