Bug 2359067 (rust-bitfield-struct)

Summary: Review Request: rust-bitfield-struct - Struct-like procedural macro for bitfields
Product: [Fedora] Fedora Reporter: Gerd Hoffmann <kraxel>
Component: Package ReviewAssignee: Fabio Valentini <decathorpe>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: decathorpe, package-review
Target Milestone: ---Keywords: AutomationTriaged
Target Release: ---Flags: decathorpe: fedora-review+
Hardware: All   
OS: Linux   
URL: https://crates.io/crates/bitfield-struct
Whiteboard:
Fixed In Version: Doc Type: ---
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2025-04-25 01:48:34 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2360119, 2359079, 2359085, 2359460    

Comment 1 Fedora Review Service 2025-04-11 11:55:32 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8890992
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2359067-rust-bitfield-struct/fedora-rawhide-x86_64/08890992-rust-bitfield-struct/fedora-review/review.txt

Please take a look if any issues were found.


---
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 2 Fabio Valentini 2025-04-15 21:28:21 UTC
Looks good to me!

============================================================

I think this is your first package for a Rust crate?

I would recommend to familiarize yourself with how SemVer version requirements are mapped to RPM dependencies, and to consider the update process for library-only Rust crates that is slightly different from other packages in Fedora (with library updates usually merged to all branches, in line with an Updates Policy exception, to make maintenance easier).

If you have any questions, please stop by the #rust:fedoraproject.org room on Matrix (or send me an email, I guess, but Matrix is definitely preferred, since that doesn't make me Mr. Bus Factor One).

============================================================

Package was generated with rust2rpm, simplifying the review.

✅ package contains only permissible content
✅ package builds and installs without errors on rawhide
❌ test suite is run and all unit tests pass
   (disabled due to missing dependencies - this is OK since defmt doesn't look trivial to package)
✅ latest version of the crate is packaged
✅ license matches upstream specification and is acceptable for Fedora
✅ license file is included with %license in %files
✅ package complies with Rust Packaging Guidelines

Package APPROVED.

===

Recommended post-import rust-sig tasks:

- set up package on release-monitoring.org:
  project: $crate
  homepage: https://crates.io/crates/$crate
  backend: crates.io
  version scheme: semantic
  version filter (*NOT* pre-release filter): alpha;beta;rc;pre
  distro: Fedora
  Package: rust-$crate

- set bugzilla assignee overrides to @rust-sig (optional)

Comment 3 Fedora Admin user for bugzilla script actions 2025-04-16 09:18:32 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-bitfield-struct

Comment 4 Gerd Hoffmann 2025-04-16 09:25:19 UTC
> I think this is your first package for a Rust crate?

Third (after rust-ucs2 and rust-uguid).

> I would recommend to familiarize yourself with how SemVer version
> requirements are mapped to RPM dependencies, and to consider the update
> process for library-only Rust crates that is slightly different from other
> packages in Fedora (with library updates usually merged to all branches,

Ah, ok.  So rawhide + 42 + 41 + 40?

/me was planning for rawhide + 42, but adding the others too is easy enough.

Comment 5 Fedora Update System 2025-04-16 09:50:13 UTC
FEDORA-2025-e02c02b31b (rust-bitfield-struct-0.10.1-2.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-e02c02b31b

Comment 6 Fedora Update System 2025-04-16 10:07:11 UTC
FEDORA-2025-5e59faffb7 (rust-bitfield-struct-0.10.1-2.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-5e59faffb7

Comment 7 Fabio Valentini 2025-04-17 14:26:40 UTC
> Ah, ok.  So rawhide + 42 + 41 + 40?
> 
> /me was planning for rawhide + 42, but adding the others too is easy enough.

For new packages, that's usually not necessary.
But for updates for existing packages, we basically always merge changes to all active branches of Fedora (and EPEL, if present). Having the branches not diverge makes maintenance much easier (especially building security updates etc.)

Comment 8 Fedora Update System 2025-04-17 21:03:20 UTC
FEDORA-2025-e02c02b31b has been pushed to the Fedora 42 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-e02c02b31b \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-e02c02b31b

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 9 Fedora Update System 2025-04-17 21:47:59 UTC
FEDORA-2025-5e59faffb7 has been pushed to the Fedora 41 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-5e59faffb7 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-5e59faffb7

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 10 Fedora Update System 2025-04-25 01:48:34 UTC
FEDORA-2025-5e59faffb7 (rust-bitfield-struct-0.10.1-2.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 11 Fedora Update System 2025-04-25 02:12:01 UTC
FEDORA-2025-e02c02b31b (rust-bitfield-struct-0.10.1-2.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.