Re: Option handling in grub-mkconfig

["Grégoire Sutre" <gregoire.sutre@gmail.com>]

On 05/21/2010 12:57 PM, Colin Watson wrote:

Thanks for the patch, it works fine.  I'm just wondering: Is keeping
`-o' worth the extra complication?

Grégoire

> How about:
>
> === modified file 'util/grub-mkconfig.in'
> --- util/grub-mkconfig.in	2010-04-19 19:25:41 +0000
> +++ util/grub-mkconfig.in	2010-05-21 10:54:18 +0000
> @@ -50,7 +50,13 @@ EOF
>   }
>
>   # Check the arguments.
> +next_grub_cfg=false
>   for option in "$@"; do
> +    if $next_grub_cfg; then
> +	grub_cfg=$option
> +	next_grub_cfg=false
> +	continue
> +    fi
>       case "$option" in
>       -h | --help)
>   	usage
> @@ -59,8 +65,7 @@ for option in "$@"; do
>   	echo "$0 (GNU GRUB ${package_version})"
>   	exit 0 ;;
>       -o)
> -	shift
> -	grub_cfg=$1
> +	next_grub_cfg=:
>   	;;
>       --output=*)
>   	grub_cfg=`echo "$option" | sed 's/--output=//'`
> @@ -72,6 +77,11 @@ for option in "$@"; do
>   	;;
>       esac
>   done
> +if $next_grub_cfg; then
> +    echo "Missing argument to \`-o'" 1>&2
> +    usage
> +    exit 1
> +fi
>
>   . ${libdir}/grub/grub-mkconfig_lib
>
>