threads / patch / 42381

patchtravis-ci: enable web server tests t55xx on Linux

Subject: [PATCH v1] travis-ci: enable web server tests t55xx on Linux

## tl;dr

4 messages between May 19, 2016 and Jul 10, 2016. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

larsxschneider@gmail.com· May 19, 2016, 08:45 UTC · lore
From: Lars Schneider <larsxschneider@gmail.com>

Install the "apache" package to run the Git web server tests on Travis-CI Linux build machines. The tests are already executed on OS X build machines since the apache web server is installed by default.

Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
---
 .travis.yml | 2 ++
 1 file changed, 2 insertions(+)
Show changes to .travis.yml +2 −1
diff --git a/.travis.yml b/.travis.yml
index adab5b8..3c54080 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -18,6 +18,7 @@ addons:
   apt:
     packages:
     - language-pack-is
+    - apache2

 env:
   global:
@@ -30,6 +31,7 @@ env:
     - DEFAULT_TEST_TARGET=prove
     - GIT_PROVE_OPTS="--timer --jobs 3 --state=failed,slow,save"
     - GIT_TEST_OPTS="--verbose --tee"
+    - GIT_TEST_HTTPD=true
     - GIT_TEST_CLONE_2GB=YesPlease
     # t9810 occasionally fails on Travis CI OS X
     # t9816 occasionally fails with "TAP out of sequence errors" on Travis CI OS X
--
2.5.1
Lars Schneider· Jun 19, 2016, 17:05 UTC · re: larsxschneider@gmail.com · lore

Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux

Show 36 quoted lines
> On 19 May 2016, at 10:45, larsxschneider@gmail.com wrote:
> 
> From: Lars Schneider <larsxschneider@gmail.com>
> 
> Install the "apache" package to run the Git web server tests on
> Travis-CI Linux build machines. The tests are already executed on OS X
> build machines since the apache web server is installed by default.
> 
> Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
> ---
> .travis.yml | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/.travis.yml b/.travis.yml
> index adab5b8..3c54080 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -18,6 +18,7 @@ addons:
>   apt:
>     packages:
>     - language-pack-is
> +    - apache2
> 
> env:
>   global:
> @@ -30,6 +31,7 @@ env:
>     - DEFAULT_TEST_TARGET=prove
>     - GIT_PROVE_OPTS="--timer --jobs 3 --state=failed,slow,save"
>     - GIT_TEST_OPTS="--verbose --tee"
> +    - GIT_TEST_HTTPD=true
>     - GIT_TEST_CLONE_2GB=YesPlease
>     # t9810 occasionally fails on Travis CI OS X
>     # t9816 occasionally fails with "TAP out of sequence errors" on Travis CI OS X
> --
> 2.5.1
> 
Hi Junio,

is there any chance to pick this up? It would enable the web server tests on Travis CI.

Thanks, Lars

Junio C Hamano· Jun 19, 2016, 18:29 UTC · re: Lars Schneider · lore

Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux

Lars Schneider <larsxschneider@gmail.com> writes:
> Hi Junio,
>
> is there any chance to pick this up? It would enable the web server
> tests on Travis CI.

There always are chances unless a patch was rejected with some reason, like "no, I do not like it" ;-)

I do not see anybody commenting on it, and suspect that it simply did not register in anybody's radar.

Folks who actually read the Travis report of their own commits, do you have any comment (I just reproduced it from the original so that we can easily apply it when we agree it is a good thing to do)?

Thanks.
-- >8 --
From: Lars Schneider <larsxschneider@gmail.com>
Date: Thu, 19 May 2016 10:45:11 +0200
Subject: [PATCH v1] travis-ci: enable web server tests t55xx on Linux

Install the "apache" package to run the Git web server tests on Travis-CI Linux build machines. The tests are already executed on OS X build machines since the apache web server is installed by default.

Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
---
 .travis.yml | 2 ++
 1 file changed, 2 insertions(+)
Show changes to .travis.yml +2 −1
diff --git a/.travis.yml b/.travis.yml
index adab5b8..3c54080 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -18,6 +18,7 @@ addons:
   apt:
     packages:
     - language-pack-is
+    - apache2

 env:
   global:
@@ -30,6 +31,7 @@ env:
     - DEFAULT_TEST_TARGET=prove
     - GIT_PROVE_OPTS="--timer --jobs 3 --state=failed,slow,save"
     - GIT_TEST_OPTS="--verbose --tee"
+    - GIT_TEST_HTTPD=true
     - GIT_TEST_CLONE_2GB=YesPlease
     # t9810 occasionally fails on Travis CI OS X
     # t9816 occasionally fails with "TAP out of sequence errors" on Travis CI OS X
--
2.5.1
Johannes Schindelin· Jul 10, 2016, 07:12 UTC · re: Junio C Hamano · lore

Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux

Hi,
On Sun, 19 Jun 2016, Junio C Hamano wrote:
Show 16 quoted lines
> Lars Schneider <larsxschneider@gmail.com> writes:
> 
> > Hi Junio,
> >
> > is there any chance to pick this up? It would enable the web server
> > tests on Travis CI.
> 
> There always are chances unless a patch was rejected with some
> reason, like "no, I do not like it" ;-)
> 
> I do not see anybody commenting on it, and suspect that it simply
> did not register in anybody's radar.
> 
> Folks who actually read the Travis report of their own commits, do
> you have any comment (I just reproduced it from the original so that
> we can easily apply it when we agree it is a good thing to do)?

As long as we're not running the danger of timing out, I am all in favor of including the tests.

I exercise Travis quite a bit these days, also because it catches more bugs than I can catch locally.

Ciao, Dscho

← back to recent threads