[Freeipa-devel] [PATCH] 0235 tests: Use ipa-getkeytab from /usr/sbin instead of the in-tree one
Simo Sorce
simo at redhat.com
Tue Jun 4 12:53:23 UTC 2013
On Tue, 2013-06-04 at 13:48 +0200, Petr Viktorin wrote:
> Hardcoding the in-tree location for ipa-getkeytab makes testing outside
> the source tree impossible. This patch makes the tests use the installed
> location.
>
> In other places the test suite assumes IPA is installed system-wide,
> even if running from the source tree.
> I know I frequently forget to run `make` before testing, which makes the
> ipa-getkeytab tests fail. So this patch would work well for me (and
> probably other Python devs), but I guess others might be used to `make
> test` checking what `make` built.
>
> C developers, are you OK with e.g. adding `cp ipa-client/ipa-getkeytab
> /usr/sbin/ipa-getkeytab` to your testing workflow?
Absolutely not.
> Or should this be made configurable (or auto-detected)?
You must not break a machine just to do make test.
I often do make test, then make rpms and install rpms, I *never*
directly install on my development machine or VMs, I always go through
RPM in order to keep the system clean, and tests repeatable.
ipa-getkeytab specifically do not need root to be tested so I really do
not see that copying over a system path would ever be a good idea.
Simo.
--
Simo Sorce * Red Hat, Inc * New York
More information about the Freeipa-devel
mailing list