Bug 2434112 - Review Request: rust-cond_sync - Hides the boilerplate code needed with std::sync::Condvar
Summary: Review Request: rust-cond_sync - Hides the boilerplate code needed with std::...
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL: https://crates.io/crates/cond_sync
Whiteboard:
Depends On:
Blocks: 2434118
TreeView+ depends on / blocked
 
Reported: 2026-01-28 18:41 UTC by Gwyn Ciesla
Modified: 2026-03-09 21:44 UTC (History)
3 users (show)

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


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github emabee rust-cond_sync issues 2 0 None open License files 2026-03-03 17:09:24 UTC

Description Gwyn Ciesla 2026-01-28 18:41:37 UTC
SRPM:https://fedorapeople.org/~limb/review/rust-cond_sync/rust-cond_sync-0.2.1-1.fc44.src.rpm
SPEC:https://fedorapeople.org/~limb/review/rust-cond_sync/rust-cond_sync.spec

Description:
Hides the boilerplate code needed with std::sync::Condvar.

Comment 1 Fedora Review Service 2026-01-28 21:56:05 UTC
There seems to be some problem with the following file.
SRPM URL: https://fedorapeople.org/~limb/review/rust-cond_sync/rust-cond_sync-0.2.1-1.fc44.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 2 Ben Beasley 2026-03-02 08:06:24 UTC
The repository link in Cargo.toml is broken; I found the correct repository and filed https://github.com/emabee/rust-cond_sync/pull/1 to fix it. You might consider correcting the repository link downstream with "rust2rpm -p".

Both MIT and Apache-2.0 license texts are missing from the published crate and the upstream repository. Both licenses require the text to be distributed, so you have to actually fix this. See https://docs.fedoraproject.org/en-US/packaging-guidelines/LicensingGuidelines/#_license_text, and try working with upstream first.

Comment 4 Fedora Review Service 2026-03-04 09:06:08 UTC
There seems to be some problem with the following file.
SRPM URL: https://fedorapeople.org/~limb/review/rust-cond_sync/rust-cond_sync-0.2.1-1.fc45.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 7 Fedora Review Service 2026-03-09 21:44:03 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10205132
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2434112-rust-cond_sync/fedora-rawhide-x86_64/10205132-rust-cond_sync/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.


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