Bug 2116603 - Review Request: rust-lebe - Tiny, dead simple, high performance endianness conversions with a generic API
Summary: Review Request: rust-lebe - Tiny, dead simple, high performance endianness co...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Fabio Valentini
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-08-09 02:32 UTC by Orion Poplawski
Modified: 2022-08-30 01:45 UTC (History)
2 users (show)

Fixed In Version: rust-lebe-0.5.2-1.fc38
Clone Of:
Environment:
Last Closed: 2022-08-30 01:45:57 UTC
Type: Bug
Embargoed:
decathorpe: fedora-review+


Attachments (Terms of Use)

Description Orion Poplawski 2022-08-09 02:32:48 UTC
Spec URL: https://orion.fedorapeople.org/rust-lebe.spec
SRPM URL: https://orion.fedorapeople.org/rust-lebe-0.5.1-1.fc37.src.rpm
Description:
Tiny, dead simple, high performance endianness conversions with a generic API.

Fedora Account System Username: orion

Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=90632511

Comment 1 Fabio Valentini 2022-08-11 15:59:02 UTC
There's a big red "FIXME" instead of the License tag.

The project doesn't specify its license in the standard place in Cargo.toml metadata (package.license), but uses a custom license, and I don't even recognize it (though askalono claims it's BSD-3-Clause). I wonder why the upstream project doesn't just specify "license = BSD-3-Clause" in the metadata, but instead uses "license-file = LICENSE.md", which is intended for non-standard licenses.

Comment 2 Orion Poplawski 2022-08-14 03:56:08 UTC
I've fixed the license and reported the issue upstream.

Comment 3 Fabio Valentini 2022-08-17 10:07:36 UTC
Thanks for fixing the issue upstream! 🎉

Looks like you still need to replace
%license %{crate_instdir}/LICENSE.md
with
%{crate_instdir}/LICENSE-BSD-3-Clause
in %files of the devel subpackage?

And the linked SRPM file doesn't reflect the latest changes yet.

Other than those two issues, package looks good to me now.

Comment 4 Orion Poplawski 2022-08-18 02:38:36 UTC
Upstream did a release:

Spec URL: https://orion.fedorapeople.org/rust-lebe.spec
SRPM URL: https://orion.fedorapeople.org/rust-lebe-0.5.2-1.fc38.src.rpm

Comment 5 Fabio Valentini 2022-08-18 09:56:38 UTC
Awesome, thanks! Package looks good to me now.

===

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 (BSD-3-Clause) 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

Comment 6 Jens Petersen 2022-08-28 05:35:12 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-lebe

Comment 7 Orion Poplawski 2022-08-30 01:45:57 UTC
Built.  Thanks all.


Note You need to log in before you can comment on or make changes to this bug.