Skip to content

fwupd: update to latest/2.1.1 release#28914

Open
ma-neumann wants to merge 1 commit intoopenwrt:masterfrom
ma-neumann:fwupd_2.1.1
Open

fwupd: update to latest/2.1.1 release#28914
ma-neumann wants to merge 1 commit intoopenwrt:masterfrom
ma-neumann:fwupd_2.1.1

Conversation

@ma-neumann
Copy link

@ma-neumann ma-neumann commented Mar 19, 2026

📦 Package Details

Maintainer: @lvoegl

Description:


🧪 Run Testing Details

  • OpenWrt Version: 25.12.1
  • OpenWrt Target/Subtarget: x86_64
  • OpenWrt Device: APU2

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

Copy link
Contributor

@lvoegl lvoegl left a comment

Choose a reason for hiding this comment

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

We should explicitly disable -Dlibmnl and -Dsystemd. Otherwise fwupd may or may not use them depending on whether they are available in the current build.

Thanks for updating the package.

-Dlvfs=$(if $(CONFIG_FWUPD_LVFS),true,false) \
-Dman=false \
-Dmetainfo=false \
-Dudev_hotplug=true \
Copy link
Contributor

Choose a reason for hiding this comment

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

There is no udev in OpenWrt. If udev_hotplug=false, fwupd uses the kernel's netlink hotplug socket directly, which is what we want here.

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for the explanation

-Dtests=false \
-Dumockdev_tests=disabled \
-Dvalgrind=disabled \
-Dvendor_ids_dir=/usr/share/hwdata \
Copy link
Contributor

Choose a reason for hiding this comment

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

Duplicate (line 126)

Copy link
Author

Choose a reason for hiding this comment

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

Just as a note: I tried to re-order the options based on https://github.com/fwupd/fwupd/blob/main/meson_options.txt

-Dpolkit=disabled \
-Dpython=$(HOST_PYTHON3_BIN) \
-Dqubes=false \
-Dreadline=disabled \
Copy link
Author

Choose a reason for hiding this comment

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

@lvoegl Would it make sense to add readline as dependency?

Copy link
Contributor

@lvoegl lvoegl Mar 19, 2026

Choose a reason for hiding this comment

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

If it's not added fwupd provides it's own readline implementation. Adding it as a config option was possible, but disabling it wouldn't be a problem for me.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks again :)

@ma-neumann ma-neumann force-pushed the fwupd_2.1.1 branch 2 times, most recently from 9d1a2b3 to 73b859f Compare March 21, 2026 06:22
Signed-off-by: Martin Neumann <mneumann@teco.edu>
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