diff options
author | Michael DeHaan <mdehaan@redhat.com> | 2006-05-08 15:02:06 -0400 |
---|---|---|
committer | Jim Meyering <jim@meyering.net> | 2006-05-08 15:02:06 -0400 |
commit | d4f71b4318fedf374844030095c6c8dd544f0e92 (patch) | |
tree | dd5bb88c2587a0ab4ea2638374ad4a71bbd868a9 /cobbler/item_distro.py | |
parent | 038a4383ccb6230f927960a34288c5cf6fbd3455 (diff) | |
download | third_party-cobbler-d4f71b4318fedf374844030095c6c8dd544f0e92.tar.gz third_party-cobbler-d4f71b4318fedf374844030095c6c8dd544f0e92.tar.xz third_party-cobbler-d4f71b4318fedf374844030095c6c8dd544f0e92.zip |
Interim checkin while straightening out exceptions. The last_error bit reminded me of the
thing I hated most about Microsoft SDK/DDK programming (that being, last_error and inconsistant
error handling), so it had to go.
Diffstat (limited to 'cobbler/item_distro.py')
-rw-r--r-- | cobbler/item_distro.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/cobbler/item_distro.py b/cobbler/item_distro.py index aa2d463..59250af 100644 --- a/cobbler/item_distro.py +++ b/cobbler/item_distro.py @@ -10,6 +10,7 @@ import utils import item import weakref import os +from cobbler_exception import CobblerException class Distro(item.Item): @@ -50,8 +51,7 @@ class Distro(item.Item): if utils.find_kernel(kernel): self.kernel = kernel return True - utils.set_error("no_kernel") - return False + raise CobblerException("no_kernel") def set_initrd(self,initrd): """ @@ -61,8 +61,7 @@ class Distro(item.Item): if utils.find_initrd(initrd): self.initrd = initrd return True - utils.set_error("no_initrd") - return False + raise CobblerException("no_initrd") def is_valid(self): """ |