bpo-35024: Remove redundant and possibly incorrect verbose message after writing '.pyc' (GH-9998)

Quentin Agren · miss-islington · commit 9e14e49f13ef · 2018-10-26T11:36:30.000-07:00
Since `SourceFileLoader.set_data()` catches exceptions raised by `_write_atomic()` and logs an informative message consequently, always logging successful outcome in 'SourceLoader.get_code()' seems redundant. https://bugs.python.org/issue35024
diff --git a/Lib/importlib/_bootstrap_external.py b/Lib/importlib/_bootstrap_external.py
@@ -920,7 +920,6 @@ def get_code(self, fullname):
                                               len(source_bytes))
             try:
                 self._cache_bytecode(source_path, bytecode_path, data)
-                _bootstrap._verbose_message('wrote {!r}', bytecode_path)
             except NotImplementedError:
                 pass
         return code_object
diff --git a/Misc/NEWS.d/next/Library/2018-10-25-15-43-32.bpo-35024.ltSrtr.rst b/Misc/NEWS.d/next/Library/2018-10-25-15-43-32.bpo-35024.ltSrtr.rst
@@ -0,0 +1,3 @@
+`importlib` no longer logs `wrote <bytecode path>` redundantly after
+`(created|could not create) <bytecode path>` is already logged.
+Patch by Quentin Agren.
diff --git a/Python/importlib_external.h b/Python/importlib_external.h

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+`importlib` no longer logs `wrote <bytecode path>` redundantly after
	`2`	+`(created\|could not create) <bytecode path>` is already logged.
	`3`	`+Patch by Quentin Agren.`