Hello,

    I believe the change below is required for --enable-disk-io to
actually default to auto, which is what we promise in AS_HELP_STRING. On
the other hand, the comment insists that we should "do nothing" and that
the "auto is ok". Did I miss something?

>  AC_MSG_CHECKING([for DiskIO modules to be enabled])
>  squid_disk_module_candidates=""
>  squid_opt_enable_diskio="auto"  #values: no, yes, "auto"(=yes+detect modules)
>  AC_ARG_ENABLE(disk-io,
>    AS_HELP_STRING([--enable-disk-io="list of modules"],
>      [Build support for the list of disk I/O modules.
>       Set without a value or omitted, all available modules will be built.
>       See src/DiskIO for a list of available modules, or
>       Programmers Guide section on DiskIO
>       for details on how to build your custom disk module]), [ 
>  case $enableval in
> -  yes)
> -    ${TRUE}
> -    #do nothing, "auto" is ok
> +  yes|auto)
> +    squid_opt_enable_diskio=$enableval
>         ;;
>    no)
>      squid_opt_enable_diskio="no"
>         ;;
>    *)
>      squid_opt_enable_diskio="yes"
>      squid_disk_module_candidates=" `echo $enableval| sed -e 's/,/ /g;s/  */ 
> /g'` "
>      SQUID_CLEANUP_MODULES_LIST([squid_disk_module_candidates])
>         ;;
>    esac
>  ])
>  
>  # if requested to autodetect, find out what we have
>  if test $squid_opt_enable_diskio = "auto"; then
>      squid_opt_enable_diskio="yes"
>      
> SQUID_LOOK_FOR_MODULES([$srcdir/src/DiskIO],[squid_disk_module_candidates])
>  fi

Thank you,

Alex.

Reply via email to