Skip to content

Commit a1f6da9

Browse files
committed
CI tweaks:
- don't install CPAN modules if NO_TEST_FRAMEWORK is set - remove the workaround for mod_h2 APR build caching which should no longer be necessary now caching is fixed - fix capturing specific perl-framework failures with "TEST -v" mode git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1916055 13f79535-47bb-0310-9956-ffa450edef68
1 parent 3736c7a commit a1f6da9

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

test/travis_before_linux.sh

+3-7
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,6 @@ function install_apx() {
4242
# Blow away the cached install root if the cached install is stale
4343
# or doesn't match the expected configuration.
4444
grep -q "${version} ${revision} ${config} CC=$CC" ${HOME}/root/.key-${name} || rm -rf ${prefix}
45-
# TEST_H2 APR cache seems to be broken, do not use.
46-
# Unknown why this happens on this CI job only and how to fix it
47-
if test -v TEST_H2; then
48-
rm -rf ${prefix}
49-
fi
5045

5146
if test -d ${prefix}; then
5247
return 0
@@ -66,8 +61,9 @@ function install_apx() {
6661
# Allow to load $HOME/build/apache/httpd/.gdbinit
6762
echo "add-auto-load-safe-path $HOME/work/httpd/httpd/.gdbinit" >> $HOME/.gdbinit
6863

69-
# Prepare perl-framework test environment
70-
if ! test -v SKIP_TESTING; then
64+
# Unless either SKIP_TESTING or NO_TEST_FRAMEWORK are set, install
65+
# CPAN modules required to run the Perl test framework.
66+
if ! test -v SKIP_TESTING -o -v NO_TEST_FRAMEWORK; then
7167
# Clear CPAN cache if necessary
7268
if [ -v CLEAR_CACHE ]; then rm -rf ~/perl5; fi
7369

test/travis_run_linux.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ if ! test -v NO_TEST_FRAMEWORK; then
136136
FAILERS="$FAILERS $FAILER"
137137
done < <(awk '/Failed:/{print $1}' test.log)
138138
if [ -n "$FAILERS" ]; then
139-
t/TEST -v $FAILERS || true
139+
make check TESTS="-v $FAILERS" || true
140140
fi
141141
# set -e would have killed us after the original t/TEST
142142
rm -f test.log

0 commit comments

Comments
 (0)