[blivet:master+? 6/9] Rewrite _parseSpec()

Vratislav Podzimek vpodzime at redhat.com
Fri Oct 24 09:18:34 UTC 2014


On Thu, 2014-10-23 at 13:57 -0400, mulhern wrote:
> This fixes a bug where most English specifiers would not be parsed
> unless the locale was a lot like English; after failing to be parsed
> in the English section, they still got their chance to be parsed in
> the locale-specific section, and if it was close enough, as it usually
> is for me, it would be parsed there.
> 
> Add a method, _parseUnits(), which handles the parsing of the units
> part of the spec. Abstract this for both translated and non-translated.
> 
> The change that introduced the bug was intended to make it so that
> "8 M" was interpreted as "8 MiB", not "8 MB". This patch undoes that
> change, so "8 M" is interpreted as "8 MB".
Same opinion as dlehman here. We should continue interpreting "8 M" as
"8 MiB" as we want to use binary units by default.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list