s4/scripting/demodirsync: print usage if no host named

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/selftest/knownfail.d/usage b/selftest/knownfail.d/usage
index c5028e1d31f69943e5de5a5cfdcac9b6e9408231..0e125c6317de00ed26ef012353985a812aeaaaeb 100644 (file)
--- a/selftest/knownfail.d/usage
+++ b/selftest/knownfail.d/usage
@@ -1,5 +1,4 @@
 samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_chgtdcpass.none.
-samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_demodirsync_py.none.
 samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_findprovisionusnranges.none.
 samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_rebuildextendeddn.none.
 samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_renamedc.none.

diff --git a/source4/scripting/devel/demodirsync.py b/source4/scripting/devel/demodirsync.py
index 260104cc198625e75e300e9871a8a42afdd7cb67..e615c0fb865cdd864dfec3e1014e51a6dec1c646 100755 (executable)
--- a/source4/scripting/devel/demodirsync.py
+++ b/source4/scripting/devel/demodirsync.py
@@ -28,6 +28,9 @@ creds = credopts.get_credentials(lp)
 
 opts = parser.parse_args()[0]
 
+if opts.host is None:
+    print("Usage: demodirsync.py --host HOST [-b BASE]")
+    sys.exit(1)
 
 def printdirsync(ctl):
         arr = ctl.split(':')