Bug 1748678 - Review Request: usbtop - Utility to show USB bandwidth
Summary: Review Request: usbtop - Utility to show USB bandwidth
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-09-04 01:31 UTC by Carl George
Modified: 2019-09-14 16:33 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-09-08 02:59:13 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Carl George 2019-09-04 01:31:48 UTC
Spec URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01027810-usbtop/usbtop.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01027810-usbtop/usbtop-1.0-1.fc32.src.rpm

Description:
usbtop is a top-like utility that shows an estimated instantaneous bandwidth on
USB buses and devices.

Fedora Account System Username: carlwgeorge

Comment 1 Neal Gompa 2019-09-04 17:52:19 UTC
Taking this review.

Comment 2 Neal Gompa 2019-09-04 18:12:24 UTC
> install -d %{buildroot}%{_usr}/lib/modules-load.d

Replace "%{_usr}/lib/modules-load.d" with "%{_modulesloaddir}". This requires adding "BuildRequires: systemd" (or for Fedora 30+, "BuildRequires: systemd-rpm-macros" is sufficient).

> %setup -q -n usbtop-release-%{version}

You can use "%autosetup -n %{name}-release-%{version}" instead.

Comment 3 Carl George 2019-09-04 20:10:17 UTC
Spec URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01028279-usbtop/usbtop.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01028279-usbtop/usbtop-1.0-2.fc32.src.rpm

As requested:

+BuildRequires:  systemd-rpm-macros

-%setup -q -n usbtop-release-%{version}
+%autosetup -n usbtop-release-%{version}

-install -d %{buildroot}%{_usr}/lib/modules-load.d
-echo usbmon > %{buildroot}%{_usr}/lib/modules-load.d/usbtop.conf
+install -d %{buildroot}%{_modulesloaddir}
+echo usbmon > %{buildroot}%{_modulesloaddir}/usbtop.conf

-%{_usr}/lib/modules-load.d/usbtop.conf
+%{_modulesloaddir}/usbtop.conf

Comment 4 Neal Gompa 2019-09-04 21:22:46 UTC
Package review notes:

[x]: Package is named according to packaging guidelines
[x]: Package builds and installs
[x]: Package uses macros consistently
[x]: Package licensing is correctly declared and installed

PACKAGE APPROVED.

Comment 5 Carl George 2019-09-05 13:51:28 UTC
Thanks Neal.

Comment 6 Gwyn Ciesla 2019-09-05 14:02:30 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/usbtop

Comment 7 Carl George 2019-09-05 18:30:17 UTC
rawhide: https://bodhi.fedoraproject.org/updates/FEDORA-2019-c030d7ddae

Comment 8 Fedora Update System 2019-09-05 18:56:05 UTC
FEDORA-2019-a4f86875db has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-a4f86875db

Comment 9 Fedora Update System 2019-09-05 18:56:08 UTC
FEDORA-2019-5390b38041 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2019-5390b38041

Comment 10 Fedora Update System 2019-09-06 00:22:51 UTC
usbtop-1.0-2.fc31 has been pushed to the Fedora 31 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-5390b38041

Comment 11 Fedora Update System 2019-09-06 12:11:30 UTC
usbtop-1.0-2.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-a4f86875db

Comment 12 Fedora Update System 2019-09-08 02:59:13 UTC
usbtop-1.0-2.fc30 has been pushed to the Fedora 30 stable repository. If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2019-09-14 00:10:01 UTC
usbtop-1.0-2.fc31 has been pushed to the Fedora 31 stable repository. If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2019-09-14 16:33:35 UTC
usbtop-1.0-2.fc31 has been pushed to the Fedora 31 stable repository. If problems still persist, 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.