diff options
author | Joe Guo <joeg@catalyst.net.nz> | 2018-07-30 18:13:57 +1200 |
---|---|---|
committer | Douglas Bagnall <dbagnall@samba.org> | 2018-08-24 07:49:25 +0200 |
commit | 4fc08d8f14e281fb7492d5a196e456637406c514 (patch) | |
tree | 08c9734fded7017fa5270ce499b3641ab9b8332b /python | |
parent | ada5af53f7502aaacd9cebb35a3e3288ec5b24bc (diff) | |
download | samba-4fc08d8f14e281fb7492d5a196e456637406c514.tar.gz |
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>
Diffstat (limited to 'python')
-rwxr-xr-x | python/examples/samr.py | 6 | ||||
-rw-r--r-- | python/samba/netcmd/__init__.py | 28 | ||||
-rw-r--r-- | python/samba/netcmd/domain.py | 58 | ||||
-rw-r--r-- | python/samba/netcmd/ldapcmp.py | 2 | ||||
-rw-r--r-- | python/samba/provision/__init__.py | 2 | ||||
-rw-r--r-- | python/samba/remove_dc.py | 2 | ||||
-rw-r--r-- | python/samba/tests/docs.py | 10 |
7 files changed, 54 insertions, 54 deletions
diff --git a/python/examples/samr.py b/python/examples/samr.py index 9b8e31e434b..49559e82023 100755 --- a/python/examples/samr.py +++ b/python/examples/samr.py @@ -44,7 +44,7 @@ def toArray((handle, array, num_entries)): for x in range(num_entries): ret.append((array.entries[x].idx, array.entries[x].name)) return ret - + def test_Connect(samr): """test the samr_Connect interface""" @@ -98,8 +98,8 @@ def test_EnumDomains(samr, handle): samr.Close(dom_handle) if len(sys.argv) != 2: - print "Usage: samr.js <BINDING>" - sys.exit(1) + print "Usage: samr.js <BINDING>" + sys.exit(1) binding = sys.argv[1] diff --git a/python/samba/netcmd/__init__.py b/python/samba/netcmd/__init__.py index 9b144d97f43..ceb1be93113 100644 --- a/python/samba/netcmd/__init__.py +++ b/python/samba/netcmd/__init__.py @@ -29,17 +29,17 @@ class Option(optparse.Option): # This help formatter does text wrapping and preserves newlines class PlainHelpFormatter(optparse.IndentedHelpFormatter): def format_description(self,description=""): - desc_width = self.width - self.current_indent - indent = " "*self.current_indent - paragraphs = description.split('\n') - wrapped_paragraphs = [ - textwrap.fill(p, - desc_width, - initial_indent=indent, - subsequent_indent=indent) - for p in paragraphs] - result = "\n".join(wrapped_paragraphs) + "\n" - return result + desc_width = self.width - self.current_indent + indent = " "*self.current_indent + paragraphs = description.split('\n') + wrapped_paragraphs = [ + textwrap.fill(p, + desc_width, + initial_indent=indent, + subsequent_indent=indent) + for p in paragraphs] + result = "\n".join(wrapped_paragraphs) + "\n" + return result def format_epilog(self, epilog): if epilog: @@ -160,11 +160,11 @@ class Command(object): undetermined_max_args = False for i, arg in enumerate(self.takes_args): if arg[-1] != "?" and arg[-1] != "*": - min_args += 1 + min_args += 1 if arg[-1] == "+" or arg[-1] == "*": - undetermined_max_args = True + undetermined_max_args = True else: - max_args += 1 + max_args += 1 if (len(args) < min_args) or (not undetermined_max_args and len(args) > max_args): parser.print_usage() return -1 diff --git a/python/samba/netcmd/domain.py b/python/samba/netcmd/domain.py index 9bdd17e3bcc..6f2e04b8ad7 100644 --- a/python/samba/netcmd/domain.py +++ b/python/samba/netcmd/domain.py @@ -156,31 +156,31 @@ def get_testparm_var(testparm, smbconf, varname): return "" try: - import samba.dckeytab + import samba.dckeytab except ImportError: - cmd_domain_export_keytab = None + cmd_domain_export_keytab = None else: - class cmd_domain_export_keytab(Command): - """Dump Kerberos keys of the domain into a keytab.""" + class cmd_domain_export_keytab(Command): + """Dump Kerberos keys of the domain into a keytab.""" - synopsis = "%prog <keytab> [options]" + synopsis = "%prog <keytab> [options]" - takes_optiongroups = { - "sambaopts": options.SambaOptions, - "credopts": options.CredentialsOptions, - "versionopts": options.VersionOptions, - } + takes_optiongroups = { + "sambaopts": options.SambaOptions, + "credopts": options.CredentialsOptions, + "versionopts": options.VersionOptions, + } - takes_options = [ - Option("--principal", help="extract only this principal", type=str), - ] + takes_options = [ + Option("--principal", help="extract only this principal", type=str), + ] - takes_args = ["keytab"] + takes_args = ["keytab"] - def run(self, keytab, credopts=None, sambaopts=None, versionopts=None, principal=None): - lp = sambaopts.get_loadparm() - net = Net(None, lp) - net.export_keytab(keytab=keytab, principal=principal) + def run(self, keytab, credopts=None, sambaopts=None, versionopts=None, principal=None): + lp = sambaopts.get_loadparm() + net = Net(None, lp) + net.export_keytab(keytab=keytab, principal=principal) class cmd_domain_info(Command): @@ -597,7 +597,7 @@ class cmd_domain_dcpromo(Command): takes_options.extend(common_provision_join_options) if samba.is_ntvfs_fileserver_built(): - takes_options.extend(common_ntvfs_options) + takes_options.extend(common_ntvfs_options) takes_args = ["domain", "role?"] @@ -1893,7 +1893,7 @@ class DomainTrustCommand(Command): def netr_DomainTrust_to_name(self, t): if t.trust_type == lsa.LSA_TRUST_TYPE_DOWNLEVEL: - return t.netbios_name + return t.netbios_name return t.dns_name @@ -1901,11 +1901,11 @@ class DomainTrustCommand(Command): primary = None primary_parent = None for _t in a: - if _t.trust_flags & netlogon.NETR_TRUST_FLAG_PRIMARY: - primary = _t - if not _t.trust_flags & netlogon.NETR_TRUST_FLAG_TREEROOT: - primary_parent = a[_t.parent_index] - break + if _t.trust_flags & netlogon.NETR_TRUST_FLAG_PRIMARY: + primary = _t + if not _t.trust_flags & netlogon.NETR_TRUST_FLAG_TREEROOT: + primary_parent = a[_t.parent_index] + break if t.trust_flags & netlogon.NETR_TRUST_FLAG_IN_FOREST: if t is primary_parent: @@ -3432,10 +3432,10 @@ class cmd_domain_trust_namespaces(DomainTrustCommand): self.outf.write("Stored uPNSuffixes attributes[%d]:\n" % len(stored_upn_vals)) for v in stored_upn_vals: - self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) + self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) self.outf.write("Stored msDS-SPNSuffixes attributes[%d]:\n" % len(stored_spn_vals)) for v in stored_spn_vals: - self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) + self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) if not require_update: return @@ -3494,10 +3494,10 @@ class cmd_domain_trust_namespaces(DomainTrustCommand): self.outf.write("Update uPNSuffixes attributes[%d]:\n" % len(update_upn_vals)) for v in update_upn_vals: - self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) + self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) self.outf.write("Update msDS-SPNSuffixes attributes[%d]:\n" % len(update_spn_vals)) for v in update_spn_vals: - self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) + self.outf.write("TLN: %-32s DNS[*.%s]\n" % ("", v)) update_msg = ldb.Message() update_msg.dn = stored_msg.dn diff --git a/python/samba/netcmd/ldapcmp.py b/python/samba/netcmd/ldapcmp.py index 4e5abad7cfe..2d302fa1e24 100644 --- a/python/samba/netcmd/ldapcmp.py +++ b/python/samba/netcmd/ldapcmp.py @@ -859,7 +859,7 @@ class LDAPBundel(object): self.outf.write("Failed search of base=%s\n" % self.search_base) raise for x in res: - dn_list.append(x["dn"].get_linearized()) + dn_list.append(x["dn"].get_linearized()) # global summary # diff --git a/python/samba/provision/__init__.py b/python/samba/provision/__init__.py index 26d11a59ded..dd875f8065a 100644 --- a/python/samba/provision/__init__.py +++ b/python/samba/provision/__init__.py @@ -678,7 +678,7 @@ def guess_names(lp=None, hostname=None, domain=None, dnsdomain=None, dnsdomain = netbiosname.lower() if rootdn is None: - rootdn = domaindn + rootdn = domaindn if configdn is None: configdn = "CN=Configuration," + rootdn diff --git a/python/samba/remove_dc.py b/python/samba/remove_dc.py index e6513ae04fa..d814a8e8edd 100644 --- a/python/samba/remove_dc.py +++ b/python/samba/remove_dc.py @@ -309,7 +309,7 @@ def offline_remove_ntds_dc(samdb, except LdbError as e5: (enum, estr) = e5.args if enum == ldb.ERR_NO_SUCH_OBJECT: - raise DemoteException("Given DN %s doesn't exist" % ntds_dn) + raise DemoteException("Given DN %s doesn't exist" % ntds_dn) else: raise if (len(msgs) == 0): diff --git a/python/samba/tests/docs.py b/python/samba/tests/docs.py index 0f029ae02d2..86cb06c3adf 100644 --- a/python/samba/tests/docs.py +++ b/python/samba/tests/docs.py @@ -50,7 +50,7 @@ def get_documented_parameters(sourcedir): for parameter in root: name = parameter.attrib.get('name') if parameter.attrib.get('removed') == "1": - continue + continue yield name syn = parameter.findall('synonym') if syn is not None: @@ -74,7 +74,7 @@ def get_documented_tuples(sourcedir, omit_no_default=True): name = parameter.attrib.get("name") param_type = parameter.attrib.get("type") if parameter.attrib.get('removed') == "1": - continue + continue values = parameter.findall("value") defaults = [] for value in values: @@ -193,7 +193,7 @@ class SmbDotConfTests(TestCase): elif context == "S": section = "test" else: - self.fail("%s has no valid context" % param) + self.fail("%s has no valid context" % param) p = subprocess.Popen(program + ["-s", self.smbconf, "--section-name", section, "--parameter-name", param], stdout=subprocess.PIPE, stderr=subprocess.PIPE, cwd=self.topdir).communicate() @@ -223,7 +223,7 @@ class SmbDotConfTests(TestCase): elif context == "S": section = "test" else: - self.fail("%s has no valid context" % param) + self.fail("%s has no valid context" % param) p = subprocess.Popen(program + ["-s", self.smbconf, "--section-name", section, "--parameter-name", param, "--option", "%s = %s" % (param, default)], @@ -277,7 +277,7 @@ class SmbDotConfTests(TestCase): elif context == "S": section = "test" else: - self.fail("%s has no valid context" % param) + self.fail("%s has no valid context" % param) value_to_use = arbitrary.get(param_type) if value_to_use is None: |