<div dir="ltr">Thanks for the help.&nbsp; I am attaching an alternative patch.<br><br>I did not think it was proper to use a BUILD_WITH_... variable since I did not see a --with-____ configure option for ipvs.&nbsp; I followed your second suggestion with this new patch.<br>
<br><div class="gmail_quote">On Mon, Aug 4, 2008 at 15:21, Sebastian Harl <span dir="ltr">&lt;<a href="mailto:sh@tokkee.org">sh@tokkee.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Boian,<br>
<div class="Ih2E3d"><br>
On Mon, Aug 04, 2008 at 02:02:21PM -0600, Boian Berberov wrote:<br>
&gt; I am writing an updated ebuild for gentoo and I made this patch that allows<br>
&gt; ip_vs.h to be located and the ipvs module to build.<br>
<br>
</div>Thanks for your patch. It basically looks fine for me. See a few minor<br>
comments below though. If you update your patch accordingly, it would<br>
imho be well suited for inclusion.<br>
<br>
&gt; --- configure.in.orig 2008-08-04<br>
&gt; +++ <a href="http://configure.in" target="_blank">configure.in</a> &nbsp; &nbsp; &nbsp;2008-08-04<br>
&gt; @@ -2332,6 +2338,8 @@<br>
&gt; &nbsp; &nbsp; &nbsp; if test &quot;x$have_net_ip_vs_h&quot; = &quot;xyes&quot; -o &quot;x$have_ip_vs_h&quot; = &quot;xyes&quot;<br>
&gt; &nbsp; &nbsp; &nbsp; then<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; plugin_ipvs=&quot;yes&quot;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; IPVS_CFLAGS=&quot;$ipvs_cflags&quot;<br>
&gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; AC_SUBST(IPVS_CFLAGS)<br>
&gt; &nbsp; &nbsp; &nbsp; fi<br>
&gt; &nbsp;fi<br>
<br>
Usually, those kinds of variables are called BUILD_WITH_&lt;module&gt;_CFLAGS.<br>
So, to be consistent with the rest of collectd, IPVS_CFLAGS should be<br>
renamed to BUILD_WITH_IPVS_CFLAGS.<br>
<br>
&gt; --- src/Makefile.am.orig &nbsp; &nbsp; &nbsp;2008-08-04<br>
&gt; +++ src/Makefile.am &nbsp; 2008-08-04<br>
&gt; @@ -311,6 +311,7 @@<br>
&gt; &nbsp;if BUILD_PLUGIN_IPVS<br>
&gt; &nbsp;pkglib_LTLIBRARIES += <a href="http://ipvs.la" target="_blank">ipvs.la</a><br>
&gt; &nbsp;ipvs_la_SOURCES = ipvs.c<br>
&gt; +ipvs_la_CFLAGS = $(IPVS_CFLAGS)<br>
&gt; &nbsp;ipvs_la_LDFLAGS = -module -avoid-version<br>
&gt; &nbsp;collectd_LDADD += &quot;-dlopen&quot; <a href="http://ipvs.la" target="_blank">ipvs.la</a><br>
&gt; &nbsp;collectd_DEPENDENCIES += <a href="http://ipvs.la" target="_blank">ipvs.la</a><br>
<br>
Alternatively, you could simply AC_SUBST() KERNEL_DIR and use<br>
&quot;-I$(KERNEL_DIR)&quot; in Makefile.am directly. I guess, I&#39;d use the latter<br>
solution but either one is fine for me, so implement whatever you like<br>
better ;-)<br>
<br>
Cheers,<br>
Sebastian<br>
<font color="#888888"><br>
--<br>
Sebastian &quot;tokkee&quot; Harl +++ GnuPG-ID: 0x8501C7FC +++ <a href="http://tokkee.org/" target="_blank">http://tokkee.org/</a><br>
<br>
Those who would give up Essential Liberty to purchase a little Temporary<br>
Safety, deserve neither Liberty nor Safety. &nbsp; &nbsp; &nbsp; &nbsp; -- Benjamin Franklin<br>
<br>
</font><br>-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1.4.6 (GNU/Linux)<br>
<br>
iD8DBQFIl3LtEFEKc4UBx/wRAjTuAJ96zCBKglvoJQYzW0IikfrzKVD2wgCfSVbn<br>
SnqJsvhx8iYgJgZPW32ofQI=<br>
=Z/E8<br>
-----END PGP SIGNATURE-----<br>
<br></blockquote></div><br></div>