Here is small patch to enable link against higher-version db for nfs4d and tabled. :-)

2010-04-14 Thread Samba - BoYang
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 ---

Re: Here is small patch to enable link against higher-version db for nfs4d and tabled. :-)

2010-04-14 Thread Jeff Garzik
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

Re: Here is small patch to enable link against higher-version db for nfs4d and tabled. :-)

2010-04-14 Thread Samba - BoYang
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.

Re: [Patch 1/3] CLD: End-to-end verbosity v3

2010-04-14 Thread Pete Zaitcev
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

[Patch 0/8] Cleanups

2010-04-14 Thread Pete Zaitcev
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

[Patch 1/8] CLD: cleanup: add cld_msg_rpc.x

2010-04-14 Thread Pete Zaitcev
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

[Patch 2/8] CLD: cleanup: add a log entry about sent packet

2010-04-14 Thread Pete Zaitcev
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

[Patch 3/8] CLD: cleanup: if() to switch()

2010-04-14 Thread Pete Zaitcev
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

[Patch 4/8] CLD: cleanup: add #include

2010-04-14 Thread Pete Zaitcev
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 #

[Patch 6/8] CLD: cleanup: add a event stub just to test

2010-04-14 Thread Pete Zaitcev
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

[Patch 7/8] tabled: cleanup: add #include

2010-04-14 Thread Pete Zaitcev
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

[Patch 8/8] tabled: cleanup: add protype for main()

2010-04-14 Thread Pete Zaitcev
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

[Patch 5/8] CLD: cleanup: wrap a line

2010-04-14 Thread Pete Zaitcev
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

Re: [Patch 2/8] CLD: cleanup: add a log entry about sent packet

2010-04-14 Thread Jeff Garzik
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

Re: [Patch 5/8] CLD: cleanup: wrap a line

2010-04-14 Thread Jeff Garzik
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

Re: [Patch 7/8] tabled: cleanup: add #include

2010-04-14 Thread Jeff Garzik
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

Re: [Patch 1/8] CLD: cleanup: add cld_msg_rpc.x

2010-04-14 Thread Jeff Garzik
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.