Change to a semantically equivalent version of FileDevice.path

Prompted by an unused variable status message.

Signed-off-by: mulhern <amulhern@redhat.com>
Change param default [] to None and convert in method

A default of [] (or any mutable structure) is considered dangerous because the
default argument is packed up with the function at its definition. Thus, the
same object is shared among all execution instances of the function that
are called with the default argument.

Signed-off-by: mulhern <amulhern@redhat.com>
Indent with spaces, not tabs

Signed-off-by: mulhern <amulhern@redhat.com>
Move lines beneath __main__ into a main method

So that the variables now defined in main do not have global scope.

Signed-off-by: mulhern <amulhern@redhat.com>
Omit compile flag

It is not relevant to making updates for blivet.

Signed-off-by: mulhern <amulhern@redhat.com>
Keep first parameter "self"

Signed-off-by: mulhern <amulhern@redhat.com>
Remove appendiceal assignment

Signed-off-by: mulhern <amulhern@redhat.com>
Put docstring in class

Signed-off-by: mulhern <amulhern@redhat.com>
Remove unused variable but retain call for its side-effects

Signed-off-by: mulhern <amulhern@redhat.com>
Get rid of writeRandomUUID.

It is not used anywhere and pylint complains about all the subclasses
of FS in which it is not overridden.

Signed-off-by: mulhern <amulhern@redhat.com>
Suppress unused variable warning

Signed-off-by: mulhern <amulhern@redhat.com>
Do not import logging twice.

Signed-off-by: mulhern <amulhern@redhat.com>
Raise exceptions using the new syntax

Signed-off-by: mulhern <amulhern@redhat.com>
Rearrange _startEdd to be a little more obvious.

This change was prompted by an unused variable warning.

The method should behave identically, except for the unlikely
scenario where buf is "NIC=", in which case the previous version
would raise a ValueError, and the new version logs an error.