[PATCH] Add "Starting installer..." messages

Chris Lumens clumens at redhat.com
Thu Jul 19 15:50:05 UTC 2012


> So, which would be better:
> 
>     prerel = ""
>     if product.isFinal:
>         prerel = " (pre-release)"
>     print "anaconda %s for %s %s%s started." % (
>            getAnacondaVersion(),
>            product.productName,
>            product.productVersion,
>            prerel,
>            )
> 
> Or:
> 
>     verdesc = "%s for %s %s" % (getAnacondaVersion(),
>                         product.productName, product.productVersion)
>     if product.isFinal:
>         print "anaconda %s started." % verdesc
>     else:
>         print "anaconda %s (pre-release) started." % verdesc
> 
> I think I prefer the latter.

Me too.

Though, this just occurred to me - do you expect this string to ever be
translated?  It's possible, because one could specify inst.lang= on the
command line.

- Chris


More information about the anaconda-patches mailing list