Re: [PATCH 1/2] configure: libnfs + gnutls

[Vincent Fu <vincentfu@gmail.com>]

On 7/25/25 2:17 AM, Luis Chamberlain wrote:
> The problem is that the configure script tries to get both libnfs and
> gnutls flags together, but gnutls is missing. Let me check the configure
> script's logic and fix it:
> 
> The configure script is trying to link both libnfs and gnutls together,
> but gnutls is not installed. Let me fix this by modifying the configure
> script to only require libnfs:
> 
> Generated-by: Claude AI
> Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
> ---
>   configure | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/configure b/configure
> index 9e69dc4b..1769ef38 100755
> --- a/configure
> +++ b/configure
> @@ -2359,8 +2359,8 @@ print_config "DAOS File System (dfs) Engine" "$dfs"
>   if test "$libnfs" != "no" ; then
>     if $(pkg-config libnfs > /dev/null 2>&1); then
>       libnfs="yes"
> -    libnfs_cflags=$(pkg-config --cflags libnfs gnutls)
> -    libnfs_libs=$(pkg-config --libs libnfs gnutls)
> +    libnfs_cflags=$(pkg-config --cflags libnfs)
> +    libnfs_libs=$(pkg-config --libs libnfs)
>     else
>       if test "$libnfs" = "yes" ; then
>         feature_not_found "libnfs" "libnfs"

The gnutls requirement was added via

https://lore.kernel.org/fio/4369d6e2-48d4-43cb-965a-b376dc559dad@gmail.com/T/#t

I can think of two ways to resolve this:

- Make sure gnutls is installed when building fio
- Modify the configure script to detect whether or not gnutls really is 
needed by libnfs and set up the flags accordingly

Vincent