On Wed, Dec 17, 2014 at 09:33:42AM -0500, mulhern wrote:
Things I ran across while working on tasks refactoring.
Use correct package... is dead obvious. Tests never hit it, because there
is a guard in writeLabel() that prevents that code path from executing.
Pylint never saw it because of the unfortunate choice of fs for a parameter.
Both of those problems should be fixed, but they already are as a side-effect
of other changes in my tasks branch, and I don't want any more rebasing
headaches.
Change allowDirty to False...is a no brainer.
Eliminate dirtyCB parameter...should not be too controversial and makes
next patch a bit easier.
Factor out calculation...shouldn't be too hard to accept.
The last two are really desirable as they ease refactoring in my tasks branch
a good deal. Otherwise, I get to spend a lot of time disentanglishing dirty
calculation from minimum size calculation, and if those dirty and error
values are never used, as they aren't in existing anaconda set up, that
would be a depressing task.
mulhern (6):
Use correct package for FSError.
Change allowDirty to False (from None).
Eliminate dirtyCB parameter from mountExistingSystem() params.
Factor out calculation of dirtyDevices and be less eager in
calculating.
Remove allowDirty parameter and dependent code.
Remove needsFSCheck() and what only it depends on.
blivet/__init__.py | 27 +--------------------------
blivet/formats/fs.py | 21 ---------------------
blivet/formats/fslabel.py | 2 +-
tests/formats_test/fstesting.py | 1 -
4 files changed, 2 insertions(+), 49 deletions(-)
These all look ok to me.
--
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)