[prev in list] [next in list] [prev in thread] [next in thread] 

List:       net-snmp-coders
Subject:    Re: Fwd: SF.net SVN: net-snmp:[19116]
From:       Bart Van Assche <bvanassche () acm ! org>
Date:       2010-06-30 10:13:23
Message-ID: AANLkTik_9pN3y-aDWlpmviiJc_2_NbfpOHZCwcehe0_Y () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


 On Wed, Jun 30, 2010 at 12:30 AM, Thomas Anders <
thomas.anders@blue-cable.de> wrote:

> Bart Van Assche wrote:
> > On Tue, Jun 29, 2010 at 6:28 PM, Thomas Anders
> > <thomas.anders@blue-cable.de <mailto:thomas.anders@blue-cable.de>>
> wrote:
> >     Where exactly do you propose to add
> >     #include <net-snmp/agent/agent_module_config.h>?
> >
> > The safest place to add that include statement IMHO is inside the source
> > files apps/snmptrapd.c and apps/snmptrapd_auth.c.
>
> See r19134 for my proposed fix.
>
> Please note that apps/snmptrapd.c already includes
> net-snmp/agent/net-snmp-agent-includes.h which includes
> net-snmp/agent/agent_module_config.h, so we're safe there.
>

Thanks ! That fixed it at least on the trunk (haven't retested the branches
yet).

Bart.

[Attachment #5 (text/html)]

<div class="gmail_quote">On Wed, Jun 30, 2010 at 12:30 AM, Thomas Anders <span \
dir="ltr">&lt;<a href="mailto:thomas.anders@blue-cable.de">thomas.anders@blue-cable.de</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; \
border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <div class="im">Bart \
Van Assche wrote:<br> &gt; On Tue, Jun 29, 2010 at 6:28 PM, Thomas Anders<br>
</div><div class="im">&gt; &lt;<a \
href="mailto:thomas.anders@blue-cable.de">thomas.anders@blue-cable.de</a> \
&lt;mailto:<a href="mailto:thomas.anders@blue-cable.de">thomas.anders@blue-cable.de</a>&gt;&gt; \
wrote:<br> &gt;     Where exactly do you propose to add<br>
&gt;     #include &lt;net-snmp/agent/agent_module_config.h&gt;?<br>
&gt;<br>
</div><div class="im">&gt; The safest place to add that include statement IMHO is \
inside the source<br> &gt; files apps/snmptrapd.c and apps/snmptrapd_auth.c.<br>
<br>
</div>See r19134 for my proposed fix.<br>
<br>
Please note that apps/snmptrapd.c already includes \
net-snmp/agent/net-snmp-agent-includes.h which includes \
net-snmp/agent/agent_module_config.h, so we&#39;re safe \
there.<br></blockquote></div><br>Thanks ! That fixed it at least on the trunk \
(haven&#39;t retested the branches yet).<br> <br>Bart.<br>



------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first

_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders


[prev in list] [next in list] [prev in thread] [next in thread]