[Bug 1427182] Review Request: libzypp - A package management library

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

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |zbyszek@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |zbyszek@xxxxxxxxx



--- Comment #1 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
Peachy! I was just wondering why systemd-nspawn(1) does not carry a opensuse
example. If zypper is packaged, I'll submit a patch to add that.

+ 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

BuildRequires should be specified one-per-line, for legibility and diffability.

Summary should probably mention that this is suse-related. Maybe something
like "Summary: Package management library used by OpenSUSE and ...".

libudev-devel → systemd-devel

%make_build in %tests should be moved to %build.

fedora-review finds the following minor issues:
[ ]: Package requires other packages for directories it uses.
     Note: No known owner of /usr/share/doc/libzypp
(Remove "/*" from "%doc %{_docdir}/libzypp/*".)

[ ]: Package must own all directories that it creates.
     Note: Directories without known owners: /usr/share/doc/libzypp,
     /etc/logrotate.d
(I think you should add %dir /etc/logrotate.d to %files.)

Rpmlint
-------
Checking: libzypp-16.4.3-1.fc27.x86_64.rpm
          libzypp-devel-16.4.3-1.fc27.x86_64.rpm
          libzypp-devel-doc-16.4.3-1.fc27.noarch.rpm
          libzypp-debuginfo-16.4.3-1.fc27.x86_64.rpm
          libzypp-16.4.3-1.fc27.src.rpm
libzypp.x86_64: W: spelling-error %description -l en_US libsolv -> absolve
OK.

libzypp.x86_64: E: missing-call-to-chdir-with-chroot
/usr/lib64/libzypp.so.1600.4.3
Most likely false positive.

libzypp.x86_64: E: incoherent-logrotate-file /etc/logrotate.d/zypp-history.lr
Looks OK to me.

libzypp.x86_64: E: non-executable-script /usr/lib/zypp/notify-message 644
/bin/bash
chmod +x ?

libzypp.x86_64: W: dangling-symlink /etc/zypp/repos.d /etc/yum.repos.d
False positive.

libzypp.x86_64: W: hidden-file-or-dir /usr/lib/.build-id
libzypp.x86_64: W: hidden-file-or-dir /usr/lib/.build-id
OK

libzypp-devel.x86_64: W: only-non-binary-in-usr-lib
OK

libzypp-devel.x86_64: W: no-documentation
OK

libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SATResolver.h
libzypp-devel.x86_64: E: incorrect-fsf-address /usr/include/zypp/ProblemTypes.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/ProblemSolutionIgnore.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItem.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItemInstallOneOf.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItemUpdate.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItemLock.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItemInstall.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/ProblemSolutionCombi.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/SolverQueueItemDelete.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/Types.h
libzypp-devel.x86_64: E: incorrect-fsf-address
/usr/include/zypp/solver/detail/Resolver.h
Who cares ;)

libzypp-devel-doc.noarch: E: devel-dependency libzypp-devel
Is this dep necessary?

libzypp-debuginfo.x86_64: W: invalid-url URL:
https://en.opensuse.org/Portal:Libzypp HTTP Error 500: Internal Server Error
Works fine here.

libzypp.src:99: E: hardcoded-library-path in %{_prefix}/lib/zypp
libzypp.src:100: E: hardcoded-library-path in %{_prefix}/lib/zypp/plugins
libzypp.src:101: E: hardcoded-library-path in
%{_prefix}/lib/zypp/plugins/appdata
libzypp.src:102: E: hardcoded-library-path in
%{_prefix}/lib/zypp/plugins/commit
libzypp.src:103: E: hardcoded-library-path in
%{_prefix}/lib/zypp/plugins/services
libzypp.src:104: E: hardcoded-library-path in
%{_prefix}/lib/zypp/plugins/system
libzypp.src:105: E: hardcoded-library-path in
%{_prefix}/lib/zypp/plugins/urlresolver
libzypp.src:136: E: hardcoded-library-path in %{_prefix}/lib/zypp/
Dunno, might be OK. This probably breaks parallel installation of 32 and 64 bit
versions.

5 packages and 0 specfiles checked; 49 errors, 8 warnings.

Looks all good. The issues are very minor, but let's do another round, since
there's quite a few of them.

-- 
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




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]