* [Cluster-devel] fence/agents/ipmilan
@ 2007-08-30 3:48 Fabio M. Di Nitto
2007-08-30 15:41 ` James Parsons
0 siblings, 1 reply; 2+ messages in thread
From: Fabio M. Di Nitto @ 2007-08-30 3:48 UTC (permalink / raw)
To: cluster-devel.redhat.com
Hi guys,
I just noticed that ipmilan is built using -DFENCE. Looking at the code,
there is a set of ifdef/else/endif to build the agent as STONITH agent.
Does anybody know if this is still the case? If not we could remove all
the dead code from it.
There is also an ifndef TESTING floating around used to change the
copyright information for test build that IMHO could go away.
Any opinion on cleaning it up?
Cheers
Fabio
--
I'm going to make him an offer he can't refuse.
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Cluster-devel] fence/agents/ipmilan
2007-08-30 3:48 [Cluster-devel] fence/agents/ipmilan Fabio M. Di Nitto
@ 2007-08-30 15:41 ` James Parsons
0 siblings, 0 replies; 2+ messages in thread
From: James Parsons @ 2007-08-30 15:41 UTC (permalink / raw)
To: cluster-devel.redhat.com
Fabio M. Di Nitto wrote:
>Hi guys,
>
>I just noticed that ipmilan is built using -DFENCE. Looking at the code,
>there is a set of ifdef/else/endif to build the agent as STONITH agent.
>
>Does anybody know if this is still the case? If not we could remove all
>the dead code from it.
>
>There is also an ifndef TESTING floating around used to change the
>copyright information for test build that IMHO could go away.
>
>Any opinion on cleaning it up?
>
Lon wrote this agent and would be the one to ask...lon?
-J
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-08-30 15:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-30 3:48 [Cluster-devel] fence/agents/ipmilan Fabio M. Di Nitto
2007-08-30 15:41 ` James Parsons
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).