selftest: replace global with explicit environment variables
authorJamie McClymont <jamiemcclymont@catalyst.net.nz>
Fri, 8 Dec 2017 02:20:36 +0000 (15:20 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 20 Dec 2017 07:50:25 +0000 (08:50 +0100)
commit676261fa08273114b888bb46f65de3de091b615b
tree38a8acd1c5996e321ce6bc6c87e0cb5a97c4c6f7
parentaf005fcc845ae3cd233dcb929fe774f38aaf38d8
selftest: replace global with explicit environment variables

This patch removes setting of NSS_WRAPPER and RESOLV_WRAPPER variables globally
in Samba3.pm (because setting them persistently/globally can create hidden
ordering dependencies). Instead, they are set on subprocesses as required, which
appears to be the following two places (aside from those places where they are
already set explicitly):
* calls to createuser in provision
* calls to wbinfo --ping-dc in wait_for_start

Signed-off-by: Jamie McClymont <jamiemcclymont@catalyst.net.nz>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Wed Dec 20 08:50:26 CET 2017 on sn-devel-144
selftest/target/Samba3.pm