Show More
@@ -173,7 +173,12 b' def _get_hooks_client(extras):' | |||
|
173 | 173 | elif is_shadow_repo: |
|
174 | 174 | return HooksShadowRepoClient() |
|
175 | 175 | else: |
|
176 | return HooksDummyClient(extras['hooks_module']) | |
|
176 | try: | |
|
177 | import_module = extras['hooks_module'] | |
|
178 | except KeyError: | |
|
179 | log.error('Failed to get "hooks_module" from extras: %s', extras) | |
|
180 | raise | |
|
181 | return HooksDummyClient(import_module) | |
|
177 | 182 | |
|
178 | 183 | |
|
179 | 184 | def _call_hook(hook_name, extras, writer): |
@@ -186,7 +186,6 b' class SvnRemote(RemoteBase):' | |||
|
186 | 186 | # NOTE(marcink): short circuit the check for SVN repo |
|
187 | 187 | # the repos.open might be expensive to check, but we have one cheap |
|
188 | 188 | # pre condition that we can use, to check for 'format' file |
|
189 | ||
|
190 | 189 | if not os.path.isfile(os.path.join(path, 'format')): |
|
191 | 190 | return False |
|
192 | 191 |
General Comments 0
You need to be logged in to leave comments.
Login now