#1079168 firmware-siano: Add Appstream metainfo announcing HW support

Package:
firmware-siano
Source:
firmware-siano
Submitter:
Petter Reinholdtsen
Date:
2025-08-16 01:53:01 UTC
Severity:
normal
Tags:
#1079168#5
Date:
2024-08-20 19:46:48 UTC
From:
To:
Here is a draft patch to add Appstream metainfo XML announcing the
hardware handled by this package.  I was unable to figure out the build
system, so I could not test the patch.

Including this information in the package will ensure programs mapping
hardware to packages using Appstream information, like the isenkram
package, will know that this package is useful on machines where the
hardware is discovered.

The appstream metadata file can be checked using this command after
package build:

  appstreamcli validate-tree  --no-net --explain debian/uhd-host

I renamed the metainfo.xml file to match the ID used in the XML file, to
fix a warning reported by the validator.  I suspect a similar change
should be done for every firmware package, but decided to only fix the
package where I have a modalias proposal, to keep the patch small.

diff --git a/debian/firmware-siano.install b/debian/firmware-siano.install
new file mode 100644
index 0000000..64fb409
--- /dev/null
+++ b/debian/firmware-siano.install
@@ -0,0 +1 @@
+debian/org.debian.packages.firmware_siano.metainfo.xml usr/share/metainfo
diff --git a/debian/firmware-siano.metainfo.xml b/debian/org.debian.packages.firmware_siano.metainfo.xml
similarity index 96%
rename from debian/firmware-siano.metainfo.xml
rename to debian/org.debian.packages.firmware_siano.metainfo.xml
index e9ed916..a3a84c2 100644
--- a/debian/firmware-siano.metainfo.xml
+++ b/debian/org.debian.packages.firmware_siano.metainfo.xml
@@ -18,6 +18,6 @@
     <firmware type="runtime">sms1xxx-nova-b-dvbt-01.fw</firmware>
     <firmware type="runtime">sms1xxx-stellar-dvbt-01.fw</firmware>
     <firmware type="runtime">tdmb_nova_12mhz.inp</firmware>
-
+    <modalias>usb:v187Fp0201d*</modalias>
   </provides>
 </component>
diff --git a/debian/rules.real b/debian/rules.real
index dbb5052..68d89b2 100644
--- a/debian/rules.real
+++ b/debian/rules.real
@@ -28,7 +28,8 @@ install:
        done
 ifneq ($(FILES),)
        dh_installdirs /usr/share/metainfo
-       dh_install debian/$(PACKAGE_NAME).metainfo.xml /usr/share/metainfo
+       ( [ -e debian/$(PACKAGE_NAME).metainfo.xml ] && \
+         dh_install debian/$(PACKAGE_NAME).metainfo.xml /usr/share/metainfo ) || true
 endif
        dh_bugfiles
        dh_installchangelogs

#1079168#10
Date:
2025-01-25 05:32:23 UTC
From:
To:
Dear Debian Kernel Team, Maximilian, Ben, Salvatore and Bastian,

Is there anything I can do to help get this AppStream fix into unstable?

#1079168#15
Date:
2025-03-23 17:21:45 UTC
From:
To:
unstable?

You are adding one USB device ID, but smsusb supports 42 different
device IDs.

The correct fix for this would be to improve the scripts to also take
module dependencies into account, so that we can detect:

- smsusb supports USB device IDs 2014:0257, etc.
- smsusb depends on smsmdtv
  - smsmdtv requests firmware files tdmb_nova_12mhz.inp, etc.
- Therefore the package containing tdmb_nova_12mhz.inp, etc.
  (i.e. firmware-siano) is related to device IDs 2014:0257, etc.

Ben.