Hi Pino, On 04/02/2015 05:23 AM, Pino Toscano wrote:
Hi Daniel,
The one that got upstream does not work in ibm-powerkvm due to the rpm_is_avaiable verification in the detection (I've attached the wrong version in bugzilla).
On Wednesday 01 April 2015 16:37:26 Daniel Henrique Barboza wrote: the new version of the patch is somehow confusing. supermin >= 5.1.12 uses librpm to query for rpm dependencies, file listing, provides, etc. If rpm_is_available returns false, that means you built without librpm, and that supermin will not really work. Did you tried running the test suite (`make check`)?
These are the results of make check in the system I've used to test the patch. The system is in an internal isolated network, thus I believe some failures were expected to happen. PASS: test-basic.sh PASS: test-execstack.sh FAIL: test-build-bash.sh FAIL: test-binaries-exist.sh SKIP: test-harder.sh FAIL: test-build-bash-network.sh FAIL: test-binaries-exist-network.sh SKIP: test-harder-network.sh make[4]: Entering directory `/root/supermin/tests' make[4]: Nothing to be done for `all'. make[4]: Leaving directory `/root/supermin/tests' ============================================================================ Testsuite summary for supermin 5.1.12 ============================================================================ # TOTAL: 8 # PASS: 2 # SKIP: 2 # XFAIL: 0 # FAIL: 4 # XPASS: 0 # ERROR: 0 I've run the non-related network tests to see the failure cause. It is worth saying that supermin upstream code builds and runs fine as far as I can tell, thus I couldn't figure it out much reading these errors: # ./test-build-bash.sh ./test-build-bash.sh: line 34: 83872 Aborted ../src/supermin -v --prepare $USE_INSTALLED bash -o $d1 # ./test-binaries-exist.sh ./test-binaries-exist.sh: line 29: 83886 Aborted ../src/supermin -v --prepare $USE_INSTALLED bash coreutils -o $d1 I will be honest and say that I didn't dig further understanding why the rpm_is_available check fails in ibm-powerkvm. The patch I sent is similar to an internal patch we used to add temporary support to the distro on a older version of supermin (the version shipped with RHEL 7.1 GA). I assumed that rpm_is_available failed because ibm-powerkvm does not have all the rpm features fedora and rhel have, although it uses rpm/yum. Perhaps in a later release of the OS, using a newer version of supermin, we should review this code and enhance it. Daniel