https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #1 from Chris Caron <lead2gold(a)gmail.com&gt; --- The wiki mentions passing along working koji (--scratch) builds. No problem, below is all passing builds: * epel7: https://koji.fedoraproject.org/koji/taskinfo?taskID=33369676 * f28: https://koji.fedoraproject.org/koji/taskinfo?taskID=33369716 * f29: https://koji.fedoraproject.org/koji/taskinfo?taskID=33369791 * rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=33369860 -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 Robert-André Mauchin <zebob.m(a)gmail.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zebob.m(a)gmail.com --- Comment #3 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- Also LICENSE files must be included with %license not %doc -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #5 from Chris Caron <lead2gold(a)gmail.com&gt; --- Thank you guys for all your feedback. It was also suggested on the mailing lists to not use flake8 as it's to harsh to fail a %test because of a pep8 issue. I couldn't agree with this more, so i dropped this to just be py.test references instead. I'll satisfy my own PEP8 OCD with travisCI and not with RPMS. I should add that i'm also the developer of this product. So yes i realize i pushed a new version out but supplied you with the last one. It was kind of happening at the same time. I addressed all of the comments you identified above (including the new version reference): Updated Spec: https://copr-be.cloud.fedoraproject.org/results/lead2gold/apprise/fedora-... Here is the Koji output: epel7: https://koji.fedoraproject.org/koji/taskinfo?taskID=33411608 f28: https://koji.fedoraproject.org/koji/taskinfo?taskID=33411610 f29: https://koji.fedoraproject.org/koji/taskinfo?taskID=33411612 f30: https://koji.fedoraproject.org/koji/taskinfo?taskID=33411614 rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=33411616 Copr: https://copr.fedorainfracloud.org/coprs/lead2gold/apprise/build/866973/ On a side note; the license was recently changed (About a month or so ago) from GPLv3 to MIT. I didn't realize i didn't update this on the header of my SPEC file. I'll update this on the next build. Thank you guys again for all of your feedback! -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #7 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- Also remove .nuxref in Release: -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #9 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- - In that case you can remove: %if 0%{?rhel} && 0%{?rhel} > 7 # This package is not included with EPEL7 # glibc-all-langpacks used for LANG= line in %test reference # https://fedoraproject.org/wiki/Changes/Remove_glibc-langpacks-\ # all_from_buildroot#Remove_glibc-all-langpacks_from_buildroot BuildRequires: glibc-all-langpacks %endif - Also drop Group: Development/Languages and: BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) like Artur said. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 Robert-André Mauchin <zebob.m(a)gmail.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(fedora(a)svgames.pl | |) --- Comment #11 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- I'll let Artur finish if he wishes. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #13 from Chris Caron <lead2gold(a)gmail.com&gt; --- Guys, I also dropped all 'python-coverage', and 'python-tox' reference since I'm no longer using that in the %test section (so spec got much cleaner). This allowed me to enable testing for f28 and epel (previously disabled because of reference to tox). But then 'epel7' couldn't pass 3 tests because they reference features only available in newer builds of 'py.test' and 'requests'. SO then i created a patch file for epel7 to work around this which means i had to drop the %autosetup in the %prep section (so i could JUST apply the patch to epel7 builds). It's been changed to just %setup so i could wrap the %patchX entry. Anyway, there is no doubt i broke some standards here introducing the patch file. I'm not sure how to properly host the .patch file (as it is now it's just dropped in my SOURCES directory prior to my build; it's referenced relatively in .spec). Is this satisfactory? I'll attach the spec to this ticket. Everything still builds nicely with this additional clean-up. SPEC: https://copr-be.cloud.fedoraproject.org/results/lead2gold/apprise/fedora-... COPR: https://copr.fedorainfracloud.org/coprs/build/867578/ Koji: epel7: https://koji.fedoraproject.org/koji/taskinfo?taskID=33443066 f28: https://koji.fedoraproject.org/koji/taskinfo?taskID=33443069 f29: https://koji.fedoraproject.org/koji/taskinfo?taskID=33443070 f30: https://koji.fedoraproject.org/koji/taskinfo?taskID=33443071 rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=33443073 -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #15 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- - Typo: %package -n python2-apprise-cli Summary: Apprise CLI Tool Python 3 binary Should be 2 here - Remove the Group: everywhere - Typo in both cli descriptions: python3-apprise-cli.noarch: W: spelling-error %description -l en_US notificatios -> notifications - Remove the shebang in prep: python3-apprise-cli.noarch: E: wrong-script-interpreter /usr/lib/python3.7/site-packages/apprise/cli.py /usr/bin/env python python3-apprise-cli.noarch: E: non-executable-script /usr/lib/python3.7/site-packages/apprise/cli.py 644 /usr/bin/env python Package Review ============== Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed Issues: ======= - Package must not depend on deprecated() packages. Note: python2 is deprecated, you must not depend on it. See: https://docs.fedoraproject.org/en-US/packaging- guidelines/deprecating-packages/ ===== MUST items ===== Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "Unknown or generated", "Expat License", "*No copyright* Expat License". 47 files have unknown license. Detailed output of licensecheck in /home/bob/packaging/review/python-apprise/review- python-apprise/licensecheck.txt [x]: License file installed when any subpackage combination is installed. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Package is not known to require an ExcludeArch tag. [-]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 20480 bytes in 1 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %license. [x]: Package requires other packages for directories it uses. [x]: Package must own all directories that it creates. [x]: Package does not own files or directories owned by other packages. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Dist tag is present. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package does not use a name that already exists. [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local Python: [x]: Python eggs must not download any dependencies during the build process. [x]: A package which is used by another package via an egg interface should provide egg info. [x]: Package meets the Packaging Guidelines::Python [x]: Package contains BR: python2-devel or python3-devel [x]: Packages MUST NOT have dependencies (either build-time or runtime) on packages named with the unversioned python- prefix unless no properly versioned package exists. Dependencies on Python packages instead MUST use names beginning with python2- or python3- as appropriate. [x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files [x]: Binary eggs must be removed in %prep ===== SHOULD items ===== Generic: [-]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: Final provides and requires are sane (see attachments). [-]: Fully versioned dependency in subpackages if applicable. Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in python3-apprise , python3-apprise-cli [?]: Package functions as described. [x]: Latest version is packaged. [x]: Package does not include license text files separate from upstream. [-]: Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x]: Package should compile and build into binary rpms on all supported architectures. [x]: %check is present and all tests pass. [x]: Packages should try to preserve timestamps of original installed files. [x]: Reviewer should test that the package builds in mock. [x]: Buildroot is not present [x]: Package has no %clean section with rm -rf %{buildroot} (or $RPM_BUILD_ROOT) [x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin. [x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file [x]: Sources can be downloaded from URI in Source: tag [x]: SourceX is a working URL. [x]: Spec use %global instead of %define unless justified. ===== EXTRA items ===== Generic: [x]: Rpmlint is run on all installed packages. Note: There are rpmlint messages (see attachment). [x]: Spec file according to URL is the same as in SRPM. Rpmlint ------- Checking: python3-apprise-0.7.4-1.fc31.noarch.rpm python3-apprise-cli-0.7.4-1.fc31.noarch.rpm python-apprise-0.7.4-1.fc31.src.rpm python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.opt-1.pyc python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.pyc python3-apprise-cli.noarch: W: spelling-error %description -l en_US notificatios -> notifications, notification, ossification python3-apprise-cli.noarch: W: no-documentation python3-apprise-cli.noarch: E: wrong-script-interpreter /usr/lib/python3.7/site-packages/apprise/cli.py /usr/bin/env python python3-apprise-cli.noarch: E: non-executable-script /usr/lib/python3.7/site-packages/apprise/cli.py 644 /usr/bin/env python python3-apprise-cli.noarch: W: no-manual-page-for-binary apprise 3 packages and 0 specfiles checked; 2 errors, 5 warnings. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #17 from Chris Caron <lead2gold(a)gmail.com&gt; --- Created attachment 1543764 --> https://bugzilla.redhat.com/attachment.cgi?id=1543764&action=edit shebang patch just created for this release to handle rpmlint issue This has been fixed upstream already here: https://github.com/caronc/apprise/commit/764aa5393ffe9e09b7da6bde82078b88... Thus the next official packaging of apprise will allow for the removal of this patch. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #19 from Chris Caron <lead2gold(a)gmail.com&gt; --- Robert, I stuck with just setting a Provides: entry to the python2-apprise-cli and python3-apprise-cli because I couldn't think of a clever way to not have a %package conflict if i renamed both of them to just 'apprise'. You can check it my workaround here. SPEC: https://copr-be.cloud.fedoraproject.org/results/lead2gold/apprise/fedora-... epel7: https://koji.fedoraproject.org/koji/taskinfo?taskID=33494613 f29: https://koji.fedoraproject.org/koji/taskinfo?taskID=33494615 f30: https://koji.fedoraproject.org/koji/taskinfo?taskID=33494617 f31: https://koji.fedoraproject.org/koji/taskinfo?taskID=33494619 rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=33494621 COPR: https://copr.fedorainfracloud.org/coprs/build/868455 You'll be happy to know i built a man page too to start with; I'll probably need more to explain how configuration files work, but it's a start and also greatly reduces your rpmlint warnings to just 2 now: python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.opt-1.pyc python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.pyc I'll need to Google these warnings because i don't quite know what it refers to at this time. The man page (to clear up 2 of the warnings) was added as an additional Source which has also already been pushed upstream; so it will be bundled in upcoming releases. As always, I respect any advice or feedback you have! Chris -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #21 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- - 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 Robert-André Mauchin <zebob.m(a)gmail.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody(a)fedoraproject.org |zebob.m(a)gmail.com Flags|needinfo- |fedora-review+ --- Comment #23 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- - You could use sed instead of a patch to remove the shebang, it's a bbit overkill for one line. - %{__install} → install Package is approved. You still need to find a sponsor: https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group (Yes I need to find time to finish the review I gave you) Package Review ============== Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed ===== MUST items ===== Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "Unknown or generated", "Expat License", "*No copyright* Expat License". 47 files have unknown license. Detailed output of licensecheck in /home/bob/packaging/review/python-apprise/review- python-apprise/licensecheck.txt [x]: License file installed when any subpackage combination is installed. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Package is not known to require an ExcludeArch tag. [-]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 20480 bytes in 1 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %license. [x]: Package requires other packages for directories it uses. [x]: Package does not own files or directories owned by other packages. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Dist tag is present. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package does not use a name that already exists. [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local Python: [x]: Python eggs must not download any dependencies during the build process. [x]: A package which is used by another package via an egg interface should provide egg info. [x]: Package meets the Packaging Guidelines::Python [x]: Package contains BR: python2-devel or python3-devel [x]: Packages MUST NOT have dependencies (either build-time or runtime) on packages named with the unversioned python- prefix unless no properly versioned package exists. Dependencies on Python packages instead MUST use names beginning with python2- or python3- as appropriate. [x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files [x]: Binary eggs must be removed in %prep ===== SHOULD items ===== Generic: [-]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: Final provides and requires are sane (see attachments). [-]: Fully versioned dependency in subpackages if applicable. Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in apprise , python3-apprise [?]: Package functions as described. [x]: Latest version is packaged. [x]: Package does not include license text files separate from upstream. [x]: Patches link to upstream bugs/comments/lists or are otherwise justified. [-]: Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x]: Package should compile and build into binary rpms on all supported architectures. [x]: %check is present and all tests pass. [x]: Packages should try to preserve timestamps of original installed files. [x]: Reviewer should test that the package builds in mock. [x]: Buildroot is not present [x]: Package has no %clean section with rm -rf %{buildroot} (or $RPM_BUILD_ROOT) [x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin. [x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file [x]: Sources can be downloaded from URI in Source: tag [x]: SourceX is a working URL. [x]: Spec use %global instead of %define unless justified. ===== EXTRA items ===== Generic: [x]: Rpmlint is run on all installed packages. Note: There are rpmlint messages (see attachment). [x]: Spec file according to URL is the same as in SRPM. Rpmlint ------- Checking: apprise-0.7.4-1.fc31.noarch.rpm python3-apprise-0.7.4-1.fc31.noarch.rpm python-apprise-0.7.4-1.fc31.src.rpm apprise.noarch: W: name-repeated-in-summary C Apprise python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.opt-1.pyc python3-apprise.noarch: W: python-bytecode-without-source /usr/lib/python3.7/site-packages/apprise/__pycache__/cli.cpython-37.pyc python-apprise.src:208: W: macro-in-comment %{__install} python-apprise.src:209: W: macro-in-comment %{buildroot} python-apprise.src:209: W: macro-in-comment %{_mandir} python-apprise.src:209: W: mixed-use-of-spaces-and-tabs (spaces: line 45, tab: line 209) 3 packages and 0 specfiles checked; 0 errors, 7 warnings. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #25 from Robert-André Mauchin <zebob.m(a)gmail.com&gt; --- (In reply to Chris Caron from comment #24) yes it is a possibility, you might still do other informal reviews and post them here to get sponsored, in order to show that you grok the guidelines. -- 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

https://bugzilla.redhat.com/show_bug.cgi?id=1687178 --- Comment #27 from Igor Gnatenko <i.gnatenko.brain(a)gmail.com&gt; --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-apprise -- 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