Packages plugin doesn't check dependencies properly

[David Strauss <[email protected]…>]

Currently, the Packages plugin considers "rsyslog" a dependency of "vsftpd" on RHEL5. This is incorrect because the syslog dependency is already satisfied by a different package.

Edited excerpt of the debugging output ([...] indicates omission):

loop.pluginsPackages?.complete(m, vsftpd?, debug=True)

Package vsftpd: adding new deps ['/bin/bash', '/lib64/security/pam_loginuid.so',

'/usr/sbin/usermod', 'rpmlib(CompressedFileNames?)', 'logrotate', '/sbin/chkconf

ig', 'libpam.so.0(LIBPAM_1.0)(64bit)', 'libwrap.so.0()(64bit)', 'libcrypto.so.6( )(64bit)', 'libpam.so.0()(64bit)', 'libdl.so.2()(64bit)', 'config(vsftpd)', 'lib c.so.6(GLIBC_2.2.5)(64bit)', '/bin/sh', 'libcap', 'pam', 'libc.so.6(GLIBC_2.3)(6 4bit)', '/sbin/service', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libnsl.so.1()(64bit)' , 'rtld(GNU_HASH)', 'libssl.so.6()(64bit)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpml ib(PayloadFilesHavePrefix?)', 'libc.so.6()(64bit)', 'openssl', 'libcap.so.1()(64b it)'] [...] Package openssl: adding new deps ['libgssapi_krb5.so.2()(64bit)', 'rpmlib(Compre ssedFileNames)', 'mktemp', 'libdl.so.2(GLIBC_2.2.5)(64bit)', 'libcrypto.so.6()(6 4bit)', 'libdl.so.2()(64bit)', 'libcom_err.so.2()(64bit)', 'libc.so.6(GLIBC_2.2. 5)(64bit)', '/bin/sh', 'libc.so.6(GLIBC_2.3)(64bit)', 'libk5crypto.so.3()(64bit) ', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libk5crypto.so.3(k5crypto_3_MIT)(64bit)', ' config(openssl)', 'libkrb5.so.3(krb5_3_MIT)(64bit)', 'libz.so.1()(64bit)', 'rtld (GNU_HASH)', 'libc.so.6(GLIBC_2.4)(64bit)', 'libssl.so.6()(64bit)', 'rpmlib(Payl oadFilesHavePrefix)', 'libc.so.6()(64bit)', '/sbin/ldconfig', 'libkrb5.so.3()(64 bit)'] [...] Package krb5-libs: adding new deps ['libresolv.so.2(GLIBC_2.2.5)(64bit)', 'rtld( GNU_HASH)', 'libkeyutils.so.1()(64bit)', 'libresolv.so.2()(64bit)', 'libkrb5supp ort.so.0()(64bit)', 'rpmlib(CompressedFileNames?)', 'libkadm5clnt.so.5()(64bit)',

'libdl.so.2(GLIBC_2.2.5)(64bit)', 'libdl.so.2()(64bit)', 'libc.so.6(GLIBC_2.2.5

)(64bit)', 'libgssapi_krb5.so.2(gssapi_krb5_2_MIT)(64bit)', 'libc.so.6(GLIBC_2.3 )(64bit)', 'libkrb4.so.2()(64bit)', 'libgssrpc.so.4(gssrpc_4_MIT)(64bit)', 'libk db5.so.4()(64bit)', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libdes425.so.3()(64bit)', 'libkadm5srv.so.5()(64bit)', 'libdes425.so.3(des425_3_MIT)(64bit)', 'libkdb5.so. 4(kdb5_4_MIT)(64bit)', 'libkrb5.so.3()(64bit)', 'sh-utils', 'grep', 'config(krb5 -libs)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpmlib(PayloadFilesHavePrefix?)', 'libc. so.6()(64bit)', '/sbin/ldconfig', 'libkeyutils.so.1(KEYUTILS_0.3)(64bit)', 'libk rb5support.so.0(krb5support_0_MIT)(64bit)', 'libgssrpc.so.4()(64bit)'] [...] VPackage config(openssl): got provides openssl? Package initscripts: adding new deps ['/bin/bash', 'udev', 'rpmlib(CompressedFil? eNames)', '/etc/redhat-release', '/sbin/chkconfig', '/bin/sed', 'mingetty', 'lib glib-2.0.so.0()(64bit)', '/sbin/ip', 'libc.so.6(GLIBC_2.2.5)(64bit)', 'coreutils ', 'config(initscripts)', '/bin/sh', '/bin/grep', 'ethtool', 'libc.so.6(GLIBC_2. 3)(64bit)', '/sbin/runuser', '/sbin/fuser', '/sbin/sysctl', '/bin/awk', 'libc.so .6(GLIBC_2.3.4)(64bit)', 'SysVinit?', '/sbin/arping', 'util-linux', 'rtld(GNU_HAS H)', 'libc.so.6(GLIBC_2.4)(64bit)', 'syslog', 'rpmlib(PayloadFilesHavePrefix?)', 'libc.so.6()(64bit)', 'mount', '/usr/sbin/groupadd', 'dev', 'e2fsprogs', 'module -init-tools', 'net-tools'] [...] VPackage syslog: got provides rsyslog? Package udev: adding new deps ['rtld(GNU_HASH)', 'libc.so.6(GLIBC_2.3)(64bit)', '/bin/sh', 'libc.so.6(GLIBC_2.4)(64bit)', '/bin/bash', 'rpmlib(PayloadFilesHaveP refix)', 'libc.so.6()(64bit)', 'libsepol.so.1()(64bit)', 'libc.so.6(GLIBC_2.2.5) (64bit)', 'rpmlib(CompressedFileNames?)', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'sed',

'rpmlib(VersionedDependencies?)', 'fileutils', 'config(udev)', 'MAKEDEV']

VPackage /etc/redhat-release: got provides redhat-release? [...] Package rsyslog: adding new deps ['config(rsyslog)', '/bin/bash', 'rpmlib(Compre ssedFileNames)', '/sbin/chkconfig', '/sbin/service', 'libdl.so.2(GLIBC_2.2.5)(64 bit)', 'libdl.so.2()(64bit)', 'libc.so.6(GLIBC_2.2.5)(64bit)', '/bin/sh', 'librt .so.1()(64bit)', 'libc.so.6(GLIBC_2.3)(64bit)', 'libc.so.6(GLIBC_2.3.4)(64bit)',

'libpthread.so.0(GLIBC_2.3.2)(64bit)', 'librt.so.1(GLIBC_2.2.5)(64bit)', 'rtld(

GNU_HASH)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpmlib(PayloadFilesHavePrefix?)', 'li bc.so.6()(64bit)', 'libpthread.so.0()(64bit)'] VPackage config(rsyslog): got provides rsyslog? [...] Package filesystem: adding new deps ['rpmlib(VersionedDependencies?)', 'rpmlib(Pa yloadFilesHavePrefix)', 'rpmlib(CompressedFileNames?)'] (set(['termcap', 'iputils', 'audit-libs', 'udev', 'python', 'libstdc++', 'iproute', 'logrotate', 'sed', 'libgcc', 'findutils', 'rsyslog', 'popt', 'gdbm', 'chkconfig', 'net-tools', 'mktemp', 'tzdata', 'mingetty', 'libacl', 'keyutils-libs', 'glibc', 'device-mapper', 'shadow-utils', 'coreutils', 'bash', 'libcap', 'psmisc', 'pam', 'bzip2-libs', 'cracklib', 'ethtool', 'pcre', 'mcstrans', 'cracklib-dicts', 'ncurses', 'tcp_wrappers', 'gawk', 'SysVinit?', 'readline', 'util-linux', 'MAKEDEV', 'db4', 'basesystem', 'glibc-common', 'info', 'glib2', 'grep', 'vsftpd', 'setup', 'zlib', 'openssl', 'e2fsprogs', 'procps', 'krb5-libs', 'initscripts', 'libtermcap', 'module-init-tools', 'libsepol', 'filesystem', 'e2fsprogs-libs', 'libselinux', 'redhat-release', 'libattr']), set([]), 'yum')

[David Strauss <…>]

Formatted as code:

{{{>>> loop.pluginsPackages?.complete(m, vsftpd?, debug=True) Package vsftpd: adding new deps ['/bin/bash', '/lib64/security/pam_loginuid.so',

'/usr/sbin/usermod', 'rpmlib(CompressedFileNames?)', 'logrotate', '/sbin/chkconf

ig', 'libpam.so.0(LIBPAM_1.0)(64bit)', 'libwrap.so.0()(64bit)', 'libcrypto.so.6( )(64bit)', 'libpam.so.0()(64bit)', 'libdl.so.2()(64bit)', 'config(vsftpd)', 'lib c.so.6(GLIBC_2.2.5)(64bit)', '/bin/sh', 'libcap', 'pam', 'libc.so.6(GLIBC_2.3)(6 4bit)', '/sbin/service', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libnsl.so.1()(64bit)' , 'rtld(GNU_HASH)', 'libssl.so.6()(64bit)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpml ib(PayloadFilesHavePrefix?)', 'libc.so.6()(64bit)', 'openssl', 'libcap.so.1()(64b it)'] [...] Package openssl: adding new deps ['libgssapi_krb5.so.2()(64bit)', 'rpmlib(Compre ssedFileNames)', 'mktemp', 'libdl.so.2(GLIBC_2.2.5)(64bit)', 'libcrypto.so.6()(6 4bit)', 'libdl.so.2()(64bit)', 'libcom_err.so.2()(64bit)', 'libc.so.6(GLIBC_2.2. 5)(64bit)', '/bin/sh', 'libc.so.6(GLIBC_2.3)(64bit)', 'libk5crypto.so.3()(64bit) ', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libk5crypto.so.3(k5crypto_3_MIT)(64bit)', ' config(openssl)', 'libkrb5.so.3(krb5_3_MIT)(64bit)', 'libz.so.1()(64bit)', 'rtld (GNU_HASH)', 'libc.so.6(GLIBC_2.4)(64bit)', 'libssl.so.6()(64bit)', 'rpmlib(Payl oadFilesHavePrefix)', 'libc.so.6()(64bit)', '/sbin/ldconfig', 'libkrb5.so.3()(64 bit)'] [...] Package krb5-libs: adding new deps ['libresolv.so.2(GLIBC_2.2.5)(64bit)', 'rtld( GNU_HASH)', 'libkeyutils.so.1()(64bit)', 'libresolv.so.2()(64bit)', 'libkrb5supp ort.so.0()(64bit)', 'rpmlib(CompressedFileNames?)', 'libkadm5clnt.so.5()(64bit)',

'libdl.so.2(GLIBC_2.2.5)(64bit)', 'libdl.so.2()(64bit)', 'libc.so.6(GLIBC_2.2.5

)(64bit)', 'libgssapi_krb5.so.2(gssapi_krb5_2_MIT)(64bit)', 'libc.so.6(GLIBC_2.3 )(64bit)', 'libkrb4.so.2()(64bit)', 'libgssrpc.so.4(gssrpc_4_MIT)(64bit)', 'libk db5.so.4()(64bit)', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'libdes425.so.3()(64bit)', 'libkadm5srv.so.5()(64bit)', 'libdes425.so.3(des425_3_MIT)(64bit)', 'libkdb5.so. 4(kdb5_4_MIT)(64bit)', 'libkrb5.so.3()(64bit)', 'sh-utils', 'grep', 'config(krb5 -libs)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpmlib(PayloadFilesHavePrefix?)', 'libc. so.6()(64bit)', '/sbin/ldconfig', 'libkeyutils.so.1(KEYUTILS_0.3)(64bit)', 'libk rb5support.so.0(krb5support_0_MIT)(64bit)', 'libgssrpc.so.4()(64bit)'] [...] VPackage config(openssl): got provides openssl? Package initscripts: adding new deps ['/bin/bash', 'udev', 'rpmlib(CompressedFil? eNames)', '/etc/redhat-release', '/sbin/chkconfig', '/bin/sed', 'mingetty', 'lib glib-2.0.so.0()(64bit)', '/sbin/ip', 'libc.so.6(GLIBC_2.2.5)(64bit)', 'coreutils ', 'config(initscripts)', '/bin/sh', '/bin/grep', 'ethtool', 'libc.so.6(GLIBC_2. 3)(64bit)', '/sbin/runuser', '/sbin/fuser', '/sbin/sysctl', '/bin/awk', 'libc.so .6(GLIBC_2.3.4)(64bit)', 'SysVinit?', '/sbin/arping', 'util-linux', 'rtld(GNU_HAS H)', 'libc.so.6(GLIBC_2.4)(64bit)', 'syslog', 'rpmlib(PayloadFilesHavePrefix?)', 'libc.so.6()(64bit)', 'mount', '/usr/sbin/groupadd', 'dev', 'e2fsprogs', 'module -init-tools', 'net-tools'] [...] VPackage syslog: got provides rsyslog? Package udev: adding new deps ['rtld(GNU_HASH)', 'libc.so.6(GLIBC_2.3)(64bit)', '/bin/sh', 'libc.so.6(GLIBC_2.4)(64bit)', '/bin/bash', 'rpmlib(PayloadFilesHaveP refix)', 'libc.so.6()(64bit)', 'libsepol.so.1()(64bit)', 'libc.so.6(GLIBC_2.2.5) (64bit)', 'rpmlib(CompressedFileNames?)', 'libc.so.6(GLIBC_2.3.4)(64bit)', 'sed',

'rpmlib(VersionedDependencies?)', 'fileutils', 'config(udev)', 'MAKEDEV']

VPackage /etc/redhat-release: got provides redhat-release? [...] Package rsyslog: adding new deps ['config(rsyslog)', '/bin/bash', 'rpmlib(Compre ssedFileNames)', '/sbin/chkconfig', '/sbin/service', 'libdl.so.2(GLIBC_2.2.5)(64 bit)', 'libdl.so.2()(64bit)', 'libc.so.6(GLIBC_2.2.5)(64bit)', '/bin/sh', 'librt .so.1()(64bit)', 'libc.so.6(GLIBC_2.3)(64bit)', 'libc.so.6(GLIBC_2.3.4)(64bit)',

'libpthread.so.0(GLIBC_2.3.2)(64bit)', 'librt.so.1(GLIBC_2.2.5)(64bit)', 'rtld(

GNU_HASH)', 'libc.so.6(GLIBC_2.4)(64bit)', 'rpmlib(PayloadFilesHavePrefix?)', 'li bc.so.6()(64bit)', 'libpthread.so.0()(64bit)'] VPackage config(rsyslog): got provides rsyslog? [...] Package filesystem: adding new deps ['rpmlib(VersionedDependencies?)', 'rpmlib(Pa yloadFilesHavePrefix)', 'rpmlib(CompressedFileNames?)'] (set(['termcap', 'iputils', 'audit-libs', 'udev', 'python', 'libstdc++', 'iproute', 'logrotate', 'sed', 'libgcc', 'findutils', 'rsyslog', 'popt', 'gdbm', 'chkconfig', 'net-tools', 'mktemp', 'tzdata', 'mingetty', 'libacl', 'keyutils-libs', 'glibc', 'device-mapper', 'shadow-utils', 'coreutils', 'bash', 'libcap', 'psmisc', 'pam', 'bzip2-libs', 'cracklib', 'ethtool', 'pcre', 'mcstrans', 'cracklib-dicts', 'ncurses', 'tcp_wrappers', 'gawk', 'SysVinit?', 'readline', 'util-linux', 'MAKEDEV', 'db4', 'basesystem', 'glibc-common', 'info', 'glib2', 'grep', 'vsftpd', 'setup', 'zlib', 'openssl', 'e2fsprogs', 'procps', 'krb5-libs', 'initscripts', 'libtermcap', 'module-init-tools', 'libsepol', 'filesystem', 'e2fsprogs-libs', 'libselinux', 'redhat-release', 'libattr']), set([]), 'yum')

}}}

[David Strauss <…>]

I give up. Here it is in pastebin: ​http://pastebin.com/f7fd732b2

[David Strauss <…>]

I hear this may be fixed in trunk.

[David Strauss <…>]

This is not fixed in trunk.

[desai]

We've now verified that this is properly fixed in trunk. (Doc issues remain, but the correctness issue has been rectified)