[Bug 1687178] Review Request: python-apprise - Push Notifications that work with just about every platform!

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1687178



--- Comment #21 from Robert-André Mauchin <zebob.m@xxxxxxxxx> ---

 - Remove Group: everywhere,  still here

 - This is not needed:

%if 0%{?rhel} && 0%{?rhel} <= 7
# Backwards Compatibility
Provides: python-apprise = %{version}
Obsoletes:python-apprise < %{version}
%endif # rhel


I didn't realise it was a binary, I still think you should use apprise as a
*single* package, but you need to rework the conditional here, Py 2 and Py 3
being exclusionary to each other.

Remove && 0%{?fedora} > 29 and add RHEL8

%if 0%{?fedora} || 0%{?rhel} >= 8
# Python v2 Support dropped
%global with_python2 0
%endif # fedora



%package -n apprise
Summary: Apprise CLI Tool

%if 0%{?with_python2}
Requires: python2-click >= 5.0
Requires: python2-apprise = %{version}-%{release}
%endif # with_python2

%if 0%{?with_python3}
Requires: python%{python3_pkgversion}-click >= 5.0
Requires: python3-apprise = %{version}-%{release}
%endif # with_python3

%description -n apprise
An accompanied CLI tool that can be used as part of Apprise
to issue notifications from the command line to you favorite
services.


[…]


%files -n apprise
%{_mandir}/man1/apprise.1*
%{_bindir}/apprise

%if 0%{?with_python2}
%{python2_sitelib}/apprise/cli.*
%endif # with_python2
%if 0%{?with_python3}
%{python3_sitelib}/apprise/cli.*
%endif # 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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux