# [PATCH v1] travis-ci: enable web server tests t55xx on Linux 4 messages from 2016-05-19 to 2016-07-10. Participants: larsxschneider@gmail.com, Lars Schneider, Junio C Hamano, Johannes Schindelin. Thread: https://gitlist.dev/t/42381 ## larsxschneider@gmail.com, 2016-05-19 08:45 Subject: [PATCH v1] travis-ci: enable web server tests t55xx on Linux Message-ID: <1463647511-54811-1-git-send-email-larsxschneider@gmail.com> URL: https://gitlist.dev/e/1463647511-54811-1-git-send-email-larsxschneider%40gmail.com ``` From: Lars Schneider 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 --- .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 ``` ## Lars Schneider, 2016-06-19 17:05 Subject: Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux Message-ID: <6C0F6649-C238-475E-BAA5-5F3A0EC5DA2D@gmail.com> URL: https://gitlist.dev/e/6C0F6649-C238-475E-BAA5-5F3A0EC5DA2D%40gmail.com In-Reply-To: <1463647511-54811-1-git-send-email-larsxschneider@gmail.com> ``` > On 19 May 2016, at 10:45, larsxschneider@gmail.com wrote: > > From: Lars Schneider > > 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 > --- > .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, 2016-06-19 18:29 Subject: Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux Message-ID: URL: https://gitlist.dev/e/xmqqd1ndvxqf.fsf%40gitster.mtv.corp.google.com In-Reply-To: <6C0F6649-C238-475E-BAA5-5F3A0EC5DA2D@gmail.com> ``` Lars Schneider 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 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 --- .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 ``` ## Johannes Schindelin, 2016-07-10 07:12 Subject: Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux Message-ID: URL: https://gitlist.dev/e/alpine.DEB.2.20.1607100911090.6426%40virtualbox In-Reply-To: ``` Hi, On Sun, 19 Jun 2016, Junio C Hamano wrote: > Lars Schneider 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 ```