Spec URL: https://raw.githubusercontent.com/retis-org/copr-playground/main/rust-byteorder_slice.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/@retis/playground/fedora-rawhide-x86_64/07884762-rust-byteorder_slice/rust-byteorder_slice-3.0.0-1.fc41.src.rpm Description: Byteorder like crate for &[u8] Build: https://copr.fedorainfracloud.org/coprs/g/retis/playground/build/07884762/ Fedora Account System Username: atenart, pvaler, amorenoz I’m starting this review request on behalf of Antoine, Adrián and me. We want to maintain it as a group. It’s our first package and we’ll need a sponsor.
Copr build: https://copr.fedorainfracloud.org/coprs/build/7708289 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2295811-rust-byteorder_slice/fedora-rawhide-x86_64/07708289-rust-byteorder_slice/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.
Please provide URLs to raw files, not HTML pages. Otherwise tooling will not be able to parse this ticket.
[fedora-review-service-build]
Copr build: https://copr.fedorainfracloud.org/coprs/build/7773634 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2295811-rust-byteorder_slice/fedora-rawhide-x86_64/07773634-rust-byteorder_slice/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.
> # FIXME: no license files detected It looks like you used "rust2rpm --ignore-missing-license-files" *and* didn't actually fix the FIXME? The MIT license requires that redistributed sources contain a copy of the license text. Looks like the upstream project does not contain a license text at all: https://github.com/courvoif/byteorder_slice Please file an issue there about adding the MIT license text, and including it in published crates.
Thank you for the feedback. There's an open pull request introducing a LICENSE file, but at the moment it's received no feedback: https://github.com/courvoif/byteorder_slice/pull/4 I added our request under the PR. The spec file now contains a comment with the link to the pull request. I'll continue monitoring the upstream status and ping if needed.
Thank you for filing that ticket. I think with 2 months passed, it's safe to say that you *tried*. In this case, please include the file you're proposing to add as a separate source (or patch), and replace this: > # FIXME: no license files detected > # https://github.com/courvoif/byteorder_slice/pull/4 (Similar to what was done for derive-into-owned.)
Thanks Fabio, I added the LICENSE as a patch and set it as a %license. Spec URL: https://raw.githubusercontent.com/retis-org/copr-playground/main/rust-byteorder_slice.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/@retis/playground/fedora-rawhide-x86_64/07953227-rust-byteorder_slice/rust-byteorder_slice-3.0.0-1.fc42.src.rpm Description: Byteorder like crate for &[u8] Build: https://copr.fedorainfracloud.org/coprs/g/retis/playground/build/07953227/
Created attachment 2045037 [details] The .spec file difference from Copr build 7773634 to 7953366
Copr build: https://copr.fedorainfracloud.org/coprs/build/7953366 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2295811-rust-byteorder_slice/fedora-rawhide-x86_64/07953366-rust-byteorder_slice/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 pretty good now - can you add the link to the PR to the spec file please? Something like this: > # Add license text as this is missing upstream: > # https://github.com/courvoif/byteorder_slice/pull/4 > Patch: byteorder_slice-add-mit-license-file.diff Otherwise people would need to dig into where this patch comes from.
(In reply to Fabio Valentini from comment #11) > Thanks, looks pretty good now - can you add the link to the PR to the spec > file please? Something like this: > > > # Add license text as this is missing upstream: > > # https://github.com/courvoif/byteorder_slice/pull/4 > > Patch: byteorder_slice-add-mit-license-file.diff > > Otherwise people would need to dig into where this patch comes from. Makes sense, I just added it.
Thanks, looks good to me now. However, it looks like you updated the SPEC behind the URL but not the uploaded SRPM. When importing the package, please ensure that you take the most recent version that includes the last improvement. === 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: A PR submitted to upstream to include a license file two months ago, but was not addressed yet. A patch that corresponds to the PR that adds the missing license file is temporarily included in the package. ✅ 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: 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) === Your sponsor should be able to help you with these tasks and with importing the package if you have any questions.
The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-byteorder_slice
FEDORA-2024-c3c9b3bbe3 (rust-byteorder_slice-3.0.0-1.fc42) has been submitted as an update to Fedora 42. https://bodhi.fedoraproject.org/updates/FEDORA-2024-c3c9b3bbe3
FEDORA-2024-c3c9b3bbe3 (rust-byteorder_slice-3.0.0-1.fc42) has been pushed to the Fedora 42 stable repository. If problem still persists, please make note of it in this bug report.