Skip to content

Travis CI cleanups

SZEDER Gábor git 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest8 patches 11 repliesIn review
  2. v14 patches 7 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8travis-ci: use 'set -x' in select 'ci/*' scripts for extra tracingci/lib-travisci.sh ci/print-test-failures.sh ci/run-linux32-build.sh+9 -22 replies
2/8travis-ci: introduce a $jobname variable for 'ci/*' scripts.travis.yml ci/install-dependencies.sh ci/lib-travisci.sh+15 -100 replies
3/8travis-ci: move setting environment variables to 'ci/lib-travisci.sh'.travis.yml ci/lib-travisci.sh+22 -170 replies
4/8travis-ci: set GIT_TEST_HTTPD in 'ci/lib-travisci.sh'ci/install-dependencies.sh ci/lib-travisci.sh+2 -20 replies
5/8travis-ci: don't install default addon packages for the 32 bit Linux build.travis.yml+1 -00 replies
6/8travis-ci: don't install 'language-pack-is' package.travis.yml+0 -15 replies
7/8travis-ci: save prove state for the 32 bit Linux buildci/run-linux32-build.sh ci/run-linux32-docker.sh+2 -00 replies
8/8travis-ci: only print test failures if there are test results availableci/print-test-failures.sh+6 -03 replies

Reviews trailers per patch

None of the 8 patches has a Reviewed-by or Acked-by.

PatchSZEDER Gábor
Seriesigned-off-by
1/8
2/8
3/8
4/8
5/8
6/8
7/8
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/8] Travis CI cleanups

This is a reroll of 'sg/travis-fixes'.

Changes since the previous round:

- Patch 1 got updated following the discussion:

- I went with enabling tracing executed commands everywhere, including the Windows build job, except where we know it causes way too much clutter, which is currently only 'ci/print-test-failures.sh'.

- Also enable this tracing in 'ci/run-linux32-build.sh', which doesn't source 'ci/lib-travisci.sh' as it's run inside a Docker container.

- The commit message got updated accordingly, including a note about the Windows build job's secret token.
I would like to get an Acked-by: from Dscho on this patch before it gets merged.

- Patches 5-8 are new. They are various fixes/cleanups unrelated to the Travis CI scriptification, but I don't think it's worth to have them in separate patch series.

SZEDER Gábor (8):
travis-ci: use 'set -x' in select 'ci/*' scripts for extra tracing
travis-ci: introduce a $jobname variable for 'ci/*' scripts
travis-ci: move setting environment variables to 'ci/lib-travisci.sh'
travis-ci: set GIT_TEST_HTTPD in 'ci/lib-travisci.sh'
travis-ci: don't install default addon packages for the 32 bit Linux
build
travis-ci: don't install 'language-pack-is' package
travis-ci: save prove state for the 32 bit Linux build
travis-ci: only print test failures if there are test results
available

 .travis.yml                | 28 ++++++----------------------
 ci/install-dependencies.sh |  8 +++-----
 ci/lib-travisci.sh         | 38 ++++++++++++++++++++++++++++++++++----
 ci/print-test-failures.sh  |  9 +++++++++
 ci/run-linux32-build.sh    |  3 +++
 ci/run-linux32-docker.sh   |  1 +
 6 files changed, 56 insertions(+), 31 deletions(-)
Signature · 1 line
2.15.1.429.ga000dd9c7