Bug 2324027 - Review Request: rust-libhandy-sys - Sys part of the rust bindings for libhandy
Summary: Review Request: rust-libhandy-sys - Sys part of the rust bindings for libhandy
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kalev Lember
QA Contact: Fedora Extras Quality Assurance
URL: https://crates.io/crates/libhandy-sys
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-11-05 21:11 UTC by Fabio Valentini
Modified: 2024-11-14 15:26 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-11-14 15:26:26 UTC
Type: ---
Embargoed:
klember: fedora-review+


Attachments (Terms of Use)

Description Fabio Valentini 2024-11-05 21:11:19 UTC
Spec URL: https://decathorpe.fedorapeople.org/rust-libhandy-sys.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-libhandy-sys-0.11.1-1.fc41.src.rpm

Description:
The sys part of the rust bindings for libhandy.

Fedora Account System Username: decathorpe

Comment 1 Fabio Valentini 2024-11-05 21:11:22 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=125535796

Comment 2 Fedora Review Service 2024-11-05 21:18:52 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8217153
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2324027-rust-libhandy-sys/fedora-rawhide-x86_64/08217153-rust-libhandy-sys/fedora-review/review.txt

Found issues:

- No gcc, gcc-c++ or clang found in BuildRequires
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/
- A package with this name already exists. Please check https://src.fedoraproject.org/rpms/rust-libhandy-sys
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names

Please know that there can be false-positives.

---
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.

Comment 3 Kalev Lember 2024-11-07 10:34:12 UTC
Taking for review.

Comment 4 Kalev Lember 2024-11-07 22:15:58 UTC
Fedora review rust-libhandy-sys-0.11.1-1.fc41.src.rpm 2024-10-07

$ rpmlint rust-libhandy-sys*
============================================= rpmlint session starts =============================================
rpmlint: 2.5.0
configuration:
    /usr/lib/python3.13/site-packages/rpmlint/configdefaults.toml
    /etc/xdg/rpmlint/fedora-spdx-licenses.toml
    /etc/xdg/rpmlint/fedora.toml
    /etc/xdg/rpmlint/scoring.toml
    /etc/xdg/rpmlint/users-groups.toml
    /etc/xdg/rpmlint/warn-on-functions.toml
checks: 32, packages: 7

rust-libhandy-sys+default-devel.noarch: W: no-documentation
rust-libhandy-sys+v1_0-devel.noarch: W: no-documentation
rust-libhandy-sys+v1_2-devel.noarch: W: no-documentation
rust-libhandy-sys+v1_4-devel.noarch: W: no-documentation
rust-libhandy-sys+v1_6-devel.noarch: W: no-documentation
rust-libhandy-sys-devel.noarch: W: no-documentation
======= 7 packages and 0 specfiles checked; 0 errors, 6 warnings, 32 filtered, 0 badness; has taken 0.6 s ========

+ OK
! needs attention

+ rpmlint output looks good
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the base package name.
+ The package meets the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
  Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The license text is included in %license
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match the sources in the srpm
  SHA512 (libhandy-sys-0.11.1.crate) = e816298b040fceb8e269c3285d14f0968e4baa685279a5d58eff470033e51f6b893ac7ef39746940f76ba8aa971ccb098e0707a99ddaaf534c41cfc19107d1e2
  SHA512 (Download/libhandy-sys-0.11.1.crate) = e816298b040fceb8e269c3285d14f0968e4baa685279a5d58eff470033e51f6b893ac7ef39746940f76ba8aa971ccb098e0707a99ddaaf534c41cfc19107d1e2
+ Package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a locale handling
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ Package owns all the directories it creates
+ No duplicate files in %files (LICENSE is listed twice but that's expected with the rust2rpm generator)
+ Permissions are properly set
+ Consistent use of macros
+ The package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect the runtime of application
n/a Static libraries should be in -static
+ Development files should be in -devel
n/a -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Proper .desktop file handling
+ Doesn't own files or directories already owned by other packages
+ Filenames are valid UTF-8
+ Package does not depend on deprecated packages

APPROVED

Comment 5 Fabio Valentini 2024-11-13 16:34:37 UTC
Thank you for the review!

Unretirement request:
https://pagure.io/releng/issue/12458

Comment 6 Fedora Update System 2024-11-14 15:22:37 UTC
FEDORA-2024-2954d32134 (rust-libhandy-0.11.1-1.fc42 and rust-libhandy-sys-0.11.1-1.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-2954d32134

Comment 7 Fedora Update System 2024-11-14 15:26:26 UTC
FEDORA-2024-2954d32134 (rust-libhandy-0.11.1-1.fc42 and rust-libhandy-sys-0.11.1-1.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.


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