[Ovmsdev] Mongoose update & dev merge

[Michael Balzer]

Greg,

you need to do the git submodule update in the OVMS directory. Mongoose is a submodule of our tree, not the idf.

Regards,
Michael


Am 16.03.2018 um 22:36 schrieb Greg D.:
> 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 <http://component_wrapper.mk:274>: src/ovms_webserver.o] Error 1
> make: *** [/home/greg/esp/esp-idf/make/project.mk:450 <http://project.mk:450>: component-ovms_webserver-build] Error 2
> greg at 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 at 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 at lists.teslaclub.hk
>>>> http://lists.teslaclub.hk/mailman/listinfo/ovmsdev
>>> _______________________________________________
>>> OvmsDev mailing list
>>> OvmsDev at lists.teslaclub.hk
>>> http://lists.teslaclub.hk/mailman/listinfo/ovmsdev
>
>
>
> _______________________________________________
> OvmsDev mailing list
> OvmsDev at 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openvehicles.com/pipermail/ovmsdev/attachments/20180316/79fb8d4d/attachment.htm>