fixed english mistakes and some logical errors

Author: Patrik Kopkan

news/6340.feature

@@ -1,3 +1,4 @@
-Added a new option `--save-wheel-names` to command `wheel`.
-The option takes path and creates file in which
-is saved the basenames of wheels outputed by `wheel` command.
+Added a new option ``--save-wheel-names`` to command ``wheel``.
+The option takes a path and creates a file in which
+the filenames of built or downloaded wheels will be stored,
+separated by newlines

src/pip/_internal/commands/wheel.py

@@ -102,7 +102,7 @@ def __init__(self, *args, **kw):
             dest='path_to_wheelnames',
             action='store',
             metavar='path',
-            help="stores the filenames of wheels in file of given path"
+            help="stores the filenames of the built or downloaded wheels in a new file of given path"
         )
 
         index_opts = cmdoptions.make_option_group(
@@ -191,13 +191,16 @@ def run(self, options, args):
                         for req in requirement_set.requirements.values():
                             if req.link.filename.endswith('whl'):
                                 entries_to_save.append(req.link.filename)
-
-                        with open(wb.path_to_wheelnames, 'w') as file:
-                            file.write(
-                                '\n'.join(
-                                    entries_to_save
-                                ) + '\n'
-                            )
+                        try:
+                            with open(wb.path_to_wheelnames, 'w') as file:
+                                file.write(
+                                    '\n'.join(
+                                        entries_to_save
+                                    ) + '\n'
+                                )
+                        except PermissionError as e:
+                            logger.error('Cannot write to the given path: %s', wb.path_to_wheelnames)
+                            logger.error('PermissionError: %s', str(e))
 
                 except PreviousBuildDirError:
                     options.no_clean = True

tests/functional/test_wheel.py

@@ -249,8 +249,7 @@ def test_legacy_wheels_are_not_confused_with_other_files(script, tmpdir, data):
 
 
 def test_pip_option_save_wheel_name(script, data):
-    """
-    Test 'pip wheel ' success.
+    """Check if the option saves the filenames of built wheels
     """
     script.pip(
         'wheel', '--no-index', '-f', data.find_links,
@@ -262,6 +261,6 @@ def test_pip_option_save_wheel_name(script, data):
                         'simple-3.0-py%s-none-any.whl' % pyversion[0],
                         ]
     wheelnames_path = script.scratch_path / 'wheelnames'
-    wheelnames_file = open(wheelnames_path, 'r')
-    wheelnames_entries = (wheelnames_file.read()).splitlines()
+    with open(wheelnames_path, 'r') as wheelnames_file:
+        wheelnames_entries = (wheelnames_file.read()).splitlines()
     assert wheel_file_names == wheelnames_entries