Skip to content

Commit 573ce03

Browse files
committed
Merge branch 'sg/travis-build-during-script-phase' into maint
Build the executable in 'script' phase in Travis CI integration, to follow the established practice, rather than during 'before_script' phase. This allows the CI categorize the failures better ('failed' is project's fault, 'errored' is build environment's). * sg/travis-build-during-script-phase: travis-ci: build Git during the 'script' phase
2 parents 38e79b1 + 3c93b82 commit 573ce03

File tree

3 files changed

+4
-15
lines changed

3 files changed

+4
-15
lines changed

.travis.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ matrix:
3333
compiler:
3434
addons:
3535
before_install:
36-
before_script:
3736
script:
3837
- >
3938
test "$TRAVIS_REPO_SLUG" != "git/git" ||
@@ -46,7 +45,6 @@ matrix:
4645
services:
4746
- docker
4847
before_install:
49-
before_script:
5048
script: ci/run-linux32-docker.sh
5149
- env: jobname=StaticAnalysis
5250
os: linux
@@ -56,7 +54,6 @@ matrix:
5654
packages:
5755
- coccinelle
5856
before_install:
59-
before_script:
6057
script: ci/run-static-analysis.sh
6158
after_failure:
6259
- env: jobname=Documentation
@@ -68,13 +65,11 @@ matrix:
6865
- asciidoc
6966
- xmlto
7067
before_install:
71-
before_script:
7268
script: ci/test-documentation.sh
7369
after_failure:
7470

7571
before_install: ci/install-dependencies.sh
76-
before_script: ci/run-build.sh
77-
script: ci/run-tests.sh
72+
script: ci/run-build-and-tests.sh
7873
after_failure: ci/print-test-failures.sh
7974

8075
notifications:

ci/run-tests.sh renamed to ci/run-build-and-tests.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
#!/bin/sh
22
#
3-
# Test Git
3+
# Build and test Git
44
#
55

66
. ${0%/*}/lib-travisci.sh
77

88
ln -s $HOME/travis-cache/.prove t/.prove
9+
10+
make --jobs=2
911
make --quiet test
1012

1113
check_unignored_build_artifacts

ci/run-build.sh

Lines changed: 0 additions & 8 deletions
This file was deleted.

0 commit comments

Comments
 (0)