[ Re-adding submitter… ]
Philip Hands <phil@hands.com> (2025-09-30):
> alexander barakin <alex@barak.in> writes:
>
> > list_deb_firmware () {
> > udpkg -c "$1" \
> > - | grep '^\./lib/firmware/' \
> > - | sed -e 's!^\./lib/firmware/!!' \
> > + | grep '^\.\(/usr\)\?/lib/firmware/' \
> > + | sed -e 's!^\.\(/usr\)\?/lib/firmware/!!' \
> > | grep -v '^$'
> > }
>
> Here's a variation on that which avoids the `grep | sed | grep` pipe,
> and takes advantage of the fact that we have support for -E in d-i.
>
> https://salsa.debian.org/installer-team/hw-detect/-/merge_requests/14
If we go for that, maybe include some comments to explain what that's
doing, so that non-sed experts can grok what's happening?
> BTW I notice mentions of /lib/firmware in a few other places:
>
> =-=-=-=
> packages/hw-detect/check-missing-firmware.sh
> 184: if [ ! -e /lib/firmware/$fwfile ] ; then
Probably can be updated, I'm not expecting changes because of the
symlink.
> 217: target="/lib/firmware${sdir:+/$sdir}"
Ditto.
> packages/busybox/util-linux/mdev.c
> 64://config: /lib/firmware/ and if it exists, send it to the kernel for
> 900: * - userspace checks /lib/firmware/$FIRMWARE
> 903: * - userspace copies /lib/firmware/$FIRMWARE into /sys/$DEVPATH/data
> 912: /* check for /lib/firmware/$FIRMWARE */
> 914: if (chdir("/lib/firmware") == 0)
That's a topic for busybox upstream, no opinion.
> installation-guide/en/using-d-i/loading-firmware.xml
> 31:copied to the correct location (<filename>/lib/firmware</filename>) and
Would probably make sense to update.
Unless I missed something, none of those need fixing in stable.
Cheers,
--
Cyril Brulebois (kibi@debian.org) <https://debamax.com/>
D-I release manager -- Release team member -- Freelance Consultant
Attachment:
signature.asc
Description: PGP signature