[Ovmsdev] Scripting: Duktape update to version 2.5.0

[Michael Balzer]

Craig,

sorry, missed that one, my new commit should do it.

You don't get a missing config dialog when we update the default sdkconfigs, only when a Kconfig is extended. Our default sdkconfigs have no special meaning to
the build system.

To know about changes to the existing sdkconfig entries, you need to follow the discussions here or check the diff to your sdkconfig once in a while. The SSL
change was introduced by & for the pushover component and became default.

I haven't seen that make error up to now. "make -n" will only print the commands, maybe that helps locating that.

Regards,
Michael


Am 02.01.20 um 20:13 schrieb Craig Leres:
> On 2020-01-02 00:29, Michael Balzer wrote:
> > Regarding my latest changes, you seem to build without SSL support, and I forgot some #ifdefs, will add them now.
>
> I didn't do anything to intentionally disable SSL... And I even did a "make clean" before building which I thought was a way to pick up new sdkconfig options.
> But when I diff sdkconfig with support/sdkconfig.default.hw31 I see a lot is missing including CONFIG_MG_ENABLE_SSL=y
>
> Ah, I found something. At the start of a clean build (rm -rf build && gmake -j8):
>
>     ice 509 % gmake -j8
>     gmake: gcc: Command not found
>     expr: syntax error
>     GENCONFIG
>     #
>     # configuration written to /home/ice/u0/leres/src/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/sdkconfig
>
> This seems familiar but I can't figure out where "gcc" is hardcoded. Even when I use "gmake -j8 CC=gcc9" I don't get a dialog to add missing things to my
> sdkconfig. It looks like even "gmake defconfig" is a noop.
>
> At this point I think I need a little more help.
>
> On 2020-01-02 01:52, Michael Balzer wrote:
>> I've added handling for the cases of missing SSL or Duktape support, please test.
>
> 72ae790 only fixed one of the two errors, see appended.
>
>         Craig
>
>
> ice 1251 % gmake
> Toolchain path: /home/ice/u0/leres/bin/xtensa-esp32-elf-gcc
> Toolchain version: crosstool-ng-1.22.0-80-g6c4433a5
> Compiler version: 5.2.0
> Python requirements from /home/ice/u0/leres/esp/openvehicles-xtensa-esp32-elf/requirements.txt are satisfied.
>
> App "ovms3" version: 3.2.008-59-g051be7e4-dirty
> CXX /home/ice/u0/leres/src/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/build/ovms_script/src/ovms_script.o
> /home/ice/u0/leres/src/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/components/ovms_script/src/ovms_script.cpp: In member function 'void
> DuktapeHTTPRequest::MongooseCallback(mg_connection*, int, void*)':
> /home/ice/u0/leres/src/Open-Vehicle-Monitoring-System-3/vehicle/OVMS.V3/components/ovms_script/src/ovms_script.cpp:1212:20: error: 'MG_SSL_ERROR' was not
> declared in this scope
>          if (err == MG_SSL_ERROR)
>                     ^
> gmake[1]: *** [/home/ice/u0/leres/esp/openvehicles-xtensa-esp32-elf/make/component_wrapper.mk:290: src/ovms_script.o] Error 1
> gmake: *** [/home/ice/u0/leres/esp/esp-idf/make/project.mk:552: component-ovms_script-build] Error 2

-- 
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26