[master] Warn users about liveinst usage of --updates (#1153550)

Samantha N. Bueno sbueno+anaconda at redhat.com
Tue Oct 28 12:39:24 UTC 2014


On Mon, Oct 27, 2014 at 05:17:58PM -0700, Brian C. Lane wrote:
> liveinst is a thin wrapper over anaconda. It can download updates using
> --updates=<url> but cannot understand --updates <url> without the =.
> Detect this and tell the user.

Ack.

> ---
>  data/liveinst/liveinst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/data/liveinst/liveinst b/data/liveinst/liveinst
> index 7f67e38..3436642 100755
> --- a/data/liveinst/liveinst
> +++ b/data/liveinst/liveinst
> @@ -113,6 +113,17 @@ for opt in `cat /proc/cmdline` $*; do
>      --inst.updates=*)
>          UPDATES="${opt#--inst.updates=}"
>          ;;
> +    --updates)
> +        title="updates error"
> +        text="liveinst requires --updates=<url> instead of --updates <url>"
> +        if which zenity &> /dev/null; then
> +            zenity --error --no-markup --title="$title" --text="$text"
> +        else
> +            echo "$title" >&2
> +            echo "$text" >&2
> +        fi
> +        exit 1
> +        ;;
>      ks=*|kickstart=*|--ks=*|--kickstart=*|inst.ks=*|--inst.ks=*|inst.kickstart=*|--inst-kickstart=*)
>          title="Configuration not supported"
>          text="Kickstart is not supported on live installs.  This installation will continue interactively."
> -- 
> 1.9.3
> 
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches


More information about the anaconda-patches mailing list