pkg/jerryscript: set -Wno-conversion for FreeBSD

[smlng]

Contribution description

Fixes compile issue on FreeBSD because of float to double conversion
by disabling the corresponding compiler warning.

Testing procedure

Compile examples/javascript on FreeBSD for native, it fails on master with compile warning -Wfloat-conversion. This PR disables this warning similar to other exceptions already in place for jerryscript package.

Issues/PRs references

[miri64]

@smlng do you have some vagrant or puppet config for your FreeBSD setup?

[smlng]

nope, currently its just a box that I connected to our HIL Jenkins and I install stuff by demand, i.e. when a build fails I check if its a tool missing or an issue (like this one).

But I could certainly provide something to setup FreeBSD for RIOT from scratch, soonish.

[smlng]

@miri64 here build results for this PR, specifically look at examples/javascript:

• master, failing: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/18/pipeline/21
• this PR, fixed: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/19/pipeline/23

Ignore the error for dtls-echo, I already made PR on upstream:

eclipse-tinydtls/tinydtls#21

[miri64]

@miri64 here build results for this PR, specifically look at examples/javascript:

* master, failing: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/18/pipeline/21

How can I read the actual error? It just reads

+ BOARDS=native gmake -C examples/javascript buildtest

gmake: Entering directory '/usr/home/jenkins/workspace/RIOT-FreeBSD/examples/javascript'

Building for native ... failed!

gmake: *** [/usr/home/jenkins/workspace/RIOT-FreeBSD/makefiles/buildtests.inc.mk:6: buildtest] Error 1

gmake: Leaving directory '/usr/home/jenkins/workspace/RIOT-FreeBSD/examples/javascript'

script returned exit code 2

* this PR, fixed: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/19/pipeline/23

That one only shows results for tests.

Ignore the error for dtls-echo, I already made PR on upstream:

eclipse/tinydtls#21

Ignored ;-)

[smlng]

in the CI script I use make buildtest which hides most of the details. I think I need to unset BUILDTEST_MAKE_REDIRECT to show results, I'll try and report back

[smlng]

more verbose results (still running, when writing this):

• master, failing for javascript: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/20/pipeline
• this PR, fixing javascript: https://ci.riot-os.org/hil/blue/organizations/jenkins/RIOT-FreeBSD/detail/RIOT-FreeBSD/21/pipeline/23

[miri64]

With regards to the specific warning fixed the deactivated warning option is fine

In file included from /usr/home/jenkins/workspace/RIOT-FreeBSD/examples/javascript/bin/pkg/native/jerryscript/jerry-ext/arg/arg-transform-functions.c:16:

/usr/home/jenkins/workspace/RIOT-FreeBSD/examples/javascript/bin/pkg/native/jerryscript/jerry-ext/arg/arg-transform-functions.c: In function 'jerryx_arg_helper_process_double':

/usr/home/jenkins/workspace/RIOT-FreeBSD/examples/javascript/bin/pkg/native/jerryscript/jerry-ext/arg/arg-transform-functions.c:135:14: error: conversion from 'double' to 'float' may change value [-Werror=float-conversion]

  135 |   if (isnan (*d))

isnan() is for both double and float. When Updating the package we should however re-test if no other conversion errors sneak in.

[miri64]

Thanks for the extra work in providing the results @smlng!