Douglas Bagnall
bc8201bdf4
samba-tool ldapcmp: use ValueError, not obsolete StandardError
...
The error is in the value, and StandardError is not in Python 3
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Noel Power <npower@samba.org >
2018-10-25 21:45:57 +02:00
Noel Power
370dcbb791
samba_tool: Enclose iterator with list
...
Really strange bug caused by map being updated while being iterated.
This caused keys to be skipped and inconsistent and incorrect
results from ldapcmp.
Signed-off-by: Noel Power <noel.power@suse.com >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-10-23 05:50:27 +02:00
Noel Power
e5cac2b843
python/samba: misc use of str for ldb.bytes
...
Signed-off-by: Noel Power <noel.power@suse.com >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-10-23 05:50:27 +02:00
Joe Guo
deb819913d
PEP8: fix E127: continuation line over-indented for visual indent
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-09-06 12:10:12 +02:00
Joe Guo
dc02a4c35f
PEP8: fix E125: continuation line with same indent as next logical line
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-09-06 12:10:12 +02:00
Douglas Bagnall
08d1573bd5
PEP8: add spaces after operators
...
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-08-24 07:49:31 +02:00
Joe Guo
9f5bbcc10a
PEP8: fix E713: test for membership should be 'not in'
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:31 +02:00
Joe Guo
211c9a5f85
PEP8: fix E302: expected 2 blank lines, found 1
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:29 +02:00
Joe Guo
c809a86023
PEP8: fix E261: at least two spaces before inline comment
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:29 +02:00
Joe Guo
12d3fbe15c
PEP8: fix E231: missing whitespace after ','
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:28 +02:00
Joe Guo
87bbc2df97
PEP8: fix E226: missing whitespace around arithmetic operator
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:28 +02:00
Joe Guo
32266d2d48
PEP8: fix E225: missing whitespace around operator
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:28 +02:00
Joe Guo
fb5ea356dd
PEP8: fix E222: multiple spaces after operator
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:27 +02:00
Joe Guo
562411bd95
PEP8: fix E202: whitespace before ')'
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:27 +02:00
Joe Guo
a9551edaee
PEP8: fix E201: whitespace after '('
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:27 +02:00
Joe Guo
5d532543ab
PEP8: fix E128: continuation line under-indented for visual indent
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:27 +02:00
Joe Guo
484ce0634b
PEP8: fix E123: closing bracket does not match indentation of opening bracket's line
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:26 +02:00
Joe Guo
297faf3252
PEP8: fix E122: continuation line missing indentation or outdented
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:26 +02:00
Joe Guo
4fc08d8f14
PEP8: fix E111: indentation is not a multiple of four
...
Signed-off-by: Joe Guo <joeg@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
2018-08-24 07:49:25 +02:00
Douglas Bagnall
bddf66c460
samba-tool ldapcmp: remove duplicate takes_optiongroups attribute
...
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-03-21 04:36:22 +01:00
Noel Power
52729d3549
samba-tool: convert 'except X, (tuple)' to 'except X as e'
...
In addition to converting the except line another line is also added
for each except to extract the tuple contents.
Signed-off-by: Noel Power <noel.power@suse.com >
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-02-28 23:01:40 +01:00
Douglas Bagnall
a485ac3243
samba-tool: convert 'except X, e' to 'except X as e' for all X
...
This is needed for Python 3 and is compatible with python 2.6
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2018-02-15 00:18:30 +01:00
Garming Sam
fafc6da6ab
ldapcmp: Improve the difference checker of ldapcmp for 2012 R2
...
There are a number of new attributes which may be considered DNs.
Signed-off-by: Garming Sam <garming@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Autobuild-User(master): Garming Sam <garming@samba.org >
Autobuild-Date(master): Thu Dec 21 03:41:19 CET 2017 on sn-devel-144
2017-12-21 03:41:19 +01:00
Garming Sam
0efc061a62
ldapcmp: Add otherWellKnownObjects to ignore when using --two
...
wellKnownObjects already exists in this list.
Signed-off-by: Garming Sam <garming@catalyst.net.nz >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
2017-12-20 23:13:11 +01:00
Stefan Metzmacher
a0e60e96aa
samba-tool/ldapcmp: ignore differences of whenChanged
...
This is implicitly replicated, but may diverge on updates of non-replicated
attributes.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12129
Signed-off-by: Stefan Metzmacher <metze@samba.org >
Reviewed-by: Ralph Boehme <slow@samba.org >
Autobuild-User(master): Ralph Böhme <slow@samba.org >
Autobuild-Date(master): Mon Aug 8 17:34:24 CEST 2016 on sn-devel-144
2016-08-08 17:34:24 +02:00
Stefan Metzmacher
eee14f775e
samba-tool/ldapcmp: update the list of non replicated attributes
...
Bug: https://bugzilla.samba.org/show_bug.cgi?id=10788
Signed-off-by: Stefan Metzmacher <metze@samba.org >
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org >
Autobuild-Date(master): Tue Sep 2 03:49:49 CEST 2014 on sn-devel-104
2014-09-02 03:49:48 +02:00
Björn Baumbach
4b4f4e0f4f
samba-tool ldapcmp: fix a typo
...
Signed-off-by: Björn Baumbach <bb@sernet.de >
Reviewed-by: Volker Lendecke <vl@samba.org >
Reviewed-by: Michael Adam <obnox@samba.org >
2014-05-03 01:51:08 +02:00
Andrew Bartlett
5d42260eec
samba-tool ldapcmp: Remove the GUID -> name mappings
...
These mappings are very convenient, however because they are not
one-to-one, they lead to differences being reported when none exist,
dependent only on the order the schema searches return results in.
Sadly the time saved by the names is offset by the time wasted chasing
the 'differences' that don't exist.
This in turn fixes some tests that were previously knownfail
Andrew Bartlett
Reviewed-by: Stefan Metzmacher <metze@samba.org >
2013-03-25 11:35:04 +01:00
Andrew Bartlett
24c4d818d1
samba-tool ldapcmp: Add support for checking DNSDOMAIN and DNSFOREST by default
...
Reviewed-by: Stefan Metzmacher <metze@samba.org >
2013-03-04 08:31:05 +01:00
Andrew Bartlett
606f5d6cc6
samba-tool ldapcmp: Add --skip-missing-dn to not error on DNs present in one DB but not the other
...
This is needed to compare some parts of the database, particularly in --two mode, which
are just never going to have exactly the same DNs.
Andrew Bartlett
Reviewed-by: Stefan Metzmacher <metze@samba.org >
2013-03-02 21:52:20 +01:00
Jelmer Vernooij
87afc3aee1
Move python modules from source4/scripting/python/ to python/.
...
Reviewed-by: Andrew Bartlett <abartlet@samba.org >
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org >
Autobuild-Date(master): Sat Mar 2 03:57:34 CET 2013 on sn-devel-104
2013-03-02 03:57:34 +01:00