oonef-dlep-radio: error: initialization of 'int' from 'void *' makes integer from pointer without a cast [-Wint-conversion]

[BKPepe]

Maintainer: @HRogge
OpenWrt daily snapshot, mvebu/aarch64-cortex-a53

Log:
/builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:96:108: error: initialization of 'int' from 'void *' makes integer from pointer without a cast [-Wint-conversion] 96 | static struct msghdr _netlink_rcv_msg = { &_netlink_nladdr, sizeof(_netlink_nladdr), &_netlink_rcv_iov, 1, NULL, 0, 0 }; | ^~~~ /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:96:108: note: (near initialization for '_netlink_rcv_msg.__pad1') /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:96:15: warning: missing initializer for field '__pad2' of 'struct msghdr' [-Wmissing-field-initializers] 96 | static struct msghdr _netlink_rcv_msg = { &_netlink_nladdr, sizeof(_netlink_nladdr), &_netlink_rcv_iov, 1, NULL, 0, 0 }; | ^~~~~~ In file included from /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:47: /builder/shared-workdir/build/sdk/staging_dir/toolchain-aarch64_cortex-a53_gcc-14.2.0_musl/include/sys/socket.h:39:13: note: '__pad2' declared here 39 | int __pad2; | ^~~~~~ /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:106:33: error: initialization of 'int' from 'void *' makes integer from pointer without a cast [-Wint-conversion] 106 | ARRAYSIZE(_netlink_send_iov), NULL, 0, 0 }; | ^~~~ /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:106:33: note: (near initialization for '_netlink_send_msg.__pad1') /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/src/base/os_linux/os_system_linux.c:106:3: warning: missing initializer for field '__pad2' of 'struct msghdr' [-Wmissing-field-initializers] 106 | ARRAYSIZE(_netlink_send_iov), NULL, 0, 0 }; | ^~~~~~~~~ /builder/shared-workdir/build/sdk/staging_dir/toolchain-aarch64_cortex-a53_gcc-14.2.0_musl/include/sys/socket.h:39:13: note: '__pad2' declared here 39 | int __pad2; | ^~~~~~ ninja: build stopped: subcommand failed. make[3]: *** [Makefile:70: /builder/shared-workdir/build/sdk/build_dir/target-aarch64_cortex-a53_musl/oonf-dlep-radio-2022.08.25~fb15d54d/.built] Error 1 time: package/feeds/routing/oonf-dlep-radio/compile#20.62#7.17#29.45

Log is available here:
https://downloads.openwrt.org/snapshots/faillogs/aarch64_cortex-a53/routing/oonf-dlep-radio/compile.txt

[BKPepe]

Same also applies to oonf-dlep-proxy package.

[XDjackieXD]

For the time being dlep-radio and dlep-proxy can be disabled.
In it's current state these are useless anyways for multiple reasons.
To be specific:

• OLSRD2 does not have a way to match L2 links if the Router MAC and wireless interface MAC do not match
• DLEP updates are completely broken right now (only updates once on connect of the DLEP client)

Proposing a fix for those two issues is on my todo list but I won't have time to work on this for at least a few more months so for now disabling them shouldn't be an issue for anyone.

[BKPepe]

Since you also commented on this here (#1144 (comment)), I need to respond here as well; otherwise, it might appear that I do not care.

You yourself stated that dlep-radio and dlep-proxy packages can be disabled. Yet, you are responding to my comment from May 23, and now it is November 23 — more than half a year during which the OONF packages have failed to compile and have been unavailable in OpenWrt snapshots.

Truly, no one else has examined these packages during this lengthy period. According to you, they are "useless," so why were they included at all, and why weren’t they disabled sooner? It does not help that you have this on your to-do list but cannot look into it now and only plan to address it months later. First and foremost, the package needs to compile and be available.

At this point, however, efforts to salvage these packages are counterproductive because they have been removed — see #1145 — along with other reasons mentioned there and also cross-referenced here for clarity, so duplicating them elsewhere is unnecessary.