[oe-commits] Maciej Borzecki : wic: bail out on kickstart parse errors
git at git.openembedded.org
git at git.openembedded.org
Sun Feb 8 08:00:49 UTC 2015
Module: openembedded-core.git
Branch: master
Commit: 0aa1221c16e65b3daac4aa9a9cfe1d5e9e6106c0
URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=0aa1221c16e65b3daac4aa9a9cfe1d5e9e6106c0
Author: Maciej Borzecki <maciej.borzecki at open-rnd.pl>
Date: Wed Feb 4 11:53:07 2015 +0100
wic: bail out on kickstart parse errors
Kickstart file errors are not emitted by the parser. In case errors, the
current code will continue to execute possibly creating a faulty or
incomplete image. The patch enables the parser to emit all encountered
errors. The error message is shown for the user and further execution is
ceased.
Signed-off-by: Maciej Borzecki <maciej.borzecki at open-rnd.pl>
Signed-off-by: Maciek Borzecki <maciek.borzecki at gmail.com>
Signed-off-by: Ross Burton <ross.burton at intel.com>
---
scripts/lib/wic/kickstart/__init__.py | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/scripts/lib/wic/kickstart/__init__.py b/scripts/lib/wic/kickstart/__init__.py
index 6000982..b1406a0 100644
--- a/scripts/lib/wic/kickstart/__init__.py
+++ b/scripts/lib/wic/kickstart/__init__.py
@@ -58,16 +58,13 @@ def read_kickstart(path):
def __init__(self):
superclass.__init__(self, mapping=commandMap[using_version])
- ks = ksparser.KickstartParser(KSHandlers(), errorsAreFatal=False)
+ ks = ksparser.KickstartParser(KSHandlers(), errorsAreFatal=True)
try:
ks.readKickstart(path)
except (kserrors.KickstartParseError, kserrors.KickstartError), err:
- if msger.ask("Errors occured on kickstart file, skip and continue?"):
- msger.warning("%s" % err)
- pass
- else:
- raise errors.KsError("%s" % err)
+ msger.warning("Errors occurred when parsing kickstart file: %s\n" % path)
+ msger.error("%s" % err)
return ks
More information about the Openembedded-commits
mailing list