search

yrzr / opnsense-tools

OPNsense images native builder for aarch64. Steps to build your own image: https://yrzr.github.io/tutorial-build-opnsense-arm64-on-a-rpi4/
https://yrzr.github.io/tags/opnsense/
BSD 2-Clause "Simplified" License
50 stars 2 forks source link

py39-duckdb missing from images #1

Closed maurice-w closed 1 year ago

maurice-w commented 1 year ago

py39-duckdb and some dependencies are missing from OPNsense-23.1.11-vm-aarch64.vmdk.xz and OPNsense-23.7.r1-vm-aarch64.vmdk.xz. This package is required for Unbound DNS reporting (Reporting: Settings: Unbound DNS reporting). It is available from your repository though and can be installed with pkg install py39-duckdb, which makes DNS reporting work.

I was just wondering if you have an idea why it's not included in the images.

Cheers & thanks for all your work! Maurice

yrzr commented 1 year ago

Hi Maurice. You have a good insight. I myself can hardly find a problem like this.

The problem is caused because the core only depends on py39-duckdb on AMD64.

https://github.com/opnsense/core/blob/77f9b43234f169cf809ec606aa06ead0de4043b2/Makefile#L131

I think it makes no sense to keep the package on amd64 only since the packages could be built on aarch64. And I have created this pull request.

yrzr commented 1 year ago

merged

maurice-w commented 1 year ago

Great, thanks! Looking forward to 23.7 images with duckdb.

maurice-w commented 1 year ago

I did build an OPNsense 23.7-aarch64 VM image from source and can confirm that Unbound DNS reporting now works out of the box. 👍