Spec URL: https://dcavalca.fedorapeople.org/review/rust-lcms2-sys/rust-lcms2-sys.spec SRPM URL: https://dcavalca.fedorapeople.org/review/rust-lcms2-sys/rust-lcms2-sys-4.0.1-1.fc40.src.rpm Description: Bindings for liblcms2 (Little CMS) with support for Linux, macOS, and Windows. See lcms2 crate for a safe Rust wrapper. Fedora Account System Username: dcavalca
This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=104834904
rust2rpm.conf used for this: [DEFAULT] buildrequires = lcms2-devel lib.requires = lcms2-devel
Copr build: https://copr.fedorainfracloud.org/coprs/build/6275480 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2231910-rust-lcms2-sys/fedora-rawhide-x86_64/06275480-rust-lcms2-sys/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.
Package was generated with rust2rpm, simplifying the review. - 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 (MIT) 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: 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) === Two comments that don't block the review: 1. It would be great if you could figure out a way to deal with the license file (possibly with upstream) that won't involve patching this in manually. 2. You could drop the "cc" optional build-dependency, it's only used by the static linking features that are removed.
(Looks like the "parallel" feature would need to be removed for 2. as well, but it seems this is only for parallel compilation of the C code?)