Created attachment 12062 [details]
Amend help message for samba-tool domain provision
This bug is raised as a result of the following mailing list message:
This bug covers points (a) and (b) in that email:
a) It isn't clear what domain is from 'samba-tool domain provison --help' it
stats: set domain.
It expects the netbios name.
b) The --realm option is essential but at the end of the help ...
https://lists.samba.org/archive/samba-technical/2016-April/113740.html suggests how to fix this.
Adding Andreas to CC since he is the one that noticed these issues.
What is currently holding me up from developing the second trivial patch is a need for a bit more clarity on how this --help output should look. It seems to me that if the --domain parameter is compulsory, then the heading text should say so,
and that if a realm has to be specified (or read in from an existing conf file) then perhaps that should be mentioned in the introductory text as well.
The reason that the realm is way down the bottom is that it is under a heading "Samba Common Options", along with configFile debuglevel and option.
There is a further issue in here, which perhaps doesn't need to be fixed as part of this patch. For some reason, the existing handling to check if the supplied Realm parameter is blank is not working, and so instead of raising the CommandError it is failing later at the (messier) ProvisioningError.
From talk on list, it seems this is going to be resolved from a future patch by Rowland. However, I can only mark status as NEW or RESOLVED, not cancelled, so I assume I should leave this as-is for now until test Rowland's patch is testable.
Created attachment 12073 [details]
Patch to clarify domain provision --help text
Added signed off by and bug headers.
Looks like the patch is b929008dc6f3f8ffeb376d87d30d2101567f75e9.