[OE-core] [PATCH 3/5] wic: check that filesystem is specified for a rootfs partition

[Maciej Borzęcki]

On Wed, Nov 9, 2016 at 10:44 AM, Ed Bartosh <ed.bartosh at linux.intel.com> wrote:
> On Tue, Nov 08, 2016 at 04:56:09PM +0100, Maciej Borzecki wrote:
>> Signed-off-by: Maciej Borzecki <maciej.borzecki at rndity.com>
>> ---
>>  scripts/lib/wic/partition.py | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/scripts/lib/wic/partition.py b/scripts/lib/wic/partition.py
>> index 8adc698240c8e3bd9f4118663a5d7a167e0bb4a4..24e657592738dc7c5cdff78e3740d7c373021e9d 100644
>> --- a/scripts/lib/wic/partition.py
>> +++ b/scripts/lib/wic/partition.py
>> @@ -184,6 +184,10 @@ class Partition():
>>          if os.path.isfile(rootfs):
>>              os.remove(rootfs)
>>
>> +        if not self.fstype:
>> +            msger.error("File system for partition %s not specified in kickstart, " \
>> +                        "use --fstype option" % (self.mountpoint))
>> +
> Would it make sense to make --fstype mandatory in ksparser?

I'm afraid that would make fstype mandatory in all cases, while we only
need it for rootfs source plugin.


-- 
Maciej Borzecki
RnDity