On 19 May 2015 12:42, Pádraig Brady wrote: > * tests/cp/no-ctx.sh: Scope of `var=val func` is inconsistent > across shells, so avoid that construct with functions.
i think POSIX requires the behavior dash exhibits, and bash will do the same when it's in POSIX compliant mode. > -LD_PRELOAD=./k.so returns_ 1 cp --preserve=context file_src file_dst || > fail=1 > +( LD_PRELOAD=./k.so; export LD_PRELOAD; there is no need to split these into two statements. if you're using `checkbashisms`, that script is outdated and should be taken with a grain of salt. > --- a/tests/df/no-mtab-status.sh > +++ b/tests/df/no-mtab-status.sh > @@ -88,34 +88,38 @@ EOF > gcc_shared_ k.c k.so \ > || framework_failure_ 'failed to build shared library' > > +cleanup_() { unset LD_PRELOAD; } if you're just going to unset it, why not do just this ? the rest of the changes need not be made. -mike
signature.asc
Description: Digital signature