Bug 2082036 - Review Request: obserware - An advanced system monitor utility written in Python and Qt
Summary: Review Request: obserware - An advanced system monitor utility written in Pyt...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: obserware
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Akashdeep Dhar
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-05-05 09:27 UTC by Akashdeep Dhar
Modified: 2022-05-25 01:24 UTC (History)
3 users (show)

Fixed In Version: obserware-0.2.9-1.fc36 obserware-0.2.9-1.fc35
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-05-25 01:04:53 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Akashdeep Dhar 2022-05-05 09:27:39 UTC
Spec URL: https://gitlab.com/t0xic0der/obserware/uploads/fde45232815d119f4a9929aa42c68079/obserware.spec
SRPM URL: https://gitlab.com/t0xic0der/obserware/uploads/ced4ca8cf3077230111593fed4875790/obserware-0.2.9-0.fc35.src.rpm
Description: An advanced system monitor utility written in Python and Qt
Fedora Account System Username: t0xic0der

Comment 1 Onuralp Sezer 2022-05-12 17:14:24 UTC
I'm taking this review

Comment 2 Onuralp Sezer 2022-05-12 17:18:18 UTC
First please change your SOURCE0 into pypi macro (Source0: %{pypi_source})

Start your Release: 0%{?dist} number "1" and fix your dates in your change log along with your "release versions" as well.

Comment 3 Onuralp Sezer 2022-05-12 17:25:25 UTC
Also please remove those as well.

ExclusiveArch: x86_64
%global debug_package %{nil}

Comment 4 Akashdeep Dhar 2022-05-19 17:22:43 UTC
I addressed your suggestions in the following RPM specfile and source RPM.

Spec URL: https://gitlab.com/t0xic0der/obserware/uploads/2e848e41d0e7aed8e5820615070349ac/obserware.spec
SRPM URL: https://gitlab.com/t0xic0der/obserware/uploads/4b556e02d18e9ff922b2db1749eb0e0c/obserware-0.2.9-1.fc36.src.rpm

The support for architectures like arm, arm64, ppc, ppc64, ppc64le, s390x etc. is not planned and the application does not work there.

Only x86 and x86_64 architectures are supported. Please do take a look and let me know if any more changes are needed.

Comment 5 Onuralp Sezer 2022-05-22 13:13:11 UTC
- Please remove exclude arch section
- add check section and remove app stream data check from install section 
- Add desktop utils as BR for check section 
- remove no debug nil global variable as well

Comment 7 Onuralp Sezer 2022-05-22 13:16:29 UTC
All looks LGTM to me so Package Approved

Comment 8 Gwyn Ciesla 2022-05-23 17:53:59 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/obserware

Comment 9 Fedora Update System 2022-05-24 05:23:36 UTC
FEDORA-2022-ebe881da2a has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-ebe881da2a

Comment 10 Fedora Update System 2022-05-24 05:56:53 UTC
FEDORA-2022-84bf91b9d9 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-84bf91b9d9

Comment 11 Fedora Update System 2022-05-25 01:04:53 UTC
FEDORA-2022-ebe881da2a has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 12 Fedora Update System 2022-05-25 01:24:22 UTC
FEDORA-2022-84bf91b9d9 has been pushed to the Fedora 35 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.