* `salt-call` default outputter returned from `salt.output.get_printout()` was `None` and `None` isn't callable neither outputs anything useful. In this case, the fault Outputter should be used.
* `get_printout()` also tested for `txt_out` which now is, in the cleaned up parsers, `text_out`. Fixed.
* Only use logging if the console logging has been properly setup, in case it hasn't been, write the message to `sys.stderr`.
* Removed the log argument passed to `salt.verify.check_user`, it's not needed anymore.
Exit if the configuration directory does not exist. This is because the initial parser relies on loading a configuration file, and, if not found, default values are used which won't lead the user to the initial problem, the configuration directory does not exist or is not the correct one.
Some commands and options are still missing completion,
but most do work well, including modules, functions,
grains and grain item values, nodegroups, salt-key operations, ...
This file should be installed into
/etc/bash_completion.d/salt
I am not sure where and how to do that,
so i leave this open for others to solve.
In solo mode, files are copied directly from below one of the file_roots instead
of from below the cache_dir. Therefore, when computing the relative path of the file
below the destination directory, different starting point needs to be used.
I had `host: 'loalhost'` (notice the misspelling) set for the grant but
the correct `host: 'localhost'` set for the user & database and when
applying highstate it would bail out with:
```
[INFO ] Executing state mysql_user.present for gladca_prod
[INFO ] User 'gladca_prod'@'localhost' has been created
[INFO ] {'gladca_prod': 'Present'}
[INFO ] Executing state mysql_grants.present for gladca_prod
[INFO ] User 'gladca_prod'@'loalhost' does not exist
Error running 'state.highstate': argument of type 'bool' is not iterable
```
This commit kind of reverts the previous "no more os.sep.join ..." commit by solving the issue which led to switching that code to `os.path.join`. Tested both on linux and windows.
On the parsers cleanup branch, I though about making the available options as equal as possible. Since the log_file setting for the master and the minion could be separately set, I made the logfile option for the key also log_file. Yet, since there's no key specific config file, it shares master, one could only specify the log_file settings from the cli, it could not be hardcoded on the config file, unless, we made a separate config file just for key.
So, in order to reduce required changes, and keep it all as backwards compatible as possible, the key log file options is now, once again, `--key-logfile` which will allow us to hardcode it in the masters config file as `key_logfile`.
This will also, hopefully make travis behave better too.
