Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://raw.githubusercontent.com/Laiot/rust-integer-encoding-rpm/master/rust-integer-encoding.spec SRPM URL: https://github.com/Laiot/rust-integer-encoding-rpm/raw/master/rust-integer-encoding-3.0.3-1.fc36.src.rpm Description: Varint+zigzag and fixedint integer encoding/decoding Fedora Account System Username: laiot
Two small things: - trim the Summary in the spec file the same way you did for the bug title (i.e. remove the URL in parentheses) - exclude the "coverage.sh" script from being included in packaged files (i.e. by patching Cargo.toml to add "exclude = ["/coverage.sh"]") Other than that, I'm only curious what you're packaging this crate for :) === 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: - add @rust-sig with "commit" access as package co-maintainer - set bugzilla assignee overrides to @rust-sig (optional) - 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 - track package in koschei for all built branches
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-integer-encoding
Hey, looks like you missed one of my comments: - exclude the "coverage.sh" script from being included in packaged files (i.e. by patching Cargo.toml to add "exclude = ["/coverage.sh"]") And the build failed due to what looks like endianness-related problems (s390x is our only big-endian architecture). You might want to report this with the upstream project, otherwise the crate will most probably not work correctly on s390x.
Sorry about that, I just pushed the patched .spec but apparently you're right, the build fails. I will report this to the upstream project.
Looks like this is the issue you reported? https://github.com/dermesser/integer-encoding-rs/issues/24 Please follow up there.
The package was retired from Fedora 38+.