{"thread":{"id":"44311","subject":"[PATCH v1 0/2] Fix default macOS build locally and on Travis CI","startedAt":"2016-10-17T00:26:01Z","lastAt":"2016-11-16T20:02:47Z","messageCount":35,"participants":["larsxschneider@gmail.com","Jeff King","Lars Schneider","Paul Smith","Torsten Bögershausen","Junio C Hamano","Heiko Voigt"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"304251","messageId":"20161017002550.88782-1-larsxschneider@gmail.com","threadId":"44311","inReplyTo":null,"subject":"[PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-10-17T00:25:48Z","receivedAt":"2016-10-17T00:26:01Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"From: Lars Schneider \n\nApple removed the OpenSSL header files in macOS and therefore Git does\nnot build out of the box on macOS anymore. See previous discussion with\nTorsten here: http://public-inbox.org/git/565B3036.8000604@web.de/\n\nThis mini series makes Git build out of the box on macOS, again, and\ndisables the HTTPD tests on macOS TravisCI as they don't work anymore\nwith the new macOS TravisCI default image:\nhttps://blog.travis-ci.com/2016-10-04-osx-73-default-image-live/\n\nThanks,\nLars\n\n\nLars Schneider (2):\n config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11\n travis-ci: disable GIT_TEST_HTTPD for macOS\n\n .travis.yml | 3 ++-\n config.mak.uname | 6 ++++++\n 2 files changed, 8 insertions(+), 1 deletion(-)\n\n-- \n2.10.0\n\n"},{"id":"304252","messageId":"20161017002550.88782-2-larsxschneider@gmail.com","threadId":"44311","inReplyTo":"20161017002550.88782-1-larsxschneider@gmail.com","subject":"[PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-10-17T00:25:49Z","receivedAt":"2016-10-17T00:26:06Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"From: Lars Schneider \n\nApple removed the OpenSSL header files in macOS 10.11 and above. OpenSSL\nwas deprecated since macOS 10.7.\n\nSet `NO_OPENSSL` and `APPLE_COMMON_CRYPTO` to `YesPlease` as default for\nmacOS. Make it possible to override this and use OpenSSL by defining\n`DARWIN_OPENSSL`.\n\nOriginal-patch-by: Torsten Bögershausen \nSigned-off-by: Lars Schneider \n---\n config.mak.uname | 6 ++++++\n 1 file changed, 6 insertions(+)\n\ndiff --git a/config.mak.uname b/config.mak.uname\nindex b232908..f0c94a9 100644\n--- a/config.mak.uname\n+++ b/config.mak.uname\n@@ -108,6 +108,12 @@ ifeq ($(uname_S),Darwin)\n \tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 11 && echo 1),1)\n \t\tHAVE_GETDELIM = YesPlease\n \tendif\n+\tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 15 && echo 1),1)\n+\t\tifndef DARWIN_OPENSSL\n+\t\t\tNO_OPENSSL = YesPlease\n+\t\t\tAPPLE_COMMON_CRYPTO=YesPlease\n+\t\tendif\n+\tendif\n \tNO_MEMMEM = YesPlease\n \tUSE_ST_TIMESPEC = YesPlease\n \tHAVE_DEV_TTY = YesPlease\n-- \n2.10.0\n\n"},{"id":"304253","messageId":"20161017002550.88782-3-larsxschneider@gmail.com","threadId":"44311","inReplyTo":"20161017002550.88782-1-larsxschneider@gmail.com","subject":"[PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-10-17T00:25:50Z","receivedAt":"2016-10-17T00:26:10Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"From: Lars Schneider \n\nTravisCI changed their default macOS image from 10.10 to 10.11 [1].\nUnfortunately the HTTPD tests do not run out of the box using the\npre-installed Apache web server anymore. Therefore we enable these\ntests only for Linux and disable them for macOS.\n\n[1] https://blog.travis-ci.com/2016-10-04-osx-73-default-image-live/\n\nSigned-off-by: Lars Schneider \n---\n .travis.yml | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/.travis.yml b/.travis.yml\nindex 37a1e1f..d752447 100644\n--- a/.travis.yml\n+++ b/.travis.yml\n@@ -32,7 +32,6 @@ 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@@ -57,6 +56,8 @@ before_install:\n - >\n case \"${TRAVIS_OS_NAME:-linux}\" in\n linux)\n+ export GIT_TEST_HTTPD=YesPlease\n+\n mkdir --parents custom/p4\n pushd custom/p4\n wget --quiet http://filehost.perforce.com/perforce/r$LINUX_P4_VERSION/bin.linux26x86_64/p4d\n-- \n2.10.0\n\n"},{"id":"304269","messageId":"20161017095002.bbqyp2hkbuyau66t@sigill.intra.peff.net","threadId":"44311","inReplyTo":"20161017002550.88782-2-larsxschneider@gmail.com","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-10-17T09:50:02Z","receivedAt":"2016-10-17T09:57:17Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Oct 16, 2016 at 05:25:49PM -0700, larsxschneider@gmail.com wrote:\n\n> From: Lars Schneider \n> \n> Apple removed the OpenSSL header files in macOS 10.11 and above. OpenSSL\n> was deprecated since macOS 10.7.\n> \n> Set `NO_OPENSSL` and `APPLE_COMMON_CRYPTO` to `YesPlease` as default for\n> macOS. Make it possible to override this and use OpenSSL by defining\n> `DARWIN_OPENSSL`.\n\nI like that you gave an override, but I don't think it works in all\ncases:\n\n> diff --git a/config.mak.uname b/config.mak.uname\n> index b232908..f0c94a9 100644\n> --- a/config.mak.uname\n> +++ b/config.mak.uname\n> @@ -108,6 +108,12 @@ ifeq ($(uname_S),Darwin)\n> \tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 11 && echo 1),1)\n> \t\tHAVE_GETDELIM = YesPlease\n> \tendif\n> +\tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 15 && echo 1),1)\n> +\t\tifndef DARWIN_OPENSSL\n> +\t\t\tNO_OPENSSL = YesPlease\n> +\t\t\tAPPLE_COMMON_CRYPTO=YesPlease\n> +\t\tendif\n> +\tendif\n\nThis is in config.mak.uname, which gets sourced before config.mak (and\nifndef is evaluated at the time of parsing). So it would work to do:\n\n make DARWIN_OPENSSL=Yep\n\nbut not:\n\n echo DARWIN_OPENSSL=Yep >>config.mak\n make\n\nI think you'd have to set a flag in config.mak.uname, and then resolve\nit in the Makefile proper like:\n\n ifdef DARWIN_OPENSSL\n\t# Overrides AUTO_AVOID_OPENSSL, do nothing.\n else ifdef AUTO_AVOID_OPENSSL\n\tNO_OPENSSL = YesPlease\n\tAPPLE_COMMON_CRYPTO = YesPlease\n endif\n\nbut that's totally untested.\n\n-Peff\n"},{"id":"305467","messageId":"8C67FF53-C26F-4993-908F-A5183C5E48D9@gmail.com","threadId":"44311","inReplyTo":"20161017095002.bbqyp2hkbuyau66t@sigill.intra.peff.net","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-06T19:35:04Z","receivedAt":"2016-11-06T19:50:21Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 17 Oct 2016, at 11:50, Jeff King wrote:\n> \n> On Sun, Oct 16, 2016 at 05:25:49PM -0700, larsxschneider@gmail.com wrote:\n> \n>> From: Lars Schneider \n>> \n>> Apple removed the OpenSSL header files in macOS 10.11 and above. OpenSSL\n>> was deprecated since macOS 10.7.\n>> \n>> Set `NO_OPENSSL` and `APPLE_COMMON_CRYPTO` to `YesPlease` as default for\n>> macOS. Make it possible to override this and use OpenSSL by defining\n>> `DARWIN_OPENSSL`.\n> \n> I like that you gave an override, but I don't think it works in all\n> cases:\n> \n>> diff --git a/config.mak.uname b/config.mak.uname\n>> index b232908..f0c94a9 100644\n>> --- a/config.mak.uname\n>> +++ b/config.mak.uname\n>> @@ -108,6 +108,12 @@ ifeq ($(uname_S),Darwin)\n>> \tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 11 && echo 1),1)\n>> \t\tHAVE_GETDELIM = YesPlease\n>> \tendif\n>> +\tifeq ($(shell test \"`expr \"$(uname_R)\" : '\\([0-9][0-9]*\\)\\.'`\" -ge 15 && echo 1),1)\n>> +\t\tifndef DARWIN_OPENSSL\n>> +\t\t\tNO_OPENSSL = YesPlease\n>> +\t\t\tAPPLE_COMMON_CRYPTO=YesPlease\n>> +\t\tendif\n>> +\tendif\n> \n> This is in config.mak.uname, which gets sourced before config.mak (and\n> ifndef is evaluated at the time of parsing). So it would work to do:\n> \n> make DARWIN_OPENSSL=Yep\n> \n> but not:\n> \n> echo DARWIN_OPENSSL=Yep >>config.mak\n> make\n> \n> I think you'd have to set a flag in config.mak.uname, and then resolve\n> it in the Makefile proper like:\n> \n> ifdef DARWIN_OPENSSL\n> \t# Overrides AUTO_AVOID_OPENSSL, do nothing.\n> else ifdef AUTO_AVOID_OPENSSL\n> \tNO_OPENSSL = YesPlease\n> \tAPPLE_COMMON_CRYPTO = YesPlease\n> endif\n> \n> but that's totally untested.\n\nGood point. I think I found an even easier way to achieve the same.\nWhat do you think about the patch below?\n\nThanks,\nLars\n\n-- >8 --\nSubject: Makefile: set NO_OPENSSL on macOS by default\n\nApple removed the OpenSSL header files in macOS 10.11 and above. OpenSSL\nwas deprecated since macOS 10.7.\n\nSet `NO_OPENSSL` and `APPLE_COMMON_CRYPTO` to `YesPlease` as default for\nmacOS. It is possible to override this and use OpenSSL by defining\n`NO_APPLE_COMMON_CRYPTO`.\n\nOriginal-patch-by: Torsten Bögershausen \nSigned-off-by: Lars Schneider \n---\n Makefile | 1 +\n 1 file changed, 1 insertion(+)\n\ndiff --git a/Makefile b/Makefile\nindex 9d6c245..f53fcc9 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n \t\tendif\n \tendif\n \tifndef NO_APPLE_COMMON_CRYPTO\n+\t\tNO_OPENSSL = YesPlease\n \t\tAPPLE_COMMON_CRYPTO = YesPlease\n \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n \tendif\n-- "},{"id":"305471","messageId":"203BDCB2-1975-4590-B4B8-3C5E9D210430@gmail.com","threadId":"44311","inReplyTo":"20161017002550.88782-3-larsxschneider@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-06T21:42:36Z","receivedAt":"2016-11-06T21:43:26Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 17 Oct 2016, at 02:25, larsxschneider@gmail.com wrote:\n> \n> From: Lars Schneider \n> \n> TravisCI changed their default macOS image from 10.10 to 10.11 [1].\n> Unfortunately the HTTPD tests do not run out of the box using the\n> pre-installed Apache web server anymore. Therefore we enable these\n> tests only for Linux and disable them for macOS.\n> \n> [1] https://blog.travis-ci.com/2016-10-04-osx-73-default-image-live/\n> \n> Signed-off-by: Lars Schneider \n> ---\n> .travis.yml | 3 ++-\n> 1 file changed, 2 insertions(+), 1 deletion(-)\n> \n> diff --git a/.travis.yml b/.travis.yml\n> index 37a1e1f..d752447 100644\n> --- a/.travis.yml\n> +++ b/.travis.yml\n> @@ -32,7 +32,6 @@ 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> @@ -57,6 +56,8 @@ before_install:\n> - >\n> case \"${TRAVIS_OS_NAME:-linux}\" in\n> linux)\n> + export GIT_TEST_HTTPD=YesPlease\n> +\n> mkdir --parents custom/p4\n> pushd custom/p4\n> wget --quiet http://filehost.perforce.com/perforce/r$LINUX_P4_VERSION/bin.linux26x86_64/p4d\n> -- \n> 2.10.0\n> \n\nHi Junio,\n\nthe patch above is one of two patches to make TravisCI pass, again.\nCould you queue it?\n\nThe other patch is [1] which is still under discussion.\n\nThank you,\nLars\n\n[1] http://public-inbox.org/git/8C67FF53-C26F-4993-908F-A5183C5E48D9@gmail.com/"},{"id":"305489","messageId":"20161107172617.tlcrpwbjy2w7aoyc@sigill.intra.peff.net","threadId":"44311","inReplyTo":"8C67FF53-C26F-4993-908F-A5183C5E48D9@gmail.com","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-07T17:26:17Z","receivedAt":"2016-11-07T17:26:24Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Nov 06, 2016 at 08:35:04PM +0100, Lars Schneider wrote:\n\n> Good point. I think I found an even easier way to achieve the same.\n> What do you think about the patch below?\n>\n> [...]\n>\n> diff --git a/Makefile b/Makefile\n> index 9d6c245..f53fcc9 100644\n> --- a/Makefile\n> +++ b/Makefile\n> @@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n> \t\tendif\n> \tendif\n> \tifndef NO_APPLE_COMMON_CRYPTO\n> +\t\tNO_OPENSSL = YesPlease\n> \t\tAPPLE_COMMON_CRYPTO = YesPlease\n> \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n> \tendif\n\nThat is much simpler.\n\nI have in the back of my mind a fear that it is harder to unset a make\nvariable than it is to override it with a new value (which is what you'd\nwant to do here to turn openssl back on), but I can't seem to come up\nwith a case that doesn't work. So I am probably misremembering, or just\nthinking of something that used to be a problem long ago.\n\n-Peff\n"},{"id":"305491","messageId":"1478540194.4171.19.camel@mad-scientist.net","threadId":"44311","inReplyTo":"20161107172617.tlcrpwbjy2w7aoyc@sigill.intra.peff.net","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Paul Smith","fromEmail":"paul@mad-scientist.net","sentAt":"2016-11-07T17:36:34Z","receivedAt":"2016-11-07T17:38:52Z","isPatch":true,"sender":{"key":"paul@mad-scientist.net","avatar":"https://avatars.githubusercontent.com/u/109636?v=4"},"body":"On Mon, 2016-11-07 at 12:26 -0500, Jeff King wrote:\n> I have in the back of my mind a fear that it is harder to unset a\n> make variable than it is to override it with a new value (which is\n> what you'd want to do here to turn openssl back on),\n\nIt depends on what you mean by \"unset\".\n\nIf you mean it as per the shell \"unset\" command, where the variable is\ncompletely forgotten as if it never was set at all, that's tricky.  You\nhave to use the \"undefine\" special command which was introduced in GNU\nmake 3.82 (released in 2010).\n\nBut if you just want to set the variable to the empty string, using\n\"FOO=\" works fine for that in all versions of make (GNU and otherwise)\nand using all the normal rules (command line override, etc.)\n\nIt's not easy to distinguish between a variable that is empty and one\nthat is actually not defined, in make, so it's a difference without a\ndistinction in almost all situations.\n"},{"id":"305492","messageId":"20161107174559.t72vxxkckqdbxmbg@sigill.intra.peff.net","threadId":"44311","inReplyTo":"1478540194.4171.19.camel@mad-scientist.net","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-07T17:46:00Z","receivedAt":"2016-11-07T17:46:08Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Nov 07, 2016 at 12:36:34PM -0500, Paul Smith wrote:\n\n> On Mon, 2016-11-07 at 12:26 -0500, Jeff King wrote:\n> > I have in the back of my mind a fear that it is harder to unset a\n> > make variable than it is to override it with a new value (which is\n> > what you'd want to do here to turn openssl back on),\n> \n> It depends on what you mean by \"unset\".\n> \n> If you mean it as per the shell \"unset\" command, where the variable is\n> completely forgotten as if it never was set at all, that's tricky.  You\n> have to use the \"undefine\" special command which was introduced in GNU\n> make 3.82 (released in 2010).\n> \n> But if you just want to set the variable to the empty string, using\n> \"FOO=\" works fine for that in all versions of make (GNU and otherwise)\n> and using all the normal rules (command line override, etc.)\n\nSpecifically I wanted to make sure that\n\n FOO = bar\n FOO =\n ifdef FOO\n ... something ...\n endif\n\nworks as if FOO had never been set in the first place. Which it seems\nto, at least in GNU make (and that is the only one we support, for other\nreasons).\n\n-Peff\n"},{"id":"305493","messageId":"1478540973.4171.22.camel@mad-scientist.net","threadId":"44311","inReplyTo":"20161107174559.t72vxxkckqdbxmbg@sigill.intra.peff.net","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Paul Smith","fromEmail":"paul@mad-scientist.net","sentAt":"2016-11-07T17:49:33Z","receivedAt":"2016-11-07T17:49:47Z","isPatch":true,"sender":{"key":"paul@mad-scientist.net","avatar":"https://avatars.githubusercontent.com/u/109636?v=4"},"body":"On Mon, 2016-11-07 at 12:46 -0500, Jeff King wrote:\n> Specifically I wanted to make sure that\n> \n>   FOO = bar\n>   FOO =\n>   ifdef FOO\n>   ... something ...\n>   endif\n> \n> works as if FOO had never been set in the first place. Which it seems\n> to, at least in GNU make (and that is the only one we support, for\n> other reasons).\n\nYes, it will work.  Confusingly, \"ifdef\" actually tests whether the\nvariable has a non-empty value, not whether it's defined:\n\n> The 'ifdef' form takes the _name_ of a variable as its argument, not\n> a reference to a variable.  The value of that variable has a non-\n> empty value, the TEXT-IF-TRUE is effective; otherwise, the TEXT-IF-\n> FALSE, if any, is effective\n\n*sigh* History...\n"},{"id":"305516","messageId":"20161107212004.x4y7bcl2p4chfkm6@sigill.intra.peff.net","threadId":"44311","inReplyTo":"203BDCB2-1975-4590-B4B8-3C5E9D210430@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-07T21:20:04Z","receivedAt":"2016-11-07T21:20:10Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Nov 06, 2016 at 10:42:36PM +0100, Lars Schneider wrote:\n\n> > From: Lars Schneider \n> > \n> > TravisCI changed their default macOS image from 10.10 to 10.11 [1].\n> > Unfortunately the HTTPD tests do not run out of the box using the\n> > pre-installed Apache web server anymore. Therefore we enable these\n> > tests only for Linux and disable them for macOS.\n> [...]\n> Hi Junio,\n> \n> the patch above is one of two patches to make TravisCI pass, again.\n> Could you queue it?\n\nI don't really mind disabling tests if they don't run on a platform. But\nthe more interesting question to me is: why don't they run any more? Is\nthere some config tweak needed, or is it an insurmountable problem?\n\nUsing Apache in the tests has been the source of frequent portability\nproblems and configuration headaches. I do wonder if we'd be better off\nusing some small special-purpose web server (even a short perl script\nwritten around HTTP::Server::Simple or something).\n\nOn the other hand, testing against Apache approximates a more real-world\ncase, which has value. It might be nice if our tests supported multiple\nweb servers, but that would mean duplicating the config for each\nmanually.\n\n-Peff\n"},{"id":"305617","messageId":"11cc8bbd-2e67-f53a-c8f4-2244409fd6af@web.de","threadId":"44311","inReplyTo":"20161107172617.tlcrpwbjy2w7aoyc@sigill.intra.peff.net","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2016-11-09T08:18:12Z","receivedAt":"2016-11-09T08:18:51Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On 07.11.16 18:26, Jeff King wrote:\n> On Sun, Nov 06, 2016 at 08:35:04PM +0100, Lars Schneider wrote:\n> \n>> Good point. I think I found an even easier way to achieve the same.\n>> What do you think about the patch below?\n>>\n>> [...]\n>>\n>> diff --git a/Makefile b/Makefile\n>> index 9d6c245..f53fcc9 100644\n>> --- a/Makefile\n>> +++ b/Makefile\n>> @@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n>> \t\tendif\n>> \tendif\n>> \tifndef NO_APPLE_COMMON_CRYPTO\n>> +\t\tNO_OPENSSL = YesPlease\n>> \t\tAPPLE_COMMON_CRYPTO = YesPlease\n>> \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n>> \tendif\n> \n> That is much simpler.\n[]\nI don't know if that is a correct solution.\n\nIf I have Mac OS 10.12 and Mac Ports installed, I may want to use\nOPENSSL from Mac Ports.\nHow about this:\n\n\ndiff --git a/Makefile b/Makefile\nindex ee89c06..e93511f 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -1038,17 +1038,22 @@ ifeq ($(uname_S),Darwin)\n ifeq ($(shell test -d /sw/lib && echo y),y)\n BASIC_CFLAGS += -I/sw/include\n BASIC_LDFLAGS += -L/sw/lib\n+ HAS_OPENSSL = Yes\n endif\n endif\n ifndef NO_DARWIN_PORTS\n ifeq ($(shell test -d /opt/local/lib && echo y),y)\n BASIC_CFLAGS += -I/opt/local/include\n BASIC_LDFLAGS += -L/opt/local/lib\n+ HAS_OPENSSL = Yes\n endif\n endif\n ifndef NO_APPLE_COMMON_CRYPTO\n APPLE_COMMON_CRYPTO = YesPlease\n COMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n+ ifndef HAS_OPENSSL\n+ NO_OPENSSL = YesPlease\n+ endif\n endif\n NO_REGEX = YesPlease\n PTHREAD_LIBS =\n\n"},{"id":"305622","messageId":"0966CBE3-2F08-4B89-9716-4EEE3CE2526E@gmail.com","threadId":"44311","inReplyTo":"11cc8bbd-2e67-f53a-c8f4-2244409fd6af@web.de","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-09T09:29:26Z","receivedAt":"2016-11-09T09:29:35Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 09 Nov 2016, at 09:18, Torsten Bögershausen wrote:\n> \n> On 07.11.16 18:26, Jeff King wrote:\n>> On Sun, Nov 06, 2016 at 08:35:04PM +0100, Lars Schneider wrote:\n>> \n>>> Good point. I think I found an even easier way to achieve the same.\n>>> What do you think about the patch below?\n>>> \n>>> [...]\n>>> \n>>> diff --git a/Makefile b/Makefile\n>>> index 9d6c245..f53fcc9 100644\n>>> --- a/Makefile\n>>> +++ b/Makefile\n>>> @@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n>>> \t\tendif\n>>> \tendif\n>>> \tifndef NO_APPLE_COMMON_CRYPTO\n>>> +\t\tNO_OPENSSL = YesPlease\n>>> \t\tAPPLE_COMMON_CRYPTO = YesPlease\n>>> \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n>>> \tendif\n>> \n>> That is much simpler.\n> []\n> I don't know if that is a correct solution.\n> \n> If I have Mac OS 10.12 and Mac Ports installed, I may want to use\n> OPENSSL from Mac Ports.\n\nCan't you define `NO_APPLE_COMMON_CRYPTO` in that case? \nI think if you use OpenSSL then you don't need the Apple crypto lib, right?\n\n- Lars\n\n\n> How about this:\n> \n> \n> diff --git a/Makefile b/Makefile\n> index ee89c06..e93511f 100644\n> --- a/Makefile\n> +++ b/Makefile\n> @@ -1038,17 +1038,22 @@ ifeq ($(uname_S),Darwin)\n> ifeq ($(shell test -d /sw/lib && echo y),y)\n> BASIC_CFLAGS += -I/sw/include\n> BASIC_LDFLAGS += -L/sw/lib\n> + HAS_OPENSSL = Yes\n> endif\n> endif\n> ifndef NO_DARWIN_PORTS\n> ifeq ($(shell test -d /opt/local/lib && echo y),y)\n> BASIC_CFLAGS += -I/opt/local/include\n> BASIC_LDFLAGS += -L/opt/local/lib\n> + HAS_OPENSSL = Yes\n> endif\n> endif\n> ifndef NO_APPLE_COMMON_CRYPTO\n> APPLE_COMMON_CRYPTO = YesPlease\n> COMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n> + ifndef HAS_OPENSSL\n> + NO_OPENSSL = YesPlease\n> + endif\n> endif\n> NO_REGEX = YesPlease\n> PTHREAD_LIBS =\n> \n\n"},{"id":"305625","messageId":"1fdc5333-79a8-7c1d-f7e5-e8150d0ad9cc@web.de","threadId":"44311","inReplyTo":"0966CBE3-2F08-4B89-9716-4EEE3CE2526E@gmail.com","subject":"Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2016-11-09T10:51:07Z","receivedAt":"2016-11-09T10:58:26Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On 09.11.16 10:29, Lars Schneider wrote:\n> \n>> On 09 Nov 2016, at 09:18, Torsten Bögershausen wrote:\n>>\n>> On 07.11.16 18:26, Jeff King wrote:\n>>> On Sun, Nov 06, 2016 at 08:35:04PM +0100, Lars Schneider wrote:\n>>>\n>>>> Good point. I think I found an even easier way to achieve the same.\n>>>> What do you think about the patch below?\n>>>>\n>>>> [...]\n>>>>\n>>>> diff --git a/Makefile b/Makefile\n>>>> index 9d6c245..f53fcc9 100644\n>>>> --- a/Makefile\n>>>> +++ b/Makefile\n>>>> @@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n>>>> \t\tendif\n>>>> \tendif\n>>>> \tifndef NO_APPLE_COMMON_CRYPTO\n>>>> +\t\tNO_OPENSSL = YesPlease\n>>>> \t\tAPPLE_COMMON_CRYPTO = YesPlease\n>>>> \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n>>>> \tendif\n>>>\n>>> That is much simpler.\n>> []\n>> I don't know if that is a correct solution.\n>>\n>> If I have Mac OS 10.12 and Mac Ports installed, I may want to use\n>> OPENSSL from Mac Ports.\n> \n> Can't you define `NO_APPLE_COMMON_CRYPTO` in that case? \n> I think if you use OpenSSL then you don't need the Apple crypto lib, right?\n\nAfter re-reading the Makefile: that makes sense :-)\n\nDo you want to send a new patch ?\n\nFeel free to omit\n\"Original-patch-by: Torsten Bögershausen \"\n\n\n\n\n"},{"id":"305661","messageId":"xmqqwpgc6x4t.fsf@gitster.mtv.corp.google.com","threadId":"44311","inReplyTo":"20161017002550.88782-1-larsxschneider@gmail.com","subject":"Re: [PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-11-09T23:39:46Z","receivedAt":"2016-11-09T23:39:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"larsxschneider@gmail.com writes:\n\n> From: Lars Schneider \n>\n> Apple removed the OpenSSL header files in macOS and therefore Git does\n> not build out of the box on macOS anymore. See previous discussion with\n> Torsten here: http://public-inbox.org/git/565B3036.8000604@web.de/\n>\n> This mini series makes Git build out of the box on macOS, again, and\n> disables the HTTPD tests on macOS TravisCI as they don't work anymore\n> with the new macOS TravisCI default image:\n> https://blog.travis-ci.com/2016-10-04-osx-73-default-image-live/\n>\n> Thanks,\n> Lars\n>\n>\n> Lars Schneider (2):\n> config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11\n> travis-ci: disable GIT_TEST_HTTPD for macOS\n>\n> .travis.yml | 3 ++-\n> config.mak.uname | 6 ++++++\n> 2 files changed, 8 insertions(+), 1 deletion(-)\n\nI've followed what was available at the public-inbox archive, but it\nis unclear what the conclusion was. \n\nFor the first one your \"how about\" non-patch, to which Peff said\n\"that's simple and good\", looked good to me as well, but is it\navailable as a final patch that I can just take and apply (otherwise\nI think I can do the munging myself, but I'd rather be spoon-fed\nwhen able ;-).\n\nI do not have a strong opinion on the second one. For an interim\nsolution, disabling webserver tests certainly is expedite and safe,\nso I am fine taking it as-is, but I may have missed strong\nobjections.\n\nThanks.\n\n\n"},{"id":"305667","messageId":"13e1bf7a-d404-3789-5d2a-4d6259959254@web.de","threadId":"44311","inReplyTo":"xmqqwpgc6x4t.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2016-11-10T06:41:59Z","receivedAt":"2016-11-10T06:44:23Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"\nOn 10/11/16 00:39, Junio C Hamano wrote:\n>\n> I've followed what was available at the public-inbox archive, but it\n> is unclear what the conclusion was.\n>\n> For the first one your \"how about\" non-patch, to which Peff said\n> \"that's simple and good\", looked good to me as well, but is it\n> available as a final patch that I can just take and apply (otherwise\n> I think I can do the munging myself, but I'd rather be spoon-fed\n> when able ;-).\n\nIf you can munge the Peff version that may be easiest ?\nI couldn't find a branch in your repo, but am happy to review\nwhatever shows up there and in pu.\n\n"},{"id":"305672","messageId":"CBAF806C-7E1E-4490-A07C-F98DB7488F5F@gmail.com","threadId":"44311","inReplyTo":"20161107212004.x4y7bcl2p4chfkm6@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-10T11:07:14Z","receivedAt":"2016-11-10T11:07:22Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 07 Nov 2016, at 22:20, Jeff King wrote:\n> \n> On Sun, Nov 06, 2016 at 10:42:36PM +0100, Lars Schneider wrote:\n> \n>>> From: Lars Schneider \n>>> \n>>> TravisCI changed their default macOS image from 10.10 to 10.11 [1].\n>>> Unfortunately the HTTPD tests do not run out of the box using the\n>>> pre-installed Apache web server anymore. Therefore we enable these\n>>> tests only for Linux and disable them for macOS.\n>> [...]\n>> Hi Junio,\n>> \n>> the patch above is one of two patches to make TravisCI pass, again.\n>> Could you queue it?\n> \n> I don't really mind disabling tests if they don't run on a platform. But\n> the more interesting question to me is: why don't they run any more? Is\n> there some config tweak needed, or is it an insurmountable problem?\n\nI can't really remember what the problem was. I think some apache config\nrequired some module that was not present and I wasn't able to get this\nworking quickly.\n\n\n> Using Apache in the tests has been the source of frequent portability\n> problems and configuration headaches. I do wonder if we'd be better off\n> using some small special-purpose web server (even a short perl script\n> written around HTTP::Server::Simple or something).\n> \n> On the other hand, testing against Apache approximates a more real-world\n> case, which has value. It might be nice if our tests supported multiple\n> web servers, but that would mean duplicating the config for each\n> manually.\n\nI agree that the real-world Apache test is more valuable and I really want\nto keep the Linux Apache test running. However, I don't think many people\nuse macOS as Git web server and therefore I thought it is not worth the\neffort to investigate this problem further.\n\n- Lars\n"},{"id":"305674","messageId":"20161110111348.61580-1-larsxschneider@gmail.com","threadId":"44311","inReplyTo":"1fdc5333-79a8-7c1d-f7e5-e8150d0ad9cc@web.de","subject":"[PATCH v2] Makefile: set NO_OPENSSL on macOS by default","fromName":"","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-10T11:13:48Z","receivedAt":"2016-11-10T11:13:57Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"From: Lars Schneider \n\nApple removed the OpenSSL header files in macOS 10.11 and above. OpenSSL\nwas deprecated since macOS 10.7.\n\nSet `NO_OPENSSL` and `APPLE_COMMON_CRYPTO` to `YesPlease` as default for\nmacOS. It is possible to override this and use OpenSSL by defining\n`NO_APPLE_COMMON_CRYPTO`.\n\nSigned-off-by: Lars Schneider \n---\n Makefile | 1 +\n 1 file changed, 1 insertion(+)\n\ndiff --git a/Makefile b/Makefile\nindex 9d6c245..f53fcc9 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)\n \t\tendif\n \tendif\n \tifndef NO_APPLE_COMMON_CRYPTO\n+\t\tNO_OPENSSL = YesPlease\n \t\tAPPLE_COMMON_CRYPTO = YesPlease\n \t\tCOMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO\n \tendif\n-- \n2.10.2\n\n"},{"id":"305675","messageId":"584C7DBA-71FE-4E66-85DC-EA22A6D2BB80@gmail.com","threadId":"44311","inReplyTo":"xmqqwpgc6x4t.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-10T11:19:26Z","receivedAt":"2016-11-10T11:19:41Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 10 Nov 2016, at 00:39, Junio C Hamano wrote:\n> \n> larsxschneider@gmail.com writes:\n> \n>> From: Lars Schneider \n>> \n>> Apple removed the OpenSSL header files in macOS and therefore Git does\n>> not build out of the box on macOS anymore. See previous discussion with\n>> Torsten here: http://public-inbox.org/git/565B3036.8000604@web.de/\n>> \n>> This mini series makes Git build out of the box on macOS, again, and\n>> disables the HTTPD tests on macOS TravisCI as they don't work anymore\n>> with the new macOS TravisCI default image:\n>> https://blog.travis-ci.com/2016-10-04-osx-73-default-image-live/\n>> \n>> Thanks,\n>> Lars\n>> \n>> \n>> Lars Schneider (2):\n>> config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11\n>> travis-ci: disable GIT_TEST_HTTPD for macOS\n>> \n>> .travis.yml | 3 ++-\n>> config.mak.uname | 6 ++++++\n>> 2 files changed, 8 insertions(+), 1 deletion(-)\n> \n> I've followed what was available at the public-inbox archive, but it\n> is unclear what the conclusion was. \n> \n> For the first one your \"how about\" non-patch, to which Peff said\n> \"that's simple and good\", looked good to me as well, but is it\n> available as a final patch that I can just take and apply (otherwise\n> I think I can do the munging myself, but I'd rather be spoon-fed\n> when able ;-).\n\nSure! Here you go:\nhttp://public-inbox.org/git/20161110111348.61580-1-larsxschneider@gmail.com/\n\n\n> I do not have a strong opinion on the second one. For an interim\n> solution, disabling webserver tests certainly is expedite and safe,\n> so I am fine taking it as-is, but I may have missed strong\n> objections.\n\nI haven't seen strong objections either. Just for reference, here is the patch:\nhttp://public-inbox.org/git/20161017002550.88782-3-larsxschneider@gmail.com/\n\n\nI hope you're well, again!!\n\n- Lars"},{"id":"305680","messageId":"20161110161012.jube4bwbww2wa2ew@sigill.intra.peff.net","threadId":"44311","inReplyTo":"CBAF806C-7E1E-4490-A07C-F98DB7488F5F@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-10T16:10:12Z","receivedAt":"2016-11-10T16:10:21Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Nov 10, 2016 at 12:07:14PM +0100, Lars Schneider wrote:\n\n> > Using Apache in the tests has been the source of frequent portability\n> > problems and configuration headaches. I do wonder if we'd be better off\n> > using some small special-purpose web server (even a short perl script\n> > written around HTTP::Server::Simple or something).\n> > \n> > On the other hand, testing against Apache approximates a more real-world\n> > case, which has value. It might be nice if our tests supported multiple\n> > web servers, but that would mean duplicating the config for each\n> > manually.\n> \n> I agree that the real-world Apache test is more valuable and I really want\n> to keep the Linux Apache test running. However, I don't think many people\n> use macOS as Git web server and therefore I thought it is not worth the\n> effort to investigate this problem further.\n\nIMHO, the value in the http tests is not testing the server side, but\nthe client side. Without being able to set up a dummy HTTP server, we do\nnot have any way to exercise the client side of git-over-http at all.\nAnd people on macOS _do_ use that. :)\n\n-Peff\n"},{"id":"305725","messageId":"xmqq1syj6mvq.fsf@gitster.mtv.corp.google.com","threadId":"44311","inReplyTo":"20161110161012.jube4bwbww2wa2ew@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-11-10T21:33:29Z","receivedAt":"2016-11-10T21:33:37Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> IMHO, the value in the http tests is not testing the server side, but\n> the client side. Without being able to set up a dummy HTTP server, we do\n> not have any way to exercise the client side of git-over-http at all.\n> And people on macOS _do_ use that. :)\n\nAmen to that.\n\nI however do not know what the universally available simplest dummy\nHTTP server would be. There probably are better alternative than\nApache with distro-customized ways of configuration that we have to\nadjust. \n\nA solution around HTTP::Server::Simple sounds attractive but is it\na realistic alternative or too much effort required? I dunno.\n"},{"id":"305726","messageId":"xmqqwpgb58ab.fsf@gitster.mtv.corp.google.com","threadId":"44311","inReplyTo":"584C7DBA-71FE-4E66-85DC-EA22A6D2BB80@gmail.com","subject":"Re: [PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-11-10T21:34:04Z","receivedAt":"2016-11-10T21:34:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Schneider writes:\n\n>> I've followed what was available at the public-inbox archive, but it\n>> is unclear what the conclusion was. \n>> \n>> For the first one your \"how about\" non-patch, to which Peff said\n>> \"that's simple and good\", looked good to me as well, but is it\n>> available as a final patch that I can just take and apply (otherwise\n>> I think I can do the munging myself, but I'd rather be spoon-fed\n>> when able ;-).\n>\n> Sure! Here you go:\n> http://public-inbox.org/git/20161110111348.61580-1-larsxschneider@gmail.com/\n>\n>\n>> I do not have a strong opinion on the second one. For an interim\n>> solution, disabling webserver tests certainly is expedite and safe,\n>> so I am fine taking it as-is, but I may have missed strong\n>> objections.\n>\n> I haven't seen strong objections either. Just for reference, here is the patch:\n> http://public-inbox.org/git/20161017002550.88782-3-larsxschneider@gmail.com/\n\nThanks. Picked up both of them.\n\n"},{"id":"305727","messageId":"20161110214345.cau5i4eybqdv74k3@sigill.intra.peff.net","threadId":"44311","inReplyTo":"xmqq1syj6mvq.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-10T21:43:45Z","receivedAt":"2016-11-10T21:44:05Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Nov 10, 2016 at 01:33:29PM -0800, Junio C Hamano wrote:\n\n> Jeff King writes:\n> \n> > IMHO, the value in the http tests is not testing the server side, but\n> > the client side. Without being able to set up a dummy HTTP server, we do\n> > not have any way to exercise the client side of git-over-http at all.\n> > And people on macOS _do_ use that. :)\n> \n> Amen to that.\n> \n> I however do not know what the universally available simplest dummy\n> HTTP server would be. There probably are better alternative than\n> Apache with distro-customized ways of configuration that we have to\n> adjust. \n> \n> A solution around HTTP::Server::Simple sounds attractive but is it\n> a realistic alternative or too much effort required? I dunno.\n\nI'm less concerned about the amount of effort (though I agree it may be\na blocker) than about the fact that it may not behave similarly to real\nservers. We have had real bugs and surprises with the way that various\nweb servers implement things. A few that come to mind are:\n\n 1. Buffering/deadlock issues between the webserver and the CGI (this\n was an issue with Apache, but I don't know about other servers).\n\n 2. The handling of CONTENT_LENGTH with chunked-encoding (this is still\n an issue with IIS).\n\n 3. What happens when you ask for \"foo.git/info/refs\" and \"foo.git\" is\n a bundle file (Apache gives you a 404, lighttpd serves the bundle).\n\nIdeally we'd test against a lot of different webservers, but that's\nexpensive (in CPU, but also in developer time). But I'd guess that\nApache is at least more representative than HTTP::Server::Simple of real\nservers in the wild.\n\nSo if we had a simple fallback in addition to Apache, I'd be OK with\nthat. But we still have a problem that (say) people on MacOS would never\nactually test against Apache, because it's not supported there.\n\n-Peff\n"},{"id":"305730","messageId":"xmqqk2cb57jz.fsf@gitster.mtv.corp.google.com","threadId":"44311","inReplyTo":"20161110214345.cau5i4eybqdv74k3@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-11-10T21:49:52Z","receivedAt":"2016-11-10T21:50:00Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n>> I however do not know what the universally available simplest dummy\n>> HTTP server would be. There probably are better alternative than\n>> Apache with distro-customized ways of configuration that we have to\n>> adjust. \n>> \n>> A solution around HTTP::Server::Simple sounds attractive but is it\n>> a realistic alternative or too much effort required? I dunno.\n>\n> I'm less concerned about the amount of effort (though I agree it may be\n> a blocker) than about the fact that it may not behave similarly to real\n> servers.\n\nI was wondering if it takes too much effort to make it behave\nsimilarly to real servers, so I guess we are on the same page. \n\nYes, I recall the IIS one raised and discussed at least twice on the\nlist in the past, and it sounded that we want some solution to that.\n\n> 3. What happens when you ask for \"foo.git/info/refs\" and \"foo.git\" is\n> a bundle file (Apache gives you a 404, lighttpd serves the bundle).\n\nThat's a bad one. Do we want a client-side \"I am connecting to a\nsite that knows how to talk smart-http\" option or something to work\nit around?\n"},{"id":"305732","messageId":"20161110215454.zcphqxzkvt4mlpwb@sigill.intra.peff.net","threadId":"44311","inReplyTo":"xmqqk2cb57jz.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-10T21:54:55Z","receivedAt":"2016-11-10T21:55:01Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Nov 10, 2016 at 01:49:52PM -0800, Junio C Hamano wrote:\n\n> Yes, I recall the IIS one raised and discussed at least twice on the\n> list in the past, and it sounded that we want some solution to that.\n\nThe patches had some issues. I suspect the population of people who want\nto run a git server on IIS is relatively small. I am content to wait for\nsomebody who has such a setup to produce a working patch.\n\n> > 3. What happens when you ask for \"foo.git/info/refs\" and \"foo.git\" is\n> > a bundle file (Apache gives you a 404, lighttpd serves the bundle).\n> \n> That's a bad one. Do we want a client-side \"I am connecting to a\n> site that knows how to talk smart-http\" option or something to work\n> it around?\n\nIt doesn't matter because we don't actually support fetching HTTP\nbundles via \"git fetch\" yet.\n\nBut I ran it across the issue and did make such a fix when I was\nimplementing that feature long ago. See the discussion of \"surprise\" in\n[1] and [2].\n\nWow, that series is exactly 5 years old today. Have I really been\nprocrastinating on re-rolling it that long? Yikes.\n\n-Peff\n\n[1] http://public-inbox.org/git/20111110075052.GI27950@sigill.intra.peff.net/\n\n[2] http://public-inbox.org/git/20111110075052.GI27950@sigill.intra.peff.net/\n"},{"id":"305755","messageId":"2088B631-4FE8-4232-9F3C-699122E6A7B0@gmail.com","threadId":"44311","inReplyTo":"20161110161012.jube4bwbww2wa2ew@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-11T08:22:51Z","receivedAt":"2016-11-11T08:22:59Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\nOn 10 Nov 2016, at 17:10, Jeff King wrote:\n\n> On Thu, Nov 10, 2016 at 12:07:14PM +0100, Lars Schneider wrote:\n> \n>>> Using Apache in the tests has been the source of frequent portability\n>>> problems and configuration headaches. I do wonder if we'd be better off\n>>> using some small special-purpose web server (even a short perl script\n>>> written around HTTP::Server::Simple or something).\n>>> \n>>> On the other hand, testing against Apache approximates a more real-world\n>>> case, which has value. It might be nice if our tests supported multiple\n>>> web servers, but that would mean duplicating the config for each\n>>> manually.\n>> \n>> I agree that the real-world Apache test is more valuable and I really want\n>> to keep the Linux Apache test running. However, I don't think many people\n>> use macOS as Git web server and therefore I thought it is not worth the\n>> effort to investigate this problem further.\n> \n> IMHO, the value in the http tests is not testing the server side, but\n> the client side. Without being able to set up a dummy HTTP server, we do\n> not have any way to exercise the client side of git-over-http at all.\n> And people on macOS _do_ use that. :)\n\nWell, I haven't seen it from that perspective, yet, but I agree :-)\n\n\nTo all macOS users on the list:\nDoes anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n\n\nThere would be an alternative way to approach the problem:\nSomeone (GitHub?, BitBucket?, GitLab?, ...) could setup a bunch of webservers\nwith popular configurations and a way to reset a clean test environment. Then \nthe TravisCI client tests could go against these servers.\n\nI realize that this idea is probably unrealistic because too much setup and\nmaintenance work would be required.\n\nCheers,\nLars"},{"id":"305759","messageId":"20161111084725.jkdsdt4yslzsuh6d@sigill.intra.peff.net","threadId":"44311","inReplyTo":"2088B631-4FE8-4232-9F3C-699122E6A7B0@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-11T08:47:25Z","receivedAt":"2016-11-11T08:47:32Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n\n> There would be an alternative way to approach the problem:\n> Someone (GitHub?, BitBucket?, GitLab?, ...) could setup a bunch of webservers\n> with popular configurations and a way to reset a clean test environment. Then \n> the TravisCI client tests could go against these servers.\n>\n> I realize that this idea is probably unrealistic because too much setup and\n> maintenance work would be required.\n\nYeah, it seems like it adds a lot of complexity for little gain. Plus it\ncreates a network dependency on running the tests. I know you care\nmostly about Travis, but I am much more interested in all of the people\n(developers and not) who run \"make test\" on their own platforms.\n\nIf you did want to have a more real-world network-based test, I think\nthe right solution is not for GitHub to set up a bunch of mock servers,\nbut to design client-side tests that hit the _real_ GitHub (or GitLab,\nor whatever) and perform some basic operations. OTOH, people running\n\"master\" (or \"next\", etc) are doing that implicitly every day.\n\n-Peff\n"},{"id":"305760","messageId":"C596D508-8F95-451A-ADF0-8A41BEE9842C@gmail.com","threadId":"44311","inReplyTo":"xmqqwpgb58ab.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v1 0/2] Fix default macOS build locally and on Travis CI","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-11T09:01:39Z","receivedAt":"2016-11-11T09:01:47Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\nOn 10 Nov 2016, at 22:34, Junio C Hamano wrote:\n\n> Lars Schneider writes:\n> \n>>> I've followed what was available at the public-inbox archive, but it\n>>> is unclear what the conclusion was. \n>>> \n>>> For the first one your \"how about\" non-patch, to which Peff said\n>>> \"that's simple and good\", looked good to me as well, but is it\n>>> available as a final patch that I can just take and apply (otherwise\n>>> I think I can do the munging myself, but I'd rather be spoon-fed\n>>> when able ;-).\n>> \n>> Sure! Here you go:\n>> http://public-inbox.org/git/20161110111348.61580-1-larsxschneider@gmail.com/\n>> \n>> \n>>> I do not have a strong opinion on the second one. For an interim\n>>> solution, disabling webserver tests certainly is expedite and safe,\n>>> so I am fine taking it as-is, but I may have missed strong\n>>> objections.\n>> \n>> I haven't seen strong objections either. Just for reference, here is the patch:\n>> http://public-inbox.org/git/20161017002550.88782-3-larsxschneider@gmail.com/\n> \n> Thanks. Picked up both of them.\n\nThanks! This makes \"next\" pass, again:\nhttps://travis-ci.org/git/git/builds/174946111\n\nBTW: If you want to learn about the build status of \"git/git\" branches on the\ncommand line then you can use this snippet:\n\n$ branch=next; echo \"$branch: $(curl -s https://api.travis-ci.org/repos/git/git/branches/$branch | perl -lape 's/.*\"state\":\"(\\w+)\".*\"sha\":\"(\\w{7}).*/$1 $2/g')\"\n\nCheers,\nLars"},{"id":"305764","messageId":"BAB2E4AE-7241-4991-A993-A60DBA75092B@gmail.com","threadId":"44311","inReplyTo":"20161111084725.jkdsdt4yslzsuh6d@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-11T09:13:44Z","receivedAt":"2016-11-11T09:13:56Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\nOn 11 Nov 2016, at 09:47, Jeff King wrote:\n\n> On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n> \n>> There would be an alternative way to approach the problem:\n>> Someone (GitHub?, BitBucket?, GitLab?, ...) could setup a bunch of webservers\n>> with popular configurations and a way to reset a clean test environment. Then \n>> the TravisCI client tests could go against these servers.\n>> \n>> I realize that this idea is probably unrealistic because too much setup and\n>> maintenance work would be required.\n> \n> Yeah, it seems like it adds a lot of complexity for little gain. Plus it\n> creates a network dependency on running the tests. I know you care\n> mostly about Travis, but I am much more interested in all of the people\n> (developers and not) who run \"make test\" on their own platforms.\n> \n> If you did want to have a more real-world network-based test, I think\n> the right solution is not for GitHub to set up a bunch of mock servers,\n> but to design client-side tests that hit the _real_ GitHub (or GitLab,\n> or whatever) and perform some basic operations. OTOH, people running\n> \"master\" (or \"next\", etc) are doing that implicitly every day.\n\nThat is actually a neat idea. We could setup a test repo on each of the major \nGit hosting sites and then the TravisCI run could clone a repo and push\nchanges to it. That shouldn't take long and would probably be a good real\nworld test.\n\nThe credentials of these repos could be stored encrypted in Travis CI [1].\n\nWhere would such a test repo live on github.com? On github.com/git or would\nyou prefer a separate organization? (no worries, I am not going to tackle this\nanytime soon -- too many things in my backlog already).\n\nBTW: I just noticed https://github.com/git/hello-world ... is this legitimate\nor did someone hack github.com/git? :)\n\nCheers,\nLars\n\n\n[1] https://docs.travis-ci.com/user/environment-variables/#Defining-encrypted-variables-in-.travis.yml\n\n"},{"id":"305766","messageId":"20161111092824.qqgrmhtkuw3wpbwa@sigill.intra.peff.net","threadId":"44311","inReplyTo":"BAB2E4AE-7241-4991-A993-A60DBA75092B@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-11T09:28:25Z","receivedAt":"2016-11-11T09:28:36Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Nov 11, 2016 at 10:13:44AM +0100, Lars Schneider wrote:\n\n> > If you did want to have a more real-world network-based test, I think\n> > the right solution is not for GitHub to set up a bunch of mock servers,\n> > but to design client-side tests that hit the _real_ GitHub (or GitLab,\n> > or whatever) and perform some basic operations. OTOH, people running\n> > \"master\" (or \"next\", etc) are doing that implicitly every day.\n> \n> That is actually a neat idea. We could setup a test repo on each of the major \n> Git hosting sites and then the TravisCI run could clone a repo and push\n> changes to it. That shouldn't take long and would probably be a good real\n> world test.\n> \n> The credentials of these repos could be stored encrypted in Travis CI [1].\n> \n> Where would such a test repo live on github.com? On github.com/git or would\n> you prefer a separate organization? (no worries, I am not going to tackle this\n> anytime soon -- too many things in my backlog already).\n\nI think I'd prefer for it to live outside of the \"git\" organization\nentirely, if only because it's credentials will be necessarily less\nsecure. It would be nice if people could point the suite at their own\nuser/repo, too, so they can run it outside of travis.\n\n> BTW: I just noticed https://github.com/git/hello-world ... is this legitimate\n> or did someone hack github.com/git? :)\n\nHmm. I wondered myself. There's no audit record of who created it, but\nthe creation timestamp is from 2008-07-23, which predates a lot of the\nlogging. So offhand, I'd say the likely explanation is \"Scott Chacon\nexperimenting\".\n\nIt's probably worth cleaning out now, though.\n\n-Peff\n"},{"id":"305944","messageId":"20161115120718.GA7854@book.hvoigt.net","threadId":"44311","inReplyTo":"2088B631-4FE8-4232-9F3C-699122E6A7B0@gmail.com","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2016-11-15T12:07:18Z","receivedAt":"2016-11-15T12:07:30Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n> To all macOS users on the list:\n> Does anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n\nNope. The following tests fail for me on master: 5539, 5540, 5541, 5542,\n5550, 5551, 5561, 5812.\n\nCheers Heiko\n"},{"id":"305949","messageId":"48AB4BEB-BBC1-48D4-99B9-D1AAA653EE24@gmail.com","threadId":"44311","inReplyTo":"20161115120718.GA7854@book.hvoigt.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2016-11-15T14:18:11Z","receivedAt":"2016-11-15T14:18:21Z","isPatch":true,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 15 Nov 2016, at 13:07, Heiko Voigt wrote:\n> \n> On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n>> To all macOS users on the list:\n>> Does anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n> \n> Nope. The following tests fail for me on master: 5539, 5540, 5541, 5542,\n> 5550, 5551, 5561, 5812.\n\nThat's what I see, too. Apache needs to be configured in some special way \nto make them work and I was wondering if anyone has figured that out\nalready for macOS...\n\nHowever, I much prefer Peff's idea to test against real world servers:\nhttp://public-inbox.org/git/20161111092824.qqgrmhtkuw3wpbwa@sigill.intra.peff.net/\n\n- Lars"},{"id":"305957","messageId":"20161115153159.mxfl73dnhljad5so@sigill.intra.peff.net","threadId":"44311","inReplyTo":"20161115120718.GA7854@book.hvoigt.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-15T15:31:59Z","receivedAt":"2016-11-15T15:32:55Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Nov 15, 2016 at 01:07:18PM +0100, Heiko Voigt wrote:\n\n> On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n> > To all macOS users on the list:\n> > Does anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n> \n> Nope. The following tests fail for me on master: 5539, 5540, 5541, 5542,\n> 5550, 5551, 5561, 5812.\n\nFailing how? Does apache fail to start up? Do tests fails? What does\n\"-v\" say? Is there anything interesting in httpd/error.log in the trash\ndirectory?\n\n-Peff\n"},{"id":"306048","messageId":"20161116143925.GA32631@book.hvoigt.net","threadId":"44311","inReplyTo":"20161115153159.mxfl73dnhljad5so@sigill.intra.peff.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2016-11-16T14:39:25Z","receivedAt":"2016-11-16T14:39:36Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Tue, Nov 15, 2016 at 10:31:59AM -0500, Jeff King wrote:\n> On Tue, Nov 15, 2016 at 01:07:18PM +0100, Heiko Voigt wrote:\n> \n> > On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n> > > To all macOS users on the list:\n> > > Does anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n> > \n> > Nope. The following tests fail for me on master: 5539, 5540, 5541, 5542,\n> > 5550, 5551, 5561, 5812.\n> \n> Failing how? Does apache fail to start up? Do tests fails? What does\n> \"-v\" say? Is there anything interesting in httpd/error.log in the trash\n> directory?\n\nThis is what I see for 5539:\n\n$ GIT_TEST_HTTPD=1 ./t5539-fetch-http-shallow.sh -v\nInitialized empty Git repository in /Users/hvoigt/Repository/git4/t/trash directory.t5539-fetch-http-shallow/.git/\nchecking prerequisite: NOT_ROOT\n\nmkdir -p \"$TRASH_DIRECTORY/prereq-test-dir\" &&\n(\n\tcd \"$TRASH_DIRECTORY/prereq-test-dir\" &&\n\tuid=$(id -u) &&\n\ttest \"$uid\" != 0\n\n)\nprerequisite NOT_ROOT ok\nhttpd: Syntax error on line 65 of /Users/hvoigt/Repository/git4/t/lib-httpd/apache.conf: Cannot load modules/mod_mpm_prefork.so into server: dlopen(/Users/hvoigt/Repository/git4/t/trash directory.t5539-fetch-http-shallow/httpd/modules/mod_mpm_prefork.so, 10): image not found\nerror: web server setup failed\n\n\nIt seems the other failures have the same cause.\n\nCheers Heiko\n"},{"id":"306079","messageId":"ca6e313f-912d-47a4-2dae-bddc969cd3a0@web.de","threadId":"44311","inReplyTo":"20161116143925.GA32631@book.hvoigt.net","subject":"Re: [PATCH v1 2/2] travis-ci: disable GIT_TEST_HTTPD for macOS","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2016-11-16T20:01:34Z","receivedAt":"2016-11-16T20:02:47Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On 16.11.16 15:39, Heiko Voigt wrote:\n> On Tue, Nov 15, 2016 at 10:31:59AM -0500, Jeff King wrote:\n>> On Tue, Nov 15, 2016 at 01:07:18PM +0100, Heiko Voigt wrote:\n>>\n>>> On Fri, Nov 11, 2016 at 09:22:51AM +0100, Lars Schneider wrote:\n>>>> To all macOS users on the list:\n>>>> Does anyone execute the tests with GIT_TEST_HTTPD enabled successfully?\n>>>\n>>> Nope. The following tests fail for me on master: 5539, 5540, 5541, 5542,\n>>> 5550, 5551, 5561, 5812.\n>>\n>> Failing how? Does apache fail to start up? Do tests fails? What does\n>> \"-v\" say? Is there anything interesting in httpd/error.log in the trash\n>> directory?\n> \n> This is what I see for 5539:\n> \n> $ GIT_TEST_HTTPD=1 ./t5539-fetch-http-shallow.sh -v\n> Initialized empty Git repository in /Users/hvoigt/Repository/git4/t/trash directory.t5539-fetch-http-shallow/.git/\n> checking prerequisite: NOT_ROOT\n> \n> mkdir -p \"$TRASH_DIRECTORY/prereq-test-dir\" &&\n> (\n> \tcd \"$TRASH_DIRECTORY/prereq-test-dir\" &&\n> \tuid=$(id -u) &&\n> \ttest \"$uid\" != 0\n> \n> )\n> prerequisite NOT_ROOT ok\n> httpd: Syntax error on line 65 of /Users/hvoigt/Repository/git4/t/lib-httpd/apache.conf: Cannot load modules/mod_mpm_prefork.so into server: dlopen(/Users/hvoigt/Repository/git4/t/trash directory.t5539-fetch-http-shallow/httpd/modules/mod_mpm_prefork.so, 10): image not found\n> error: web server setup failed\n> \nYes, same here.\n\nIf we take that out:\n\ndiff --git a/t/lib-httpd/apache.conf b/t/lib-httpd/apache.conf\nindex c3e6313..1925fdb 100644\n--- a/t/lib-httpd/apache.conf\n+++ b/t/lib-httpd/apache.conf\n@@ -61,9 +61,6 @@ LockFile accept.lock\n \n LoadModule access_compat_module modules/mod_access_compat.so\n \n-\n- LoadModule mpm_prefork_module modules/mod_mpm_prefork.so\n-\n \n LoadModule unixd_module modules/mod_unixd.so\n \n\nI run into other issues:\n\n [core:emerg] [pid 2502] (2)No such file or directory: AH00023: Couldn't create the rewrite-map mutex (file /private/var/run/rewrite-map.2502)\nAH00016: Configuration Failed\n\n(apache2 comes via MacPorts)\n\n\n"}]}