Skip to content

Commit b9819a6

Browse files
committed
Merge pull request #9 from Eric-Guo/enable_windows_generation
Support windows platform letter avatar generation.
2 parents d53e4b1 + 2488324 commit b9819a6

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

lib/letter_avatar.rb

+8-3
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,14 @@ def self.generate(username, size)
4141

4242
def self.execute(cmd)
4343
cmd = cmd.join(' ') if cmd.is_a?(Array)
44-
pid, stdin, stdout, stderr = POSIX::Spawn.popen4(cmd)
45-
Process.waitpid(pid)
46-
err = stderr.read
44+
if Gem.win_platform?
45+
require "open3"
46+
_stdout_str, err = Open3.capture3(cmd.tr("'", '"'))
47+
else
48+
pid, _stdin, _stdout, stderr = POSIX::Spawn.popen4(cmd)
49+
Process.waitpid(pid)
50+
err = stderr.read
51+
end
4752
if err != nil && err.length > 0
4853
raise ExecutionError.new("letter_avatar execution error (when calling '#{cmd}'): '#{err.strip}'")
4954
else

0 commit comments

Comments
 (0)