Spec URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark.spec SRPM URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark-0.10.2-1.fc43.src.rpm Description: Postmark rust client. Fedora Account System Username: lecris
Copr build: https://copr.fedorainfracloud.org/coprs/build/8888321 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2358960-rust-postmark/fedora-rawhide-x86_64/08888321-rust-postmark/builder-live.log.gz Please make sure the package builds successfully at least for Fedora Rawhide. - If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field --- 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.
Looks like the heuristics for generating package Summary failed here, can you fix it?
Spec URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark.spec SRPM URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark-0.10.2-1.fc43.src.rpm Update the package summary and description
Thanks - looks like the package is MIT / Apache-2.0 dual licensed, but only contains the Apache-2.0 license text? The README is also not helpful: > postmark is distributed under the terms of both the MIT license and the Apache License (Version 2.0). This is confusing - is it *both* or *alternatively*? Those are two different things (i.e AND or OR).
Thanks, I've raised the issue with upstream.
Spec URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark.spec SRPM URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark-0.11.3-1.fc43.src.rpm Update to postmark 0.11. They fixed the license there :) Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=132173419
Copr build: https://copr.fedorainfracloud.org/coprs/build/8986526 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2358960-rust-postmark/fedora-rawhide-x86_64/08986526-rust-postmark/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.
Looks good to me, with one exception: You generated the spec with all features enabled, this might not do what you want: There are two reqwest backends, and they should be mutually exclusive (native-tls and rustls-tls) - I'm not sure how it doesn't give you a compile error when both are enabled. It's hard to tell for me without the rust2rpm.toml file *why* you enabled all features - if it's for running / compiling tests, please just enable the features that are necessary. But enabling *both* reqwest-native-tls *and* reqwest-rustls-tls seems to be a mistake.
Spec URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark.spec SRPM URL: https://lecris.fedorapeople.org/reviews/rust-postmark/rust-postmark-0.11.3-1.fc43.src.rpm Removed the feature-all. It seems there is no reference of these features in the code, other than reqwest which I've enabled in order to get it to compile. Probably the error would be ocurring on the consumer side Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=132732012
Created attachment 2089843 [details] The .spec file difference from Copr build 8986526 to 9040407
Copr build: https://copr.fedorainfracloud.org/coprs/build/9040407 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2358960-rust-postmark/fedora-rawhide-x86_64/09040407-rust-postmark/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.
Thanks, looks good to me now! === 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 ✅ 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)
Thank you for the review, Fabio
The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-postmark
FEDORA-2025-c788c2483d (rust-postmark-0.11.3-1.fc41) has been submitted as an update to Fedora 41. https://bodhi.fedoraproject.org/updates/FEDORA-2025-c788c2483d
FEDORA-2025-ceac2606f6 (rust-postmark-0.11.3-1.fc42) has been submitted as an update to Fedora 42. https://bodhi.fedoraproject.org/updates/FEDORA-2025-ceac2606f6
FEDORA-2025-2f8b77ecfd (rust-postmark-0.11.3-1.fc43) has been submitted as an update to Fedora 43. https://bodhi.fedoraproject.org/updates/FEDORA-2025-2f8b77ecfd
FEDORA-2025-2f8b77ecfd (rust-postmark-0.11.3-1.fc43) has been pushed to the Fedora 43 stable repository. If problem still persists, please make note of it in this bug report.
FEDORA-2025-ceac2606f6 has been pushed to the Fedora 42 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-ceac2606f6` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-ceac2606f6 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
FEDORA-2025-c788c2483d has been pushed to the Fedora 41 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-c788c2483d` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-c788c2483d See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
FEDORA-2025-c788c2483d (rust-postmark-0.11.3-1.fc41) has been pushed to the Fedora 41 stable repository. If problem still persists, please make note of it in this bug report.
FEDORA-2025-ceac2606f6 (rust-postmark-0.11.3-1.fc42) has been pushed to the Fedora 42 stable repository. If problem still persists, please make note of it in this bug report.