Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get compatible with OpenWrt ImageBuilder #1000

Open
ilario opened this issue Apr 3, 2023 · 4 comments
Open

Get compatible with OpenWrt ImageBuilder #1000

ilario opened this issue Apr 3, 2023 · 4 comments

Comments

@ilario
Copy link
Member

ilario commented Apr 3, 2023

Following the discussion from #794 (comment)

Advantages would be many.
For example, being able to compile LibreMesh images through https://firmware-selector.openwrt.org/ or allowing the community to compile images without needing the full BuildRoot and without the need to create and host our own ImageBuilder.

Related issues:
#794 (and #999 would help also for this issue)
#800 (and #991 is specifically written for this issue)

Some problematic code:

define Package/$(PKG_NAME)/config
select BUSYBOX_CUSTOM
select BUSYBOX_CONFIG_NC
select BUSYBOX_CONFIG_NC_SERVER
endef

@ilario
Copy link
Member Author

ilario commented Apr 5, 2023

Likely, also the dependency from dnsmasq-dhcpv6 is going to be problematic, as it conflicts with the dnsmasq package which is included by default.

DEPENDS:=+ebtables-nft +dnsmasq-dhcpv6 +kmod-nft-bridge +libuci-lua \

For example, when compiling with both packages selected, errors such this appear:

 * check_data_file_clashes: Package dnsmasq-dhcpv6 wants to install file openwrt/build_dir/target-arm_cortex-a7+neon-vfpv4_musl_eabi/root-ipq40xx/etc/hotplug.d/ntp/25-dnsmasqsec
	But that file is already provided by package  * dnsmasq

@ilario
Copy link
Member Author

ilario commented Apr 7, 2023

Likely, also the dependency from dnsmasq-dhcpv6 is going to be problematic, as it conflicts with the dnsmasq package which is included by default.

The solution has to be: using dnsmasq for IPv4 and odhcpd-ipv6only for IPv6. See #294

@ilario
Copy link
Member Author

ilario commented Apr 15, 2023

We should also get rid of the @(PACKAGE_iw||PACKAGE_iw-full) in the Makefiles of prometheus-node-exporter-lua-location-latlon, prometheus-node-exporter-lua-wifi-params, prometheus-node-exporter-lua-wifi-stations-extra and prometheus-node-exporter-lua-wifi-survey. But these packages are not suggested on the website, so it is not important.

@ilario
Copy link
Member Author

ilario commented May 11, 2023

An important step forward has been made in #1015.

The created x86 images does not yet boot on qemu (reported by @a-gave ) and the other images are yet to be tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant