diff options
| author | Chris Marusich <cmmarusich@gmail.com> | 2020-05-30 00:20:48 -0700 |
|---|---|---|
| committer | Jim Meyering <meyering@fb.com> | 2022-02-13 11:08:03 -0800 |
| commit | b8f98f7dc3638384a6cd378c78f0a788fbf33b34 (patch) | |
| tree | 3e4964bce230e50212f00a124b68ba454324d602 | |
| parent | 0c8d88bc61e284cdb9ee3866d83ddf724bf9a8e5 (diff) | |
| download | sed-b8f98f7dc3638384a6cd378c78f0a788fbf33b34.tar.gz | |
tests: export CONFIG_HEADER to test scripts
Before this change, the testsuite/inplace-selinux test fails in some
cases when SELinux is available. The reason it fails is because the
require_selinux_ function in init.cfg expects the CONFIG_HEADER
environment variable to be defined, but in fact it is not defined.
Even though we do invoke AM_CONFIG_HEADER in configure.ac, and even
though the CONFIG_HEADER variable does get set in the resulting
Makefile, nothing actually exports the CONFIG_HEADER variable, so the
test is doomed to fail whenever it is not skipped. Fix this by
exporting the variable.
* testsuite/local.mk (TESTS_ENVIRONMENT): Add CONFIG_HEADER to the
list of variables to export to the environment of the test scripts.
Reported by Vineet Jain <vkjain@gmail.com> in
https://bugs.gnu.org/36150 . The fix was suggested by Timothy Sample
<samplet@ngyro.com> in the same bug report.
| -rw-r--r-- | testsuite/local.mk | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/testsuite/local.mk b/testsuite/local.mk index 1bb0d68..5b1f837 100644 --- a/testsuite/local.mk +++ b/testsuite/local.mk @@ -166,6 +166,7 @@ TESTS_ENVIRONMENT = \ srcdir='$(srcdir)' \ top_srcdir='$(top_srcdir)' \ CC='$(CC)' \ + CONFIG_HEADER='$(CONFIG_HEADER)' \ SED_TEST_NAME=`echo $$tst|sed 's,^\./,,;s,/,-,g'` \ MAKE=$(MAKE) \ MALLOC_PERTURB_=$(MALLOC_PERTURB_) \ |
