Bug 2342799

Summary: Review Request: rust-trustfall_derive - Derive macros for the trustfall query engine
Product: [Fedora] Fedora Reporter: Michel Lind <michel>
Component: Package ReviewAssignee: Fabio Valentini <decathorpe>
Status: POST --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: decathorpe, package-review
Target Milestone: ---Keywords: AutomationTriaged
Target Release: ---Flags: decathorpe: fedora-review+
decathorpe: needinfo? (michel)
Hardware: Unspecified   
OS: Unspecified   
URL: https://crates.io/crates/trustfall_derive
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Michel Lind 2025-01-29 13:58:19 UTC
Spec URL: https://salimma.fedorapeople.org/rust-trustfall_derive.spec
SRPM URL: https://salimma.fedorapeople.org/rust-trustfall_derive-0.3.1-1.fc41.src.rpm

Description:
Derive macros for the trustfall query engine.

Fedora Account System Username: salimma

Comment 1 Michel Lind 2025-01-29 13:58:22 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=128607847

Comment 2 Fedora Review Service 2025-01-29 14:03:14 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8584365
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2342799-rust-trustfall_derive/fedora-rawhide-x86_64/08584365-rust-trustfall_derive/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 4 Fabio Valentini 2025-03-14 17:24:45 UTC
Looks good to me!

Note that you can automate even more with rust2rpm.toml:

```
[package]
license-files = ["LICENSE.txt"]

[[package.extra-sources]]
number = 1
file = "https://raw.githubusercontent.com/obi1kenobi/trustfall/refs/tags/trustfall-v0.8.0/LICENSE.txt"
comments = ["add license: https://github.com/obi1kenobi/trustfall/pull/734"]

[scripts]
prep.post = ["cp -p %{SOURCE1} ."]
```

Explicitly telling rust2rpm that there is indeed a license file (with a non-empty "package.license-files" setting also gets you past the "no license files detected" error without needing to pass "--ignore-missing-license-files". :)

===

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 with good justification)
✅ latest version of the crate is packaged
✅ license matches upstream specification and is acceptable for Fedora
🫤 license file is included with %license in %files (included manually, fix submitted to upstream)
✅ 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

- add @rust-sig with "commit" access as package co-maintainer
  (should happen automatically)

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

- track package in koschei for all built branches
  (should happen automatically once rust-sig is co-maintainer)

Comment 5 Fabio Valentini 2026-03-13 21:53:42 UTC
Are you still interested in this package?

Comment 6 Michel Lind 2026-03-13 22:01:11 UTC
Hmm, trying to remember what this was for. I think it will be nice to have git-seek, which ultimately depends on this, so yes please, if you could re-set the + I will import this.

Comment 7 Fabio Valentini 2026-03-14 15:54:35 UTC
It was for cargo-semver-checks. I've renewed the + flag.

Comment 8 Fabio Valentini 2026-04-29 14:37:47 UTC
*ping*

It's now been over a year since the initial approval. If you're really not that interested any more, please just close the ticket.