netdev - Re: [mail@...iiwolf.com: Re: ethtool: Incorrect component type in AppStream metainfo causes issues and possible breakages]

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <i6bv6u7bepyqueeagzcpkzonicgupqk47wijpynz24mylvumzq@td444peudd2u>
Date: Fri, 25 Apr 2025 18:46:31 +0200
From: Michal Kubecek <mkubecek@...e.cz>
To: Salvatore Bonaccorso <carnil@...ian.org>
Cc: Petter Reinholdtsen <pere@...ian.org>, netdev@...r.kernel.org, 
	Robert Scheck <fedora@...ert-scheck.de>, AsciiWolf <mail@...iiwolf.com>
Subject: Re: [mail@...iiwolf.com: Re: ethtool: Incorrect component type in
 AppStream metainfo causes issues and possible breakages]

On Thu, Apr 24, 2025 at 03:29:15PM +0200, Salvatore Bonaccorso wrote:
> Hi Michal,
> 
> On Fri, Apr 11, 2025 at 10:48:44PM +0200, AsciiWolf wrote:
> > Please note that as pointed out in my previous emails, the binary
> > provides seems to be required for console-application component type.
> > 
> > Daniel
> > 
> > pá 11. 4. 2025 v 22:18 odesílatel AsciiWolf <mail@...iiwolf.com> napsal:
> > 
> > >
> > > Here is the proposed fix. It is validated using appstreamcli validate
> > > and should work without issues.
> > >
> > > --- org.kernel.software.network.ethtool.metainfo.xml_orig
> > > 2025-03-31 00:46:03.000000000 +0200
> > > +++ org.kernel.software.network.ethtool.metainfo.xml    2025-04-11
> > > 22:14:11.634355310 +0200
> > > @@ -1,5 +1,5 @@
> > >  <?xml version="1.0" encoding="UTF-8"?>
> > > -<component type="desktop">
> > > +<component type="console-application">
> > >    <id>org.kernel.software.network.ethtool</id>
> > >    <metadata_license>MIT</metadata_license>
> > >    <name>ethtool</name>
> > > @@ -11,6 +11,7 @@
> > >    </description>
> > >    <url type="homepage">https://www.kernel.org/pub/software/network/ethtool/</url>
> > >    <provides>
> > > +    <binary>ethtool</binary>
> > >      <modalias>pci:v*d*sv*sd*bc02sc80i*</modalias>
> > >    </provides>
> > >  </component>
> > >
> > > Regards,
> > > Daniel Rusek
> 
> Is there anything else you need from us here? Or are you waiting for
> us for a git am'able patch? If Daniel Rusek prefers to not submit one,
> I can re-iterate with the required changes my proposal 
> https://lore.kernel.org/netdev/20250411141023.14356-2-carnil@debian.org/
> with the needed changes.

Yes, please. I'll need a formally submitted patch.

Michal

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)