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

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

[larsxschneider]

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

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

[Lars Schneider]

> 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

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

[Junio C Hamano]

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(+)

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

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

[Johannes Schindelin]

Hi,

On Sun, 19 Jun 2016, Junio C Hamano wrote:

> 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