This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/139/ |
On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:
mof/mounting.mof (Diff revision 1) 107 "Allow an ordinary user to mount the filesystem. Corresponds to 'auto' "Shouldn't this be "Corresponds to 'user' mount option" ?
Yes, this is fixed in [3/10] part of the series.
On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:
mof/mounting.mof (Diff revision 1) 152 "Method to create and populate an LMI_MountedFileSystemSetting instance. This "an -> a
I'm pretty sure 'an' is correct here. Try pronouncing 'L' (pronounciation begins with a vowel).
On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:
mof/mounting.mof (Diff revision 1) 212 "Filesystem type. If NULL, perform a binding mount. If binding a ""If binding a local filesystem" - shouldn't it be "If mounting a local filesystem"?
Yes, it should be 'mounting'. I'll fix it.
- Jan
On April 8th, 2013, 3:22 p.m. CEST, Jan Synacek wrote:
Review request for OpenLMI Developers.
By Jan Synacek.
Updated April 8, 2013, 3:22 p.m.
Repository:
openlmi-storage
Description
Diffs
|