Skip to content

Conversation

@ChibaPet
Copy link
Contributor

@ChibaPet ChibaPet commented Nov 20, 2025

Building on Trixie I noted that dh-dkms, libcurl4-openssl-dev, and
libtirpc-dev are required.

Also, package order passed to apt is irrelevant, so I sorted the list for
easier consumption by humans.

@gmelikov
Copy link
Member

Thank you! While most of changes are valid, I don't see that we need dh-dkms in our CI to build https://github.com/openzfs/zfs/blob/1f3444f2bba42817520a8097a29d24a3b4115927/.github/workflows/scripts/qemu-3-deps-vm.sh#L44 , will try to check later but I'd be glad if there's any info about that.

@ChibaPet
Copy link
Contributor Author

Thank you! While most of changes are valid, I don't see that we need
dh-dkms in our CI to build

I don't have an opinion; I just added the minimum necessary to get the
build to complete on Debian Trixie, as I hadn't been building on Trixie
previously.

I will note that one of the packages built by the native build by default
is openzfs-zfs-dkms_2.3.4-1_all.deb, so that might explain its inclusion. I
don't strictly know that it's required to build the dkms package, but if I
remember correctly the configure wouldn't complete without it, and the
build itself wouldn't complete without the other two.

My method was to do a clean, minimal install, add the packages noted by the
docs, and then try to build.

Configure invocation:

./configure --disable-pyzfs --disable-systemd \
    --with-linux=/lib/modules/${KVERS}/source \
    --with-linux-obj=/lib/modules/${KVERS}/build

The resulting packages:

openzfs-libnvpair3_2.3.4-1_amd64.deb
openzfs-libnvpair3-dbgsym_2.3.4-1_amd64.deb
openzfs-libpam-zfs_2.3.4-1_amd64.deb
openzfs-libpam-zfs-dbgsym_2.3.4-1_amd64.deb
openzfs-libuutil3_2.3.4-1_amd64.deb
openzfs-libuutil3-dbgsym_2.3.4-1_amd64.deb
openzfs-libzfs6_2.3.4-1_amd64.deb
openzfs-libzfs6-dbgsym_2.3.4-1_amd64.deb
openzfs-libzfsbootenv1_2.3.4-1_amd64.deb
openzfs-libzfsbootenv1-dbgsym_2.3.4-1_amd64.deb
openzfs-libzfs-dev_2.3.4-1_amd64.deb
openzfs-libzpool6_2.3.4-1_amd64.deb
openzfs-libzpool6-dbgsym_2.3.4-1_amd64.deb
openzfs-python3-pyzfs_2.3.4-1_amd64.deb
openzfs-pyzfs-doc_2.3.4-1_all.deb
openzfs-zfs-dkms_2.3.4-1_all.deb
openzfs-zfs-dracut_2.3.4-1_all.deb
openzfs-zfs-initramfs_2.3.4-1_all.deb
openzfs-zfs-modules-6.12.57+deb13-amd64_2.3.4-1_amd64.deb
openzfs-zfs-test_2.3.4-1_amd64.deb
openzfs-zfs-test-dbgsym_2.3.4-1_amd64.deb
openzfs-zfsutils_2.3.4-1_amd64.deb
openzfs-zfsutils-dbgsym_2.3.4-1_amd64.deb
openzfs-zfs-zed_2.3.4-1_amd64.deb
openzfs-zfs-zed-dbgsym_2.3.4-1_amd64.deb

Copy link
Member

@gmelikov gmelikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, it may be needed for native DEB packages, got it. Thank you!

@gmelikov gmelikov merged commit 94e85fe into openzfs:master Nov 22, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants