[blivet:master 0/6] Some patches for formats package.

Brian C. Lane bcl at redhat.com
Fri Dec 19 01:24:07 UTC 2014


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)


More information about the anaconda-patches mailing list