diff options
| author | Matthew Leonberger <linuxunil@gmail.com> | 2017-09-24 20:43:41 +0900 |
|---|---|---|
| committer | Matthew Leonberger <linuxunil@gmail.com> | 2017-09-24 20:43:41 +0900 |
| commit | f2db75cea99e8376457c610da06a0a583503e595 (patch) | |
| tree | 28fe37fed0351b335f9af0b3a09a33173792bca3 | |
| parent | 9a34ee610bd864d39de49dc456078c7f335fc005 (diff) | |
Use run instead of popen so that things are closed correctly
Use run instead of popen so that things are closed correctly
| -rw-r--r-- | pywal/colors.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/pywal/colors.py b/pywal/colors.py index 9984b6e..4ed4b4b 100644 --- a/pywal/colors.py +++ b/pywal/colors.py @@ -18,12 +18,12 @@ def imagemagick(color_count, img): else: magick_command = ["convert"] - colors = subprocess.Popen([*magick_command, img, "-resize", "25%", - "+dither", "-colors", str(color_count), - "-unique-colors", "txt:-"], - stdout=subprocess.PIPE) + colors = subprocess.run([*magick_command, img, "-resize", "25%", + "+dither", "-colors", str(color_count), + "-unique-colors", "txt:-"], + stdout=subprocess.PIPE) - return colors.stdout.readlines() + return colors.stdout.splitlines() def gen_colors(img, color_count): |