[Linux-ha-dev] Re: [Linux-ha-cvs] Linux-HA CVS: heartbeat by zhenh from

[Sun Jiang Dong]

Except this environment variable, actually there are other function overlapped environment variables.
e.g.  LOGFENV,DEBUGFENV,LOGFACILITY vs "HA_*" . Do we need to remove the redundant environment variables?

Andrew Beekhof (GMail) wrote:
> This shouldnt be needed, the debug level is already set as:     "HA_"+ 
> KEY_DEBUGLEVEL
> 
> On Apr 13, 2005, at 1:47 PM, linux-ha-cvs at lists.linux-ha.org wrote:
> 
>> linux-ha CVS committal
>>
>> Author  : zhenh
>> Host    :
>> Project : linux-ha
>> Module  : heartbeat
>>
>> Dir     : linux-ha/heartbeat
>>
>>
>> Modified Files:
>>     config.c
>>
>>
>> Log Message:
>> set the env variable of debug level
>> ===================================================================
>> RCS file: /home/cvs/linux-ha/linux-ha/heartbeat/config.c,v
>> retrieving revision 1.149
>> retrieving revision 1.150
>> diff -u -3 -r1.149 -r1.150
>> --- config.c    3 Mar 2005 16:21:03 -0000    1.149
>> +++ config.c    13 Apr 2005 11:47:51 -0000    1.150
>> @@ -1,4 +1,4 @@
>> -/* $Id: config.c,v 1.149 2005/03/03 16:21:03 andrew Exp $ */
>> +/* $Id: config.c,v 1.150 2005/04/13 11:47:51 zhenh Exp $ */
>>  /*
>>   * Parse various heartbeat configuration files...
>>   *
>> @@ -1617,6 +1617,11 @@
>>  {
>>      debug = atoi(value);
>>      if (debug >= 0 && debug < 256) {
>> +        if (debug > 0) {
>> +            static char cdebug[8];
>> +            snprintf(cdebug, sizeof(debug), "%d", debug);
>> +            setenv(HADEBUGVAL, cdebug, TRUE);
>> +        }
>>          return(HA_OK);
>>      }
>>      return(HA_FAIL);
>> @@ -2154,6 +2159,9 @@
>>
>>  /*
>>   * $Log: config.c,v $
>> + * Revision 1.150  2005/04/13 11:47:51  zhenh
>> + * set the env variable of debug level
>> + *
>>   * Revision 1.149  2005/03/03 16:21:03  andrew
>>   * Use the common logging setup calls (change cleared by gshi)
>>   *
>>
>>
>> _______________________________________________
>> Linux-ha-cvs mailing list
>> Linux-ha-cvs at lists.linux-ha.org
>> http://lists.community.tummy.com/mailman/listinfo/linux-ha-cvs
>>
>>
> -- 
> Andrew Beekhof
> 
> "No means no, and no means yes, and everything in between and all the 
> rest" - TISM
> 
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev at lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
> 

-- 
BRs,

Sun Jiang Dong