Spec URL: https://decathorpe.fedorapeople.org/rust-cargo-options.spec SRPM URL: https://decathorpe.fedorapeople.org/rust-cargo-options-0.6.0-1.fc38.src.rpm Description: Reusable common Cargo command line options. Fedora Account System Username: decathorpe
This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=100098622
This package was generated using rust2rpm, which simplifies the review. Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated - [x] The specfile is sane. - [x] The latest version is packaged or packaging an earlier version is justified. - [x] The License tag uses the correct identifiers, and it reflects the package contents and Cargo metadata. - [x] The license text is included in the package and is marked with %license. - [x] The package builds successfully in mock. - [x] Package is installable (checked by fedora-review). - [x] There are no relevant rpmlint errors. (There are some warnings about zero length test files.) - [-] (Justified with a comment in the specfile) The package runs tests in %check. - [x] The package complies with the Rust and general Packaging Guidelines. Package approved. Thanks!
> (There are some warnings about zero length test files.) Hmm, should the tests be excluded from the package all together? In other packaging ecosystems I'm involved with (e.g. Python and Ansible collections), it's standard to remove the tests from the built package even if upstream installs them, but I'm not sure if Rust is different in this regard.
Thanks for the review! > Hmm, should the tests be excluded from the package all together? Usually the test sources don't hurt, and we don't remove them unless they include problematic things like architecture-specific binaries or other large test fixtures like images. > but I'm not sure if Rust is different in this regard. Not really, but we don't have Guidelines for it, so it's more done on a case-by-case basis.
The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-cargo-options
Imported and built for rawhide: https://bodhi.fedoraproject.org/updates/FEDORA-2023-38a733702d