[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 Hanns-Joachim Uhl changed: What|Removed |Added Status|NEW |CLOSED Depends On||1860811 Resolution|--- |CURRENTRELEASE Last Closed||2020-08-13 09:18:45 --- Comment #19 from Hanns-Joachim Uhl --- fyi ... with LTC bug 187118 - RH1860811- Fedora - Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes now included for Fedora 33 I am closing this Red Hat bugzilla now ... Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1860811 [Bug 1860811] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1071880 (PPCTracker) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1071880 [Bug 1071880] (PPCTracker) Fedora for PowerPC architectures (ppc64,ppc64le): Bug Tracker -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1853217 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1853217 [Bug 1853217] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 --- Comment #18 from IBM Bug Proxy --- --- Comment From sjain...@in.ibm.com 2020-07-02 06:16 EDT--- Hello Team, As suggested in the previous comment I have opened a new BZ with 'Sourabh Jain' ID on https://bugzilla.redhat.com. Also addressed the comments on the SPEC file and posted the updated SPEC on the new BZ. New BZ link: https://bugzilla.redhat.com/show_bug.cgi?id=1853217 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 --- Comment #17 from IBM Bug Proxy --- --- Comment From sjain...@in.ibm.com 2020-07-01 03:01 EDT--- Hello Team, Thanks for the review. I will open a new BUG and will post the updated SPEC file there. (In reply to comment #19) > - Don't use %define but %global: > > %define name ServiceReport > %define version 2.2.1 > %define release 1 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 --- Comment #16 from Robert-André Mauchin --- I.e. you should open a new bug as jainsourabh 'Sourabh Jain' and close this one as duplicate because it won't go anywhere. You'll need a sponsor to become a member of the packager group. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 --- Comment #15 from Robert-André Mauchin --- Also the reporter of the bug (with their bugzilla email) must be the one to request the repository, Using bugpr...@us.ibm.com instead of a real FAS account will prevent the script to request a repo to work. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854
Robert-André Mauchin changed:
What|Removed |Added
CC||zebo...@gmail.com
--- Comment #14 from Robert-André Mauchin ---
- Don't use %define but %global:
%define name ServiceReport
%define version 2.2.1
%define release 1
And you shouldn't redefine name, version and release, use the header field for
that
- Group: is not used in Fedora
- No:
%define debug_package %{nil}
Instead make you package noarch since it's Python stuff:
BuildArch: noarch
- Use Python macros: see
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_macros
%{python} setup.py build → %py3_build
%{python} setup.py install --root=$RPM_BUILD_ROOT → %py3_install
- Not needed: %defattr(-,root,root)
- Use macros for directories:
/usr/share/man → %{_mandir}
/usr/lib/systemd/system/ → %{_unitdir}
- Man pages are not to be installed with %doc:
%{_mandir}/man8/*
- Not needed:
Requires: %{python} systemd
- BR systemd-rpm-macros instead of systemd (if you need EPEL7 support put it
behind a condition)
- Some systemd scriptles are missing: see:
https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#_systemd
%post
%systemd_post servicereport.service
%preun
%systemd_preun servicereport.service
%postun
%systemd_postun_with_restart servicereport.service
- Not needed: %doc /usr/share/doc/*, install the README.md with:
%doc README.md
- You should provide a COPYING file and install it with %license in %files:
%license COPYING
- # By default python 3 is used to build the package.
Only Python 3 is available for Fedora 32 and above. If you're planning EPEL7
and Python2, use conditionals, for example
%if 0%{?fedora} || 0%{?rhel} >= 7
%bcond_withpython2
%bcond_without python3
%else
%bcond_withpython3
%bcond_without python2
%endif
although I'd recommend to stick with Python3
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854
--- Comment #13 from IBM Bug Proxy ---
--- Comment From sjain...@in.ibm.com 2020-06-19 00:52 EDT---
Hello Team,
(In reply to comment #17)
> Sourabh: SPEC and SRPM give 404 error.
I am not sure why the SPEC file and Source RPM file has been removed from the
corp portal. I guess the corp portal removes all files belongs to builds after
certain days.
The below link contains a fresh build of ServiceReport on the corp portal. It
includes SPEC file and Source RPMs.
https://download.copr.fedorainfracloud.org/results/sourabhjains/ServiceReport/fedora-31-x86_64/01486459-ServiceReport/
Note: added a new BuildRequires[: %{python}-devel] tag in the SPEC file to
handle the missing python3_sitelib macro error.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
[Bug 1789854] Review Request: ServiceReport - a tool to validate and repair system configuration for specific purposes
https://bugzilla.redhat.com/show_bug.cgi?id=1789854 Susi Lehtola changed: What|Removed |Added CC||susi.leht...@iki.fi Summary|Fedora - New package|Review Request: |request: Provide a new tool |ServiceReport - a tool to |for ppc64le to validate and |validate and repair system |auto-correct the FFDC setup |configuration for specific |(ServiceReport) |purposes --- Comment #12 from Susi Lehtola --- Sourabh: SPEC and SRPM give 404 error. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
