Merge pull request #111 from linuxunil/master

Use run instead of popen so that things are closed correctly
This commit is contained in:
Dylan Araps 2017-09-25 08:42:35 +10:00 committed by GitHub
commit 637e31e9a8

View File

@ -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):