Bug 2368362

Summary: Review Request: rust-ssh-agent-client-rs - Pure rust implementation of the ssh-agent protocol
Product: [Fedora] Fedora Reporter: Noa Resare <noa>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: NEW --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review
Target Milestone: ---Keywords: AutomationTriaged
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://crates.io/crates/ssh-agent-client-rs
Whiteboard:
Fixed In Version: Doc Type: ---
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:
Attachments:
Description Flags
The .spec file difference from Copr build 9077037 to 9078264 none

Description Noa Resare 2025-05-24 12:34:23 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/noa/rust/fedora-42-aarch64/09077024-rust-ssh-agent-client-rs/rust-ssh-agent-client-rs.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/noa/rust/fedora-42-aarch64/09077024-rust-ssh-agent-client-rs/rust-ssh-agent-client-rs-1.0.0-1.fc42.src.rpm
Description: Pure rust implementation of the ssh-agent protocol
Fedora Account System Username: noa

I believe I need a sponsor, even though I dabbled with Fedora packaging a long time ago.

I am the upstream author of this code, and my purpose in packaging it is to be able to package my pam-ssh-agent crate for Fedora.

The COPR build referenced above builds its srpm using the srpm target in this makefile: https://github.com/nresare/rpm-packaging/blob/main/ssh-agent-client-rs/Makefile

Comment 1 Fedora Review Service 2025-05-24 12:42:06 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9077037
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2368362-rust-ssh-agent-client-rs/fedora-rawhide-x86_64/09077037-rust-ssh-agent-client-rs/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 Noa Resare 2025-05-25 11:54:15 UTC
It turns out that the copr 'make srpm' feature that only invokes "dnf install rust2rpm" and not "dnf install rust2rpm rust2rpm-helper" will silently skip generating patches that removes non-matching conditional dependencies, such as the windows only interprocess.

Here is an updated .src.rpm and .spec where this is fixed:

https://download.copr.fedorainfracloud.org/results/noa/rust/epel-10-aarch64/09077642-rust-ssh-agent-client-rs/rust-ssh-agent-client-rs-1.0.0-1.el10.src.rpm
https://download.copr.fedorainfracloud.org/results/noa/rust/epel-10-aarch64/09077642-rust-ssh-agent-client-rs/rust-ssh-agent-client-rs.spec

Comment 3 Fedora Review Service 2025-05-25 12:01:46 UTC
Created attachment 2091502 [details]
The .spec file difference from Copr build 9077037 to 9078264

Comment 4 Fedora Review Service 2025-05-25 12:01:49 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9078264
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2368362-rust-ssh-agent-client-rs/fedora-rawhide-x86_64/09078264-rust-ssh-agent-client-rs/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.