{"thread":{"id":"42381","subject":"[PATCH v1] travis-ci: enable web server tests t55xx on Linux","startedAt":"2016-05-19T08:45:11Z","lastAt":"2016-07-10T07:12:54Z","messageCount":4,"participants":["larsxschneider@gmail.com","Lars Schneider","Junio C Hamano","Johannes Schindelin"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"286966","messageId":"1463647511-54811-1-git-send-email-larsxschneider@gmail.com","threadId":"42381","inReplyTo":null,"subject":"[PATCH v1] travis-ci: enable web server tests t55xx on Linux","fromName":"","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-05-19T08:45:11Z","receivedAt":"2016-05-19T08:45:11Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"From: Lars Schneider \n\nInstall the \"apache\" package to run the Git web server tests on\nTravis-CI Linux build machines. The tests are already executed on OS X\nbuild machines since the apache web server is installed by default.\n\nSigned-off-by: Lars Schneider \n---\n .travis.yml | 2 ++\n 1 file changed, 2 insertions(+)\n\ndiff --git a/.travis.yml b/.travis.yml\nindex adab5b8..3c54080 100644\n--- a/.travis.yml\n+++ b/.travis.yml\n@@ -18,6 +18,7 @@ addons:\n apt:\n packages:\n - language-pack-is\n+ - apache2\n\n env:\n global:\n@@ -30,6 +31,7 @@ env:\n - DEFAULT_TEST_TARGET=prove\n - GIT_PROVE_OPTS=\"--timer --jobs 3 --state=failed,slow,save\"\n - GIT_TEST_OPTS=\"--verbose --tee\"\n+ - GIT_TEST_HTTPD=true\n - GIT_TEST_CLONE_2GB=YesPlease\n # t9810 occasionally fails on Travis CI OS X\n # t9816 occasionally fails with \"TAP out of sequence errors\" on Travis CI OS X\n--\n2.5.1\n"},{"id":"289630","messageId":"6C0F6649-C238-475E-BAA5-5F3A0EC5DA2D@gmail.com","threadId":"42381","inReplyTo":"1463647511-54811-1-git-send-email-larsxschneider@gmail.com","subject":"Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-06-19T17:05:04Z","receivedAt":"2016-06-19T17:05:13Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 19 May 2016, at 10:45, larsxschneider@gmail.com wrote:\n> \n> From: Lars Schneider \n> \n> Install the \"apache\" package to run the Git web server tests on\n> Travis-CI Linux build machines. The tests are already executed on OS X\n> build machines since the apache web server is installed by default.\n> \n> Signed-off-by: Lars Schneider \n> ---\n> .travis.yml | 2 ++\n> 1 file changed, 2 insertions(+)\n> \n> diff --git a/.travis.yml b/.travis.yml\n> index adab5b8..3c54080 100644\n> --- a/.travis.yml\n> +++ b/.travis.yml\n> @@ -18,6 +18,7 @@ addons:\n> apt:\n> packages:\n> - language-pack-is\n> + - apache2\n> \n> env:\n> global:\n> @@ -30,6 +31,7 @@ env:\n> - DEFAULT_TEST_TARGET=prove\n> - GIT_PROVE_OPTS=\"--timer --jobs 3 --state=failed,slow,save\"\n> - GIT_TEST_OPTS=\"--verbose --tee\"\n> + - GIT_TEST_HTTPD=true\n> - GIT_TEST_CLONE_2GB=YesPlease\n> # t9810 occasionally fails on Travis CI OS X\n> # t9816 occasionally fails with \"TAP out of sequence errors\" on Travis CI OS X\n> --\n> 2.5.1\n> \n\nHi Junio,\n\nis there any chance to pick this up? It would enable the web server\ntests on Travis CI.\n\nThanks,\nLars\n\n"},{"id":"289637","messageId":"xmqqd1ndvxqf.fsf@gitster.mtv.corp.google.com","threadId":"42381","inReplyTo":"6C0F6649-C238-475E-BAA5-5F3A0EC5DA2D@gmail.com","subject":"Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-06-19T18:29:12Z","receivedAt":"2016-06-19T18:29:19Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Schneider writes:\n\n> Hi Junio,\n>\n> is there any chance to pick this up? It would enable the web server\n> tests on Travis CI.\n\nThere always are chances unless a patch was rejected with some\nreason, like \"no, I do not like it\" ;-)\n\nI do not see anybody commenting on it, and suspect that it simply\ndid not register in anybody's radar.\n\nFolks who actually read the Travis report of their own commits, do\nyou have any comment (I just reproduced it from the original so that\nwe can easily apply it when we agree it is a good thing to do)?\n\nThanks.\n\n-- >8 --\nFrom: Lars Schneider \nDate: Thu, 19 May 2016 10:45:11 +0200\nSubject: [PATCH v1] travis-ci: enable web server tests t55xx on Linux\n\nInstall the \"apache\" package to run the Git web server tests on\nTravis-CI Linux build machines. The tests are already executed on OS X\nbuild machines since the apache web server is installed by default.\n\nSigned-off-by: Lars Schneider \n---\n .travis.yml | 2 ++\n 1 file changed, 2 insertions(+)\n\ndiff --git a/.travis.yml b/.travis.yml\nindex adab5b8..3c54080 100644\n--- a/.travis.yml\n+++ b/.travis.yml\n@@ -18,6 +18,7 @@ addons:\n apt:\n packages:\n - language-pack-is\n+ - apache2\n\n env:\n global:\n@@ -30,6 +31,7 @@ env:\n - DEFAULT_TEST_TARGET=prove\n - GIT_PROVE_OPTS=\"--timer --jobs 3 --state=failed,slow,save\"\n - GIT_TEST_OPTS=\"--verbose --tee\"\n+ - GIT_TEST_HTTPD=true\n - GIT_TEST_CLONE_2GB=YesPlease\n # t9810 occasionally fails on Travis CI OS X\n # t9816 occasionally fails with \"TAP out of sequence errors\" on Travis CI OS X\n--\n2.5.1\n\n"},{"id":"291123","messageId":"alpine.DEB.2.20.1607100911090.6426@virtualbox","threadId":"42381","inReplyTo":"xmqqd1ndvxqf.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1] travis-ci: enable web server tests t55xx on Linux","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2016-07-10T07:12:36Z","receivedAt":"2016-07-10T07:12:54Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Sun, 19 Jun 2016, Junio C Hamano wrote:\n\n> Lars Schneider writes:\n> \n> > Hi Junio,\n> >\n> > is there any chance to pick this up? It would enable the web server\n> > tests on Travis CI.\n> \n> There always are chances unless a patch was rejected with some\n> reason, like \"no, I do not like it\" ;-)\n> \n> I do not see anybody commenting on it, and suspect that it simply\n> did not register in anybody's radar.\n> \n> Folks who actually read the Travis report of their own commits, do\n> you have any comment (I just reproduced it from the original so that\n> we can easily apply it when we agree it is a good thing to do)?\n\nAs long as we're not running the danger of timing out, I am all in favor\nof including the tests.\n\nI exercise Travis quite a bit these days, also because it catches more\nbugs than I can catch locally.\n\nCiao,\nDscho\n"}]}