diff options
author | Mike Frysinger <vapier@google.com> | 2021-02-02 15:49:11 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-02-02 15:49:11 +0000 |
commit | 8d4e66d9c2c10aba89fc0ce47d515b8872d03912 (patch) | |
tree | 0ef1bf0f8cb4d773c0457ad356b0d629ed1101d0 | |
parent | 018d98a5af5d78dc4e0b4ba412bae41afb6c99a2 (diff) | |
parent | 42e4aa4f4c041746bcc2b3427f35c928bfa3d291 (diff) | |
download | repohooks-8d4e66d9c2c10aba89fc0ce47d515b8872d03912.tar.gz |
config: switch to stdlib get() fallback logic am: 42e4aa4f4c
Original change: https://android-review.googlesource.com/c/platform/tools/repohooks/+/1567662
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I277a33975d2cfc201511a615d3d9e9d46a3eba69
-rw-r--r-- | rh/config.py | 22 |
1 files changed, 7 insertions, 15 deletions
diff --git a/rh/config.py b/rh/config.py index bd7e6fe..2c7b4af 100644 --- a/rh/config.py +++ b/rh/config.py @@ -63,15 +63,6 @@ class RawConfigParser(configparser.RawConfigParser): return default raise - def get(self, section, option, default=_UNSET): - """Return the value for |option| in |section| (with |default|).""" - try: - return configparser.RawConfigParser.get(self, section, option) - except (configparser.NoSectionError, configparser.NoOptionError): - if default is not _UNSET: - return default - raise - def items(self, section=_UNSET, default=_UNSET): """Return a list of (key, value) tuples for the options in |section|.""" if section is _UNSET: @@ -126,7 +117,8 @@ class PreUploadConfig(object): def custom_hook(self, hook): """The command to execute for |hook|.""" - return shlex.split(self.config.get(self.CUSTOM_HOOKS_SECTION, hook, '')) + return shlex.split(self.config.get( + self.CUSTOM_HOOKS_SECTION, hook, fallback='')) @property def builtin_hooks(self): @@ -136,13 +128,13 @@ class PreUploadConfig(object): def builtin_hook_option(self, hook): """The options to pass to |hook|.""" - return shlex.split(self.config.get(self.BUILTIN_HOOKS_OPTIONS_SECTION, - hook, '')) + return shlex.split(self.config.get( + self.BUILTIN_HOOKS_OPTIONS_SECTION, hook, fallback='')) def builtin_hook_exclude_paths(self, hook): """List of paths for which |hook| should not be executed.""" - return shlex.split(self.config.get(self.BUILTIN_HOOKS_EXCLUDE_SECTION, - hook, '')) + return shlex.split(self.config.get( + self.BUILTIN_HOOKS_EXCLUDE_SECTION, hook, fallback='')) @property def tool_paths(self): @@ -174,7 +166,7 @@ class PreUploadConfig(object): """Whether to skip hooks for merged commits.""" return rh.shell.boolean_shell_value( self.config.get(self.OPTIONS_SECTION, - self.OPTION_IGNORE_MERGED_COMMITS, None), + self.OPTION_IGNORE_MERGED_COMMITS, fallback=None), False) def update(self, preupload_config): |