<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Aug 4, 2015 at 2:12 PM, Jan Tulak <span dir="ltr"><<a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><br>
<br>
----- Original Message -----<br>
> From: "Christoph Hellwig" <<a href="mailto:hch@infradead.org" target="_blank">hch@infradead.org</a>><br>
> To: "Jan Tulak" <<a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a>><br>
> Cc: <a href="mailto:hch@infradead.org" target="_blank">hch@infradead.org</a>, <a href="mailto:xfs@oss.sgi.com" target="_blank">xfs@oss.sgi.com</a><br>
> Sent: Tuesday, August 4, 2015 1:21:03 PM<br>
> Subject: Re: [PATCH v3 08/11] xfsprogs: Add fls check into autoconf<br>
><br>
> Can't you just use something like:<br>
><br>
> AC_CHECK_DECL([fls],<br>
>         [AC_DEFINE([HAVE_FLS_PROTOTYPE], [1],<br>
>        [Define to 1 if have fls prototype])],<br>
>        [],<br>
>        [#include <string.h>])<br>
><br>
<br>
</span>You mean to replace the entire autoconf part of the patch with this?<br>
I will try it. Autoconf is still a bit of a black magic for me, so I<br>
used the already existing checks. :-)<br></blockquote><div><br></div><div>Hi Chris,</div><div>did you want the result to be this? (I didn't get any clarification of what you mean, so I'm asking again. :-) )</div><div><br></div><div><div>#</div><div># Check if we have a flc call (Mac OS X)</div><div>#</div><div>AC_DEFUN([AC_HAVE_FLS],</div><div>  [ AC_CHECK_DECL([fls],</div><div>       have_fls=yes,</div><div>       [],</div><div>       [#include <string.h>]</div><div>       )</div><div>    AC_SUBST(have_fls)</div><div>  ]) </div><div><br></div></div><div>Cheers,</div><div>Jan</div><div><br></div></div>-- <br><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a><br></div></div>
</div></div>