[PATCH 2/2] Move code related to finding existing installations to anaconda

Chris Lumens clumens at redhat.com
Tue Jan 27 21:01:12 UTC 2015


> +def populate_storage_roots(storage):

Likewise here - only one new function named differently from all the
others.

More generally, if you have run these through pylint and all and they
work, I guess I am okay with it.  I'm not real excited by moving storage
stuff back into anaconda in general, given all the time we spent getting
rid of it.  To me it makes sense that blivet has a concept of
autopartitioning and existing installed systems.  I can imagine that
other blivet-using things are going to want to use that code, and now
they're going to have to pull in anaconda for the same functionality.

- Chris


More information about the anaconda-patches mailing list