Bug 2447805 - Review Request: rust-quad-alsa-sys - Fork of original alsa-sys, but with pkg-config replaced with -lasound
Summary: Review Request: rust-quad-alsa-sys - Fork of original alsa-sys, but with pkg-...
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2026-03-15 22:10 UTC by Jonathan Steffan
Modified: 2026-08-29 21:36 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jonathan Steffan 2026-03-15 22:10:21 UTC
Spec URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys.spec
SRPM URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys-0.3.2-1.fc45.src.rpm
Description: Fork of original alsa-sys, but with pkg-config replaced with -lasound.
Fedora Account System Username: jsteffan

Comment 2 Fedora Review Service 2026-04-06 14:50:21 UTC
There seems to be some problem with the following file.
SRPM URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys-0.3.2-1.fc43.src.rpm
Fetching it results in a 404 Not Found error.
Please make sure the URL is correct and publicly available.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 4 Fabio Valentini 2026-08-19 15:42:24 UTC
A few issues:

- Built packages don't install (dependency on bindgen too old).
- Drop "regenerate_bindings" executable. This is only used for upstream development (to regenerate the pre-generated bindings).
- Make the "use-bindings" feature a noop and unconditionally regenerate bindings with bindgen at build-time (this will require patches for Cargo.toml and build.rs). If it is possible to do so, pregenerated code should be dropped and regenerated during the build.
- Drop "regenerate_bindings.sh" script, it causes a dependency on "sh" to be generated for the package.

Comment 5 Jonathan Steffan 2026-08-29 21:36:58 UTC
Changelog:

- Fixed built packages failing to install: bindgen was an optional 0.56-pinned build-dependency, unsatisfiable since Fedora only ships bindgen 0.72; made it mandatory, pinned to 0.72
- Dropped the "regenerate_bindings" dev-only helper binary and regenerate_bindings.sh (never actually buildable, and its #!/bin/sh shebang pulled an unwanted /usr/bin/sh dependency onto -devel)
- Bindings are now unconditionally regenerated from live ALSA headers at build time instead of shipping a stale pregenerated src/generated.rs; fixed the ALSA header include path and updated to the bindgen 0.72 API
- Reworked the Cargo.toml fix to use rust2rpm's own metadata-patch mechanism and added pkgconfig(alsa) Requires/BuildRequires, so rust2rpm regeneration now reproduces the spec exactly

Spec URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys.spec
SRPM URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys-0.3.2-1.fc44.src.rpm
rust2rpm.toml URL: https://jsteffan.fedorapeople.org/xrizer/rust-quad-alsa-sys-rust2rpm.toml


Note You need to log in before you can comment on or make changes to this bug.