diff options
| author | Petr Viktorin <pviktori@redhat.com> | 2015-07-15 16:38:06 +0200 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2015-07-31 15:22:19 +0200 |
| commit | b8c46f2a32d0d8c2dc6ef0867f85f63cf076a004 (patch) | |
| tree | 28e52d8fe2e3531e226d8c2249d6b7216f80bf48 /ipa-client/ipaclient | |
| parent | a908be2785d4388e3c97c7cd543c817c527d73c9 (diff) | |
| download | freeipa-b8c46f2a32d0d8c2dc6ef0867f85f63cf076a004.tar.gz freeipa-b8c46f2a32d0d8c2dc6ef0867f85f63cf076a004.tar.xz freeipa-b8c46f2a32d0d8c2dc6ef0867f85f63cf076a004.zip | |
Modernize number literals
Use Python-3 compatible syntax, without breaking compatibility with py 2.7
- Octals literals start with 0o to prevent confusion
- The "L" at the end of large int literals is not required as they use
long on Python 2 automatically.
- Using 'int' instead of 'long' for small numbers is OK in all cases except
strict type checking checking, e.g. type(0).
https://fedorahosted.org/freeipa/ticket/4985
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
Diffstat (limited to 'ipa-client/ipaclient')
| -rw-r--r-- | ipa-client/ipaclient/ipa_certupdate.py | 2 | ||||
| -rw-r--r-- | ipa-client/ipaclient/ipachangeconf.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/ipa-client/ipaclient/ipa_certupdate.py b/ipa-client/ipaclient/ipa_certupdate.py index a9530674c..26e02ea33 100644 --- a/ipa-client/ipaclient/ipa_certupdate.py +++ b/ipa-client/ipaclient/ipa_certupdate.py @@ -164,7 +164,7 @@ class CertUpdate(admintool.AdminTool): self.update_file(paths.CA_CRT, certs) - def update_file(self, filename, certs, mode=0444): + def update_file(self, filename, certs, mode=0o444): certs = (c[0] for c in certs if c[2] is not False) try: x509.write_certificate_list(certs, filename) diff --git a/ipa-client/ipaclient/ipachangeconf.py b/ipa-client/ipaclient/ipachangeconf.py index edf34f5ae..2c3185716 100644 --- a/ipa-client/ipaclient/ipachangeconf.py +++ b/ipa-client/ipaclient/ipachangeconf.py @@ -481,7 +481,7 @@ class IPAChangeConf: # we want to fail in that case shutil.copy2(file, (file + ".ipabkp")) - f = openLocked(file, 0644) + f = openLocked(file, 0o644) oldopts = self.parse(f) @@ -520,7 +520,7 @@ class IPAChangeConf: # The orign file did not exist pass - f = openLocked(file, 0644) + f = openLocked(file, 0o644) # Trunkate f.seek(0) |