Please review them, thanks!
Bo Yang
--
Samba Team boy...@samba.orghttp://www.samba.org
>From ac6d3f7cb7caa3dca498096bd0c07e6b02601239 Mon Sep 17 00:00:00 2001
From: Bo Yang
Date: Fri, 2 Apr 2010 18:35:12 +0800
Subject: [PATCH] Enable build on db-4.9+.
Signed-off-by: Bo Yang
---
On 04/14/2010 06:44 AM, Samba - BoYang wrote:
Please review them, thanks!
Applied to nfs4d and tabled, thanks!
In the future, it would be appreciated to see separate emails, one email
per patch.
Best regards,
Jeff
--
To unsubscribe from this list: send the line "unsubscribe hai
On 04/14/2010 09:40 PM, Jeff Garzik wrote:
> would be appreciated to see separate emails, o
Yah, I see. :-)
--
Samba Team boy...@samba.orghttp://www.samba.org
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majord...@vger.kernel.
On Wed, 14 Apr 2010 00:01:23 -0400
Jeff Garzik wrote:
> Long term, I think the per-packet session logging macro should be made
> more clear, perhaps calling it HAIL_SESS or even HAIL_PKT
I saved this. Then, when you have a better idea already... SUDDENLY,
A PATCH! :-)
-- Pete
--
To unsubscribe
Just a bunch of mostly useless stuff. Feel free to reject, but
let me know so I can kill them out of my trees too.
-- Pete
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/ma
You know what's weird... Without this, I cannot build an RPM at all,
the rpmbuild complains about unpackaged files and aborts. But
everyone else seems to have no problem? Strange. BTW, I am on
Fedora 14.
Signed-off-by: Pete Zaitcev
---
lib/Makefile.am |1 +
1 file changed, 1 insertion(+)
d
Currently, there's nothing in the verbose output about sent packets
at all. No, really! This is very confusing, even if I run tcpdump
in the same time. I think we should add this.
Signed-off-by: Pete Zaitcev
---
lib/cldc.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/cldc.c b/li
I distinctly remember how my precious ifs were mercilessly replaced
elsewhere, but apparently this piece escaped notice thus far.
Now, make it more style-uniform with the rest.
Signed-off-by: Pete Zaitcev
---
lib/cldc.c |9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --g
A prototype is not included again.
Signed-off-by: Pete Zaitcev
---
lib/common.c |1 +
1 file changed, 1 insertion(+)
diff --git a/lib/common.c b/lib/common.c
index fb0aae6..e399ec9 100644
--- a/lib/common.c
+++ b/lib/common.c
@@ -26,6 +26,7 @@
#include
#include
#include
+#include
#
Tests do not really need this kind of thing, in fact if everything
goes well, we should NOT receive any events while test is running.
Still, I am curious about being sure that we do not. This will also
pop if we change something in the library implementation and start
leaking some events.
Signed-o
Same as everywhere else: missing prototypes, so implementations are
not actually matched by the compiler.
Signed-off-by: Pete Zaitcev
---
lib/readport.c |1 +
test/libtest.c |1 +
2 files changed, 2 insertions(+)
diff --git a/lib/readport.c b/lib/readport.c
index afd1c91..8589ec6 10064
Completely useless warning: main is main, right? Is it even checked?
But oh well, stub it with unused arguments and hope other gcc version
does not complain about _that_.
Signed-off-by: Pete Zaitcev
---
test/wait-for-listen.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Signed-off-by: Pete Zaitcev
---
lib/cldc.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Looks like I confused myself with patch numbering. This one is
out of order, or maybe a duplicate? Sending on the premise that
it's easier to drop.
diff --git b/lib/cldc.c a/lib/cldc.c
index
On 04/14/2010 02:34 PM, Pete Zaitcev wrote:
Currently, there's nothing in the verbose output about sent packets
at all. No, really! This is very confusing, even if I run tcpdump
in the same time. I think we should add this.
Signed-off-by: Pete Zaitcev
---
lib/cldc.c |2 ++
1 file changed
On 04/14/2010 02:46 PM, Pete Zaitcev wrote:
Signed-off-by: Pete Zaitcev
---
lib/cldc.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Looks like I confused myself with patch numbering. This one is
out of order, or maybe a duplicate? Sending on the premise that
it's easier to dr
On 04/14/2010 02:35 PM, Pete Zaitcev wrote:
Same as everywhere else: missing prototypes, so implementations are
not actually matched by the compiler.
Signed-off-by: Pete Zaitcev
---
lib/readport.c |1 +
test/libtest.c |1 +
2 files changed, 2 insertions(+)
applied 7-8
--
To unsu
On 04/14/2010 02:33 PM, Pete Zaitcev wrote:
You know what's weird... Without this, I cannot build an RPM at all,
the rpmbuild complains about unpackaged files and aborts. But
everyone else seems to have no problem? Strange. BTW, I am on
Fedora 14.
Signed-off-by: Pete Zaitcev
---
lib/Makefile.
17 matches
Mail list logo