Skip to content

Commit

Permalink
Create a function for only_strs to help document its purpose.
Browse files Browse the repository at this point in the history
  • Loading branch information
jaraco committed Feb 2, 2022
1 parent 9c4ed1a commit a3bc3d4
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions setuptools/command/easy_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -1334,9 +1334,7 @@ def create_home_path(self):
if not self.user:
return
home = convert_path(os.path.expanduser("~"))
for path in self.config_vars.values():
if not isinstance(path, str):
continue
for path in only_strs(self.config_vars.values()):
if path.startswith(home) and not os.path.isdir(path):
self.debug_print("os.makedirs('%s', 0o700)" % path)
os.makedirs(path, 0o700)
Expand Down Expand Up @@ -2306,6 +2304,13 @@ def current_umask():
return tmp


def only_strs(values):
"""
Exclude non-str values. Ref #3063.
"""
return filter(lambda val: isinstance(val, str), values)


class EasyInstallDeprecationWarning(SetuptoolsDeprecationWarning):
"""
Warning for EasyInstall deprecations, bypassing suppression.
Expand Down

0 comments on commit a3bc3d4

Please sign in to comment.