diff options
| author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-06-01 13:41:23 -0400 |
|---|---|---|
| committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-06-01 13:47:10 -0400 |
| commit | 52d49d3d0e983cd043c79fd5f387e6d49975f9aa (patch) | |
| tree | f2a7065f89ce3047a39fc5e24e1337523dfa4e58 /src/sbin | |
| parent | 8ddac82ee7100fcaec2aba5f80ed7bafefe2fe76 (diff) | |
| download | bcfg2-52d49d3d0e983cd043c79fd5f387e6d49975f9aa.tar.gz bcfg2-52d49d3d0e983cd043c79fd5f387e6d49975f9aa.tar.bz2 bcfg2-52d49d3d0e983cd043c79fd5f387e6d49975f9aa.zip | |
minor option cleanup
Diffstat (limited to 'src/sbin')
| -rwxr-xr-x | src/sbin/bcfg2 | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/sbin/bcfg2 b/src/sbin/bcfg2 index 3b5877c09..3cfdf38f6 100755 --- a/src/sbin/bcfg2 +++ b/src/sbin/bcfg2 @@ -95,12 +95,14 @@ class Client: self.logger.info("The following drivers are available:") self.logger.info(Bcfg2.Client.Tools.drivers) raise SystemExit(0) - if self.setup['remove'] and 'services' in self.setup['remove']: - self.logger.error("Service removal is nonsensical; removed services will only be disabled") - if self.setup['remove'] not in [None, 'all', - 'Services', 'Packages', - 'services', 'packages']: - self.logger.error("Got unknown argument %s for -r" % (self.setup['remove'])) + if self.setup['remove'] and 'services' in self.setup['remove'].lower(): + self.logger.error("Service removal is nonsensical; " + "removed services will only be disabled") + if (self.setup['remove'] and + self.setup['remove'].lower() not in ['all', 'services', + 'packages']): + self.logger.error("Got unknown argument %s for -r" % + self.setup['remove']) if self.setup["file"] and self.setup["cache"]: print("cannot use -f and -c together") raise SystemExit(1) |