On Thu, 2009-12-24 at 11:31 +0000, Daniel Drake wrote: > This flag is documented and parsed, but not acted upon.
bump... Any further comments on this patch? > imgcreate/creator.py | 11 +++++++++-- > 1 files changed, 9 insertions(+), 2 deletions(-) > > diff --git a/imgcreate/creator.py b/imgcreate/creator.py > index 909f616..4deb49e 100644 > --- a/imgcreate/creator.py > +++ b/imgcreate/creator.py > @@ -23,6 +23,7 @@ import sys > import tempfile > import shutil > import logging > +import subprocess > > import selinux > import yum > @@ -702,11 +703,17 @@ class ImageCreator(object): > script = "/tmp/" + os.path.basename(path) > > try: > - subprocess.call([s.interp, script], > - preexec_fn = preexec, env = env) > + subprocess.check_call([s.interp, script], > + preexec_fn = preexec, env = env) > except OSError, (err, msg): > raise CreatorError("Failed to execute %%post script " > "with '%s' : %s" % (s.interp, msg)) > + except subprocess.CalledProcessError, err: > + if s.errorOnFail: > + raise CreatorError("%%post script failed with code %d " > + % err.returncode) > + logging.warning("ignoring %%post failure (code %d)" > + % err.returncode) > finally: > os.unlink(path) > -- Fedora-livecd-list mailing list Fedora-livecd-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-livecd-list