minimum buf_size doesn't fit me anymore

[Chip Seraphine]

I've been fooling around in v 2.1.3 with the new RPM stuff and so far it 
seems quite nice.    The one issue is that I had to significantly jack up 
my minimum buf_size (client.c line 543) from 2048 to 8192; this seems 
somewhat unavoidable if you are using the packages: toys, unless you are 
only using it to add the occasional odd RPM instead of the whole lot.

Cfengine does a lot more than it used to, and humongeous class strings are 
probably becoming more common. Perhaps we could up it some to reflect this? 

Here's the tiny change I made in 2.1.3 to allow me to do 
CFLAGS="-DMIN_CLIENT_BUF_SIZE=8192" on the ./configure line.   If others 
are having this issue it may prove convenient.


--- client.c    2004-04-08 11:15:53.000000000 -0500
+++ client.c.orig       2004-04-08 11:12:40.000000000 -0500
@@ -540,7 +540,7 @@

 if (buf_size < 2048)
    {
-   buf_size = MIN_CLIENT_BUF_SIZE;
+   buf_size = 2048;
    }

 if (ip->encrypt == 'y')


--- cf.defs.h   2004-04-08 11:15:20.000000000 -0500
+++ cf.defs.h.orig      2004-04-08 11:12:46.000000000 -0500
@@ -345,10 +345,6 @@

 #define CFLOGSIZE 1048576                  /* Size of lock-log before 
rotation */

-#ifndef MIN_CLIENT_BUF_SIZE
-#define MIN_CLIENT_BUF_SIZE 2048
-#endif
-
 /* Output control defines */

 #define Verbose if (VERBOSE || DEBUG || D2) printf


<FEATURE_REQUEST type="cleverly disguised as speculation">
This might not be as necessary if there were some more granular means of 
assembling ${allclasses}-like strings; like an 
AddClassnameToStringButOnlyIfItIsActuallyDefined(class1 class2 classN) sort 
of thing that jammed its argument classes into ${someclasses} if they were 
true, so that I could then call my  scripts/modules with ${someclasses} (or 
a copy of same) as an argument instead of ${allclasses}.

Or do most people not routinely use cfengine classes to turn on/off large 
numbers of options to scripts and modules?
</FEATURE_REQUEST>