Skip to content
Snippets Groups Projects
Commit 333b70e6 authored by ian%hixie.ch's avatar ian%hixie.ch
Browse files

Should have been checking for a defined value, not a true value -- there is...

Should have been checking for a defined value, not a true value -- there is nothing to stop a failure from being '0', for instance. (The important thing is to be consistent with the SelectingServiceList code, really, and it uses defined().)
parent 7c7a61f1
No related branches found
No related tags found
No related merge requests found
......@@ -81,7 +81,7 @@ sub cmdSetup {
$result = $app->getSelectingServiceList('setup.install')->setupInstall($app);
}
# report on the result
if ($result) {
if (defined($result)) {
# if we failed, first report that then signal that
# configuration has ended
$self->dump(9, "Failed to setup because argument '$result' was missing.");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment