Red Hat Bugzilla – Bug 198975
fdisk_wrapper.py has missing option-parm in sfdisk command
Last modified: 2007-11-30 17:11:37 EST
Description of problem:
from inspection, line 56 of fdisk_wrapper.py seems to omit an option parm for
the sfdisk -u option.
s/'-u', devname/'-u', 's', devname/
Version-Release number of selected component (if applicable):
I haven't actually seen any consequences, haven't tried to cause an error, but
runninf sfdisk -u without a units parm causes the devname arg to be ignored,
which I presume could cause confusion.
I believe I should have suggested an 'S' arg rather than 's'.
Another way to state it is to make line 56 look like line 87, which would give
res = execWithCapture(SFDISK, [SFDISK, '-l', '-uS', devname])
Line 56 uses fdisk (which doesn't allow argument to -u option), and not sfdisk,
which is in use on line 87.