diff options
author | Emmanuel Dreyfus <manu@netbsd.org> | 2015-03-19 12:05:16 +0100 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-03-30 23:45:10 -0700 |
commit | ffb2e85ff574891639d899cc59fcd9f75d4ce51e (patch) | |
tree | f261ca1e87f9f96671a91c5eb3781cccb929e333 /tests/features/ipc.t | |
parent | d3eacb0d83834db485061d875d95b4f6af41f30a (diff) |
Tests: portability fixes for ipc.t
This fixes portability problems in ipc.t so that it can run on NetBSD:
1) EOPNOTSUPP value is OS-dependent. Learn it from system headers
instead of hard-coding it in the script
2) liglusterfs embbeds its own UUID implementation. The function name
may be the same as in built(in implementation from libc, but with
different prototype. In that case, we must make sure python will
use libglusterfs's version, otherwise we will crash in libc's UUID
code. Since dlopen() does not make any guarantee on what symbol
will be used, me need to preload libglusterfs when loading python.
This is done using LD_PRELOAD.
3) In python code we need to load with RTLD_GLOBAL global in order
to have dependencies loaded
4) Python's ctypes.util.find_library does not lookup LD_LIBRARy_PATH
and may therefore miss the library. On failure, retry with less
portable but more reliable explicit name
BUG: 1129939
Change-Id: I024cdfd03a5a42a8ec23de38a99e7349aba92ea8
Signed-off-by: Emmanuel Dreyfus <manu@netbsd.org>
Reviewed-on: http://review.gluster.org/9944
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'tests/features/ipc.t')
-rwxr-xr-x | tests/features/ipc.t | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/tests/features/ipc.t b/tests/features/ipc.t index 2aaca6620bf..55ce8c871c1 100755 --- a/tests/features/ipc.t +++ b/tests/features/ipc.t @@ -13,9 +13,23 @@ TEST $CLI volume info; TEST $CLI volume create $V0 $H0:$B0/1 TEST $CLI volume start $V0 +# Find OS-dependent EOPNOTSUPP value from system headers +EOPNOTSUPP=$( echo '#include <errno.h>\\EOPNOTSUPP\\' | tr '\\' '\n' | \ + cc -E -c - | tail -1 ) + +# liglusterfs embbeds its own UUID implementation. The function name +# may be the same as in built(in implementation from libc, but with +# different prototype. In that case, we must make sure python will +# use libglusterfs's version, and dlopen() does not make any guarantee +# on this. By preloading libglusterfs.so before launching python, we +# ensure libglusterfs's UUID functions will be used. +LD_PRELOAD=${prefix}/lib/libglusterfs.so +export LD_PRELOAD + # This is a pretty lame test. Basically we just want to make sure that we # get all the way through the translator stacks on client and server to get a -# simple error (95 = EOPNOTUPP) instead of a crash, RPC error, etc. -EXPECT 95 $PYTHON $(dirname $0)/ipctest.py $H0 $V0 +# simple error (EOPNOTSUPP) instead of a crash, RPC error, etc. +EXPECT ${EOPNOTSUPP} $PYTHON $(dirname $0)/ipctest.py $H0 $V0 +unset LD_PRELOAD cleanup; |