https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Bug ID: 2022179 Summary: Review Request: rust-hidapi - Rust-y wrapper around hidapi Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: link@sub-pop.net QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://download.copr.fedorainfracloud.org/results/linkdupont/reviews/fedora... SRPM URL: https://download.copr.fedorainfracloud.org/results/linkdupont/reviews/fedora... Description: Rust-y wrapper around hidapi Fedora Account System Username: linkdupont Fedora Review: https://copr-be.cloud.fedoraproject.org/results/linkdupont/reviews/fedora-ra...
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@fedoraproject.org |ngompa13@gmail.com CC| |ngompa13@gmail.com Flags| |fedora-review?
--- Comment #1 from Neal Gompa ngompa13@gmail.com --- Taking this review.
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
--- Comment #2 from Neal Gompa ngompa13@gmail.com --- There seems to be a couple things wrong with this crate package:
* The build dependency "libusb-devel" is wrong, as the crate sources don't actually ask for libusb-0.1 at all (it wants libusb1 instead) * The default feature set in cargo can't be used since we don't ship the static library for hidraw or libusb1
This crate needs work to be packaged properly.
The following things need to happen here:
* The "static" feature subpackages need to be removed, as we don't have the static libraries for them * The crate's "default" feature set needs to be changed to "LINUX_SHARED_HIDRAW" in the Cargo.toml: https://github.com/ruabmbua/hidapi-rs/blob/master/Cargo.toml * The "static" features in Cargo.toml need to be removed, so that we don't generate dependencies for them * The BuildRequires needs to be switched to "pkgconfig(hidapi-hidraw)" * The +linux-shared-hidraw-devel subpackage needs "Requires: pkgconfig(hidapi-hidraw)" * The +linux-shared-libusb-devel subpackage needs "Requires: pkgconfig(hidapi-libusb)"
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
--- Comment #3 from Link Dupont link@sub-pop.net --- Fixed up these issues in this new spec. I'm not sure if there's a more Cargo-native way to disable the linux-static-hidraw feature; I just patched it out of the Cargo.toml.
Spec URL: https://download.copr.fedorainfracloud.org/results/linkdupont/reviews/fedora... SRPM URL: https://download.copr.fedorainfracloud.org/results/linkdupont/reviews/fedora... Fedora Review: https://download.copr.fedorainfracloud.org/results/linkdupont/reviews/fedora...
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ Status|ASSIGNED |POST
--- Comment #4 from Neal Gompa ngompa13@gmail.com --- Package was generated through rust2rpm, simplifying the review considerably.
- Conforms to packaging guidelines (rust2rpm generated spec) - license correct and valid - only sources installed
PACKAGE APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
--- Comment #5 from Gwyn Ciesla gwync@protonmail.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-hidapi
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |MODIFIED
--- Comment #6 from Fedora Update System updates@fedoraproject.org --- FEDORA-2021-01f7520709 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-01f7520709
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #7 from Fedora Update System updates@fedoraproject.org --- FEDORA-2021-01f7520709 has been pushed to the Fedora 35 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2021-01f7520709 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-01f7520709
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |MODIFIED
--- Comment #8 from Fedora Update System updates@fedoraproject.org --- FEDORA-2021-01f7520709 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-01f7520709
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- FEDORA-2021-01f7520709 has been pushed to the Fedora 35 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2021-01f7520709 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-01f7520709
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=2022179
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |ERRATA Status|ON_QA |CLOSED Last Closed| |2021-12-11 01:29:59
--- Comment #10 from Fedora Update System updates@fedoraproject.org --- FEDORA-2021-01f7520709 has been pushed to the Fedora 35 stable repository. If problem still persists, please make note of it in this bug report.
package-review@lists.fedoraproject.org