On Sat, Sep 08, 2012 at 09:20:48PM +0200, Jan Engelhardt wrote:
On Saturday 2012-09-08 20:59, Junio C Hamano wrote:
diff --git a/daemon.c b/daemon.c
index 4602b46..eaf08c2 100644
--- a/daemon.c
+++ b/daemon.c
@@ -1,3 +1,4 @@
+#include stdbool.h
#include cache.h
#include
Jeff King wrote:
On Sat, Sep 08, 2012 at 09:20:48PM +0200, Jan Engelhardt wrote:
On Saturday 2012-09-08 20:59, Junio C Hamano wrote:
diff --git a/daemon.c b/daemon.c
index 4602b46..eaf08c2 100644
--- a/daemon.c
+++ b/daemon.c
@@ -1,3 +1,4 @@
+#include stdbool.h
#include cache.h
#include
Jeff King wrote:
On Mon, Sep 10, 2012 at 04:38:58PM +0200, Joachim Schmitz wrote:
More importantly, though, is it actually portable? I thought it was
added in C99, and we try to stick to C89 to support older compilers
and systems. My copy of C99 is vague (it says only that the bool
macro was
On Mon, Sep 10, 2012 at 07:26:26PM +0200, Joachim Schmitz wrote:
as are non-const array intializers, e.g.:
const char *args[] = { editor, path, NULL };
^
.../git/editor.c, line 39: error(122): expression must have a
constant
From: Jeff King [mailto:p...@peff.net]
Sent: Monday, September 10, 2012 7:59 PM
To: Joachim Schmitz
Cc: git@vger.kernel.org
Subject: Re: [PATCH] daemon: restore getpeername(0,...) use
On Mon, Sep 10, 2012 at 07:26:26PM +0200, Joachim Schmitz wrote:
as are non-const array intializers
On Mon, Sep 10, 2012 at 08:27:07PM +0200, Joachim Schmitz wrote:
I think it is a mistake to set -std=c89 (or whatever similar option your
compiler supports). Like I said, we are not interested in being strictly
C89-compliant. We are interested in working on real-world systems.
If your
Jan Engelhardt wrote:
This reverts f9c87be6b42dd0f8b31a4bb8c6a44326879fdd1a, in a sense,
because that commit broke logging of Connection from ... when
git-daemon is run under xinetd.
This patch here computes the text representation of the peer and then
copies that to environment variables such
Joachim Schmitz j...@schmitz-digital.de writes:
+ setenv(REMOTE_PORT, portbuf, true);
setenv() is not a function available on all plattfomrs.
Please do some homework before adding irrelevant noise. At the
minimum, run git grep to see if we already use it in other places,
and investigate why
From: Junio C Hamano [mailto:gits...@pobox.com]
Sent: Saturday, September 08, 2012 9:04 PM
To: Joachim Schmitz
Cc: git@vger.kernel.org
Subject: Re: [PATCH] daemon: restore getpeername(0,...) use
Joachim Schmitz j...@schmitz-digital.de writes:
+ setenv(REMOTE_PORT, portbuf, true
On Saturday 2012-09-08 20:59, Junio C Hamano wrote:
diff --git a/daemon.c b/daemon.c
index 4602b46..eaf08c2 100644
--- a/daemon.c
+++ b/daemon.c
@@ -1,3 +1,4 @@
+#include stdbool.h
#include cache.h
#include pkt-line.h
#include exec_cmd.h
Platform agnostic parts of the code that use
10 matches
Mail list logo
