[lxc-devel] [PATCH] lxc-opensuse.in: Check if given argument is a valid release (was: [PATCH] Added explanation on how to use the template)

Serge Hallyn serge.hallyn at ubuntu.com
Tue Nov 18 17:37:59 UTC 2014


Quoting Johannes Kastl (mail at ojkastl.de):
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 06.11.14 Johannes Kastl wrote:
> > On 06.11.2014 Johannes Kastl wrote:
> 
> > I added some info to the usage output, on how to use the template
> > and that 12.3 is set as default
> 
> I added a check, to see if the given argument is really a valid
> release. At the moment, these are 12.3, 13.1 and 13.2.
> 
> Regards,
> Johannes
> - -- 
> The problem with the world is stupidity. Not saying there should be a
> capital punishment for stupidity, but why don't we just take the
> safety labels off of everything and let the problem solve itself?
> (Frank Zappa)
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1
> Comment: Using GnuPG with SeaMonkey - http://www.enigmail.net/
> 
> iEYEARECAAYFAlRmbLwACgkQzi3gQ/xETbJWxQCfbL+SnlmFjLU2kUi4CoZb4g7/
> BqQAnRWbBfzl7amQGdJS2S0qjkTcC6UD
> =DoDh
> -----END PGP SIGNATURE-----

> From a19b7e26f1994421328916b4ddef42ca994652e7 Mon Sep 17 00:00:00 2001
> From: Johannes Kastl <git at ojkastl.de>
> Date: Fri, 14 Nov 2014 21:20:19 +0100
> Subject: [PATCH 3/3] lxc-opensuse.in: Check if given argument is a valid
>  release
> 
> Signed-off-by: Johannes Kastl <git at ojkastl.de>

Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>

> ---
>  templates/lxc-opensuse.in | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/templates/lxc-opensuse.in b/templates/lxc-opensuse.in
> index 8c7cc7a..f26cc7f 100644
> --- a/templates/lxc-opensuse.in
> +++ b/templates/lxc-opensuse.in
> @@ -406,8 +406,29 @@ if [ -z "$path" ]; then
>  fi
>  
>  if [ -z "$DISTRO" ]; then
> +    echo ""
>      echo "No release selected, using openSUSE 12.3"
>      DISTRO=12.3
> +else
> +    echo ""
> +    case "$DISTRO" in
> +	12.3)
> +	    echo "Selected openSUSE 12.3"
> +	    ;;
> +
> +	13.1)
> +	    echo "Selected openSUSE 13.1"
> +	    ;;
> +
> +	13.2)
> +	    echo "Selected openSUSE 13.2"
> +	    ;;
> +
> +	*)
> +	    echo "You have chosen an invalid release, quitting..."
> +	    exit 1
> +	    ;;
> +    esac
>  fi
>  
>  
> -- 
> 2.1.2
> 

> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel



More information about the lxc-devel mailing list