https://bugzilla.redhat.com/show_bug.cgi?id=1427182
Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |POST Flags| |fedora-review+
--- Comment #4 from Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
Summary should probably mention that this is suse-related. Maybe something like "Summary: Package management library used by OpenSUSE and ...".
Not a requirement, but I still think it'd be a good idea.
libzypp-devel-doc.noarch: E: devel-dependency libzypp-devel Is this dep necessary?
libzypp-devel-doc contains only html pages. It should be OK to drop this dependency (and add %doc %dir /usr/share/doc/libzypp).
[ ]: Package requires other packages for directories it uses. Note: No known owner of /usr/share/doc/libzypp [ ]: Package must own all directories that it creates. Note: Directories without known owners: /usr/share/doc/libzypp, /etc/logrotate.d, /usr/libexec/zypp
zypp-tools.x86_64: E: non-executable-script /usr/libexec/zypp/notify-message 644 /bin/bash
Copying from previous comments: + package name is OK + license is acceptable (GPLv2+) + license is specified correctly + scriptlets appear correct + builds and installs OK + latest version + R/P/BR appear correct and continuing + %check is present and passes
fedora-review finds no other issues.
Package is APPROVED.